diff --git a/LearningHub.Nhs.WebUI/Controllers/LoginWizardController.cs b/LearningHub.Nhs.WebUI/Controllers/LoginWizardController.cs index ab0d07dc..8f1b57a9 100644 --- a/LearningHub.Nhs.WebUI/Controllers/LoginWizardController.cs +++ b/LearningHub.Nhs.WebUI/Controllers/LoginWizardController.cs @@ -616,13 +616,9 @@ public async Task UpdateSecurityQuestionPost(MyAcountSecurityQues this.ModelState.AddModelError("DuplicateQuestion", CommonValidationErrorMessages.DuplicateQuestion); } - if (model.SelectedFirstQuestionId > 0 && string.IsNullOrEmpty(model.SecurityFirstQuestionAnswerHash)) + if ((model.SelectedFirstQuestionId > 0 && string.IsNullOrEmpty(model.SecurityFirstQuestionAnswerHash)) || (model.SelectedSecondQuestionId > 0 && string.IsNullOrEmpty(model.SecuritySecondQuestionAnswerHash))) { this.ModelState.AddModelError(nameof(model.SecurityFirstQuestionAnswerHash), CommonValidationErrorMessages.InvalidSecurityQuestionAnswer); - } - - if (model.SelectedSecondQuestionId > 0 && string.IsNullOrEmpty(model.SecuritySecondQuestionAnswerHash)) - { this.ModelState.AddModelError(nameof(model.SecuritySecondQuestionAnswerHash), CommonValidationErrorMessages.InvalidSecurityQuestionAnswer); } diff --git a/LearningHub.Nhs.WebUI/Controllers/MyAccountController.cs b/LearningHub.Nhs.WebUI/Controllers/MyAccountController.cs index c4f54901..ac3adc56 100644 --- a/LearningHub.Nhs.WebUI/Controllers/MyAccountController.cs +++ b/LearningHub.Nhs.WebUI/Controllers/MyAccountController.cs @@ -366,14 +366,9 @@ public async Task MyAccountSecurityQuestionsDetails([FromQuery] M return this.View("MyAccountSecurityQuestionsDetails", securityViewModel); } - if (viewModel.SelectedFirstQuestionId > 0 && string.IsNullOrEmpty(viewModel.SecurityFirstQuestionAnswerHash)) + if ((viewModel.SelectedFirstQuestionId > 0 && string.IsNullOrEmpty(viewModel.SecurityFirstQuestionAnswerHash)) || (viewModel.SelectedSecondQuestionId > 0 && string.IsNullOrEmpty(viewModel.SecuritySecondQuestionAnswerHash))) { this.ModelState.AddModelError(nameof(viewModel.SecurityFirstQuestionAnswerHash), CommonValidationErrorMessages.InvalidSecurityQuestionAnswer); - return this.View("MyAccountSecurityQuestionsDetails", securityViewModel); - } - - if (viewModel.SelectedSecondQuestionId > 0 && string.IsNullOrEmpty(viewModel.SecuritySecondQuestionAnswerHash)) - { this.ModelState.AddModelError(nameof(viewModel.SecuritySecondQuestionAnswerHash), CommonValidationErrorMessages.InvalidSecurityQuestionAnswer); return this.View("MyAccountSecurityQuestionsDetails", securityViewModel); } diff --git a/LearningHub.Nhs.WebUI/Styles/nhsuk/layout.scss b/LearningHub.Nhs.WebUI/Styles/nhsuk/layout.scss index 196c1e44..4efe8d08 100644 --- a/LearningHub.Nhs.WebUI/Styles/nhsuk/layout.scss +++ b/LearningHub.Nhs.WebUI/Styles/nhsuk/layout.scss @@ -96,6 +96,16 @@ body { gap: px2rem(24); } +.nhsuk-account__myaccount { + float: right; + position: relative; + z-index: 2; + display: flex; + justify-content: space-between; + align-items: center; + gap: px2rem(8); +} + .nhsuk-header__notification-dot { position: absolute; top: px2rem(8); diff --git a/LearningHub.Nhs.WebUI/Styles/nhsuk/nhsuk-overrides.scss b/LearningHub.Nhs.WebUI/Styles/nhsuk/nhsuk-overrides.scss index 22c31975..f98023fe 100644 --- a/LearningHub.Nhs.WebUI/Styles/nhsuk/nhsuk-overrides.scss +++ b/LearningHub.Nhs.WebUI/Styles/nhsuk/nhsuk-overrides.scss @@ -239,8 +239,8 @@ form label.nhsuk-u-visually-hidden { .nhsuk-header__notification-dot { position: absolute; - top: px2rem(8); - right: px2rem(-10); + top: px2rem(-8); + right: px2rem(76); font-size: px2rem(11); line-height: px2rem(18); font-weight: 900; @@ -453,8 +453,8 @@ form label.nhsuk-u-visually-hidden { .nhsuk-header__notification-dot { position: absolute; - top: px2rem(15); - left: px2rem(115); + top: px2rem(-8); + left: px2rem(16); font-size: px2rem(11); line-height: px2rem(18); font-weight: 900; diff --git a/LearningHub.Nhs.WebUI/Views/MyAccount/ChangePassword.cshtml b/LearningHub.Nhs.WebUI/Views/MyAccount/ChangePassword.cshtml index d68e84bc..0402064d 100644 --- a/LearningHub.Nhs.WebUI/Views/MyAccount/ChangePassword.cshtml +++ b/LearningHub.Nhs.WebUI/Views/MyAccount/ChangePassword.cshtml @@ -7,7 +7,7 @@ }
- +
diff --git a/LearningHub.Nhs.WebUI/Views/MyAccount/ConfirmPassword.cshtml b/LearningHub.Nhs.WebUI/Views/MyAccount/ConfirmPassword.cshtml index 888bafb5..cf34857c 100644 --- a/LearningHub.Nhs.WebUI/Views/MyAccount/ConfirmPassword.cshtml +++ b/LearningHub.Nhs.WebUI/Views/MyAccount/ConfirmPassword.cshtml @@ -8,7 +8,7 @@
- +
diff --git a/LearningHub.Nhs.WebUI/Views/MyAccount/MyAccountSecurityQuestionsDetails.cshtml b/LearningHub.Nhs.WebUI/Views/MyAccount/MyAccountSecurityQuestionsDetails.cshtml index e470e3c2..6fdcb560 100644 --- a/LearningHub.Nhs.WebUI/Views/MyAccount/MyAccountSecurityQuestionsDetails.cshtml +++ b/LearningHub.Nhs.WebUI/Views/MyAccount/MyAccountSecurityQuestionsDetails.cshtml @@ -8,7 +8,7 @@
- + diff --git a/LearningHub.Nhs.WebUI/Views/Shared/Components/NavigationItems/Topnav.cshtml b/LearningHub.Nhs.WebUI/Views/Shared/Components/NavigationItems/Topnav.cshtml index fae78987..00e95c14 100644 --- a/LearningHub.Nhs.WebUI/Views/Shared/Components/NavigationItems/Topnav.cshtml +++ b/LearningHub.Nhs.WebUI/Views/Shared/Components/NavigationItems/Topnav.cshtml @@ -8,7 +8,27 @@ @if (Model.ShowMyAccount) { + } @if (Model.ShowSignOut) { @@ -20,3 +40,22 @@ } *@ +@functions { + public string NotificationDisplay() + { + if (Model.NotificationCount < 1) + { + return ""; + } + + var returnString = Model.NotificationCount.ToString(); + + if (Model.NotificationCount > 9) + { + returnString = "9+"; + } + + return returnString; + + } +} \ No newline at end of file