ithaka / pharos

JSTOR's design system
https://pharos.jstor.org
MIT License
119 stars 15 forks source link

Card: Improve accessibility by moving overlay slot outside of <a> tag #766

Closed ymouzakis closed 3 months ago

ymouzakis commented 3 months ago

This change: (check at least one)

Is this a breaking change? (check one)

Is the: (complete all)

What does this change address? Fixes a problem where the button inside of the overlay slot would not be picked up by the browser when tabbing through clickable elements. I believe this is because the button within the overlay was slotted inside of an anchor tag

How does this change work? Move overlay slot outside of the link tag

Additional context Add any other context here.

changeset-bot[bot] commented 3 months ago

🦋 Changeset detected

Latest commit: f5b49e3439aa05d0832eda8e275c05fef5be4996

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package | Name | Type | | -------------- | ----- | | @ithaka/pharos | Patch |

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

github-actions[bot] commented 3 months ago

size-limit report 📦

Path Size
packages/pharos/lib/index.js 64.44 KB (-0.06% 🔽)
daneah commented 3 months ago

@ymouzakis can you please update the PR description? Some information is missing and it seems like your commit message may have been truncated at the top there.