-
-
Notifications
You must be signed in to change notification settings - Fork 31
Open
Labels
good first issueGood for newcomersGood for newcomers🐛 bugSomething isn't workingSomething isn't working
Description
Describe the bug
This was reported here.
Looks really cool, but doesn't feel like a website. Any reason why hyperlinks are marked up as
? Also, why do links that lead to a different page trigger a scroll up animation on the same page while page contents changes? It feels disorienting.
Looks like there are few places where we are doing Button
or Link
is rendering a <div href/>
. Opening this issue for tracking reasons.
I have not yet triaged this myself, but we will want to make sure this site passes all accessibility tests now that it is live. This was an oversight on my part.
Steps to reproduce
Run the lighthouse on the site and report issues here. Depending on scope, we can open separate issues for fix in one PR.
Affected services
opensauced.pizza
Platforms
No response
Browsers
No response
Environment
No response
Additional context
No response
Code of Conduct
- I agree to follow this project's Code of ConductTo pick up a draggable item, press the space bar. While dragging, use the arrow keys to move the item. Press space again to drop the item in its new position, or press escape to cancel.
Contributing Docs
- I agree to follow this project's Contribution DocsTo pick up a draggable item, press the space bar. While dragging, use the arrow keys to move the item. Press space again to drop the item in its new position, or press escape to cancel.
Metadata
Metadata
Assignees
Labels
good first issueGood for newcomersGood for newcomers🐛 bugSomething isn't workingSomething isn't working
Activity
takanome-dev commentedon Dec 14, 2022
All the buttons except the
subscribe newsletter
aredivs
withhref
.I ran lighthouse on Firefox and Brave and it got pretty good score imh.
Here are some screenshots:
Brave Desktop
Brave Modile
Firefox mobile
Common for both on mobile
bdougie commentedon Jan 3, 2023
I think the focus for this issue should be improving the accessibility score. If updating the links does not improve the score, we should avoid it. I am updating the title to reflect this.
[-]Bug: <div href> [/-][+]Bug: Improve lighthouse accessibility score.[/+]CBID2 commentedon Aug 11, 2023
@EmmaDawsonDev, I think you can share some great insights on this.
EmmaDawsonDev commentedon Aug 11, 2023
Definitely happy to take a look and do an accessibility audit. I'd prioritise fixing the links/buttons over raising the lighthouse score as that's something critical for keyboard users to access the site and not something lighthouse can pick up. But I'm sure there's a lot of small tweaks that can make a big difference
dev-phantom commentedon Sep 17, 2023
.take
github-actions commentedon Sep 17, 2023
Thanks for taking this on! If you have not already, join the conversation in our Discord
shiv343 commentedon Jun 6, 2025
is this issue still open ?