Skip to content

Commit

Permalink
Merge pull request #486 from trln/TD-1399-masthead-fix
Browse files Browse the repository at this point in the history
Fix masthead a11y warning; remove duplicate link. Fixes TD-1399.
  • Loading branch information
kazymovae authored Dec 18, 2024
2 parents 73d159a + 6bb3238 commit b2b41a3
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions app/views/shared/_header_navbar.html.erb
Original file line number Diff line number Diff line change
Expand Up @@ -3,11 +3,11 @@

<nav class="navbar navbar-expand-md navbar-light bg-light topbar" role="navigation" aria-label="Masthead">
<div class="<%= container_classes %>">
<%= link_to application_name, root_path, class: 'mb-0 navbar-brand navbar-logo' %>
<%= link_to inst_long, root_path, class: 'mb-0 navbar-brand navbar-logo' %>

<button class="navbar-toggler navbar-toggler-right" type="button" data-bs-toggle="collapse" data-bs-target="#user-util-collapse" aria-controls="user-util-collapse" aria-expanded="false" aria-label="Toggle navigation">
<span class="navbar-toggler-icon"></span>
</button>
<%= link_to content_tag(:span, inst_long, class: "visually-hidden"), root_path, class: "navbar-brand" %>

<div class="collapse navbar-collapse justify-content-md-end" id="user-util-collapse">
<%= render 'shared/user_util_links' %>
Expand Down

0 comments on commit b2b41a3

Please sign in to comment.