@hlfan commented on this pull request.
> + <i class="bi bi-star-fill fs-5 role-<%= role %> align-middle"
> title="<%= t ".revoke.#{role}" %>" aria-hidden="true"></i>
+ <span class="visually-hidden"><%= t ".revoke.#{role}" %></span>
Since this logic branch is only visible for admins, I'd drop the span and leave
the title for the tooltips.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6501#discussion_r2502374826
You are receiving this because you are subscribed to this thread.
Message ID:
<openstreetmap/openstreetmap-website/pull/6501/review/[email protected]>_______________________________________________
rails-dev mailing list
[email protected]
https://lists.openstreetmap.org/listinfo/rails-dev
