Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
Looking at https://tastybackend.com/articles
It seems like there is a link there which really just shouldn't exist:
<div class="site-name visually-hidden">
<a href="/" title="Home" rel="home">Tasty Backend</a>
</div>
This is a bit confusing to keyboard only users and not really useful for anyone as it is redundant. The problem is in #6 which has focus, but has no reason to be there.
Found via https://accessibilityinsights.io/
Comment | File | Size | Author |
---|---|---|---|
#3 | remove-hidden-link-3050577-3.patch | 1007 bytes | fconnolly |
Screen Shot 2019-04-24 at 3.49.04 PM.png | 92.22 KB | mgifford |
Comments
Comment #2
mgiffordComment #3
fconnolly CreditAttribution: fconnolly at Mindgrub Technologies commentedShould be fixed. I'm not super familiar with twig or umami, so this could have had unintended consequences, but I checked it in my browser and ran the automated tests and it was ok.
Comment #4
fconnolly CreditAttribution: fconnolly at Mindgrub Technologies commentedComment #5
borisson_I think this looks good, it is a small fix and it improves a11y. +1.
Comment #6
alexpottClarifying that the title is not always "Tasty Backend" the title. It's whatever the user has set the site name to.
Comment #7
alexpottTested locally - works as advertised.
Committed and pushed 15d1f9e782 to 8.8.x and eb37d16c47 to 8.7.x. Thanks!
This part has nothing to do with the fix. Removed from commit.