Skip to content

Commit 2482ac7

Browse files
authored
Merge pull request #2732 from TechnologyEnhancedLearning/Develop/Fixes/TD-4137-'Switchapplication'navigationlinkshouldberemovedfrom'Givepagefeedback'screensforloggedinuser
TD-4137 switch application navigation link should be removed from give page feedback screens for logged in user
2 parents 3b50338 + 0dec788 commit 2482ac7

File tree

2 files changed

+10
-7
lines changed

2 files changed

+10
-7
lines changed

DigitalLearningSolutions.Web/Controllers/UserFeedbackController.cs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,6 @@ IMultiPageFormService multiPageFormService
3737
public IActionResult Index(string sourceUrl, string sourcePageTitle)
3838
{
3939
ViewData[LayoutViewDataKeys.DoNotDisplayUserFeedbackBar] = true;
40-
4140
_multiPageFormService.ClearMultiPageFormData(MultiPageFormDataFeature.AddUserFeedback, TempData);
4241

4342
_userFeedbackViewModel = new()

DigitalLearningSolutions.Web/Views/Shared/_Layout.cshtml

Lines changed: 10 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -138,12 +138,16 @@
138138

139139
@if (User.IsAdminAccount())
140140
{
141-
<li class="nhsuk-header__navigation-item @Html.GetSelectedCssClassIfTabSelected(NavMenuTab.SwitchApplication)">
142-
<a class="nhsuk-header__navigation-link" asp-controller="ApplicationSelector" asp-action="Index">
143-
Switch application
144-
<partial name="_NhsChevronRight" />
145-
</a>
146-
</li>
141+
if ((bool?)ViewData[LayoutViewDataKeys.DoNotDisplayUserFeedbackBar] != true)
142+
{
143+
<li class="nhsuk-header__navigation-item @Html.GetSelectedCssClassIfTabSelected(NavMenuTab.SwitchApplication)">
144+
<a class="nhsuk-header__navigation-link" asp-controller="ApplicationSelector" asp-action="Index">
145+
Switch application
146+
<partial name="_NhsChevronRight" />
147+
</a>
148+
</li>
149+
}
150+
147151
}
148152
</ul>
149153
</div>

0 commit comments

Comments
 (0)