Fixing TheTheme newly intruduced accessibility rules and HTML rules v…

…iolations (Lombiq Technologies: NEST-462) (OrchardCMS#14523)
dotnet-perf-bot · Oct 16, 2023 · df1a4e7 · df1a4e7
1 parent ef11011
commit df1a4e7
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/OrchardCore.Themes/TheTheme/Views/ToggleTheme.cshtml b/src/OrchardCore.Themes/TheTheme/Views/ToggleTheme.cshtml
@@ -1,5 +1,5 @@
-<li class="nav-item nav-item dropdown text-end">
-    <a class="nav-link dropdown-toggle" id="bd-theme" aria-expanded="false" data-bs-toggle="dropdown" data-bs-display="static" aria-label="@T["Toggle theme"]">
+<li class="nav-item dropdown text-end">
+    <a role="button" class="nav-link dropdown-toggle" id="bd-theme" aria-expanded="false" data-bs-toggle="dropdown" data-bs-display="static" aria-label="@T["Toggle theme"]">
         <span class="theme-icon-active"><i class="fa-solid fa-circle-half-stroke"></i></span>
         <span class="d-none" id="bd-theme-text">@T["Toggle theme"]</span>
     </a>

diff --git a/src/OrchardCore.Themes/TheTheme/Views/UserNotificationNavbar.cshtml b/src/OrchardCore.Themes/TheTheme/Views/UserNotificationNavbar.cshtml
@@ -9,7 +9,7 @@
 
 @model UserNotificationNavbarViewModel
 
-<li class="nav-item nav-item dropdown text-end">
+<li class="nav-item dropdown text-end">
     <div class="dropdown">
         <button type="button" class="nav-link dropdown-toggle" href="#" role="button" data-bs-toggle="dropdown" data-bs-auto-close="outside" aria-expanded="false">
             @if (Model.TotalUnread > 0)