Skip to content

Commit 958953e

Browse files
Merge pull request #959 from TechnologyEnhancedLearning/Develop/Fixes/TD-5299-Header-Page-Visually-hidden-content-is-in-the-focus-order
TD-5299: Removed focus on visually hidden content
2 parents 58a801a + 489cef3 commit 958953e

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

LearningHub.Nhs.WebUI/Views/Shared/Components/NavigationItems/Searchbar.cshtml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
<div class="nhsuk-header__break"></div>
1010
@if (Model.ShowSearch)
1111
{
12-
<input type="checkbox" id="header-mobile-search-control" aria-labelledby="close-search toggle-search">
12+
<input type="checkbox" tabindex="-1" id="header-mobile-search-control" aria-labelledby="close-search toggle-search">
1313
}
1414
<div class="nhsuk-header__search">
1515
@if (Model.ShowSearch)

LearningHub.Nhs.WebUI/Views/Shared/_NavPartial.cshtml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@
4949

5050
</div>
5151
@if (User.Identity.IsAuthenticated) {
52-
<input type="checkbox" id="header-dropdown-menu-control" aria-labelledby="Main navigation menu" role="button">
52+
<input type="checkbox" id="header-dropdown-menu-control" tabindex="-1" aria-labelledby="Main navigation menu" role="button">
5353
<nav class="nhsuk-header__navigation app-width-container" id="header-navigation" role="navigation" aria-label="Primary navigation" aria-labelledby="label-navigation">
5454
<p class="nhsuk-header__navigation-title"><span id="label-navigation">Menu</span>
5555
<button class="nhsuk-header__navigation-close" id="close-menu">

0 commit comments

Comments
 (0)