Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
petergoes
force-pushed
the
test--button-stories-for-attributes
branch
from
76f6229 to
0721842
Compare
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #959 +/- ##
==========================================
Coverage 100.00% 100.00%
==========================================
Files 28 14 -14
Lines 310 119 -191
Branches 52 30 -22
==========================================
- Hits 310 119 -191 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
| parameters: { | ||
| docs: { | ||
| description: { | ||
| story: 'De toegankelijke naam van de button is `Klik mij via aria-label` inplaats van `Klik mij!`', |
There was a problem hiding this comment.
Ik zou heel graag in onze Storybooks een realistisch voorbeeld hebben dat best practices volgt, bijvoorbeeld Verwijderen en aria-label="Verwijderen: preview.pdf"
| render: (props) => ( | ||
| <> | ||
| <p id="label">button label</p> | ||
| <Button {...props} aria-labelledby="label" /> |
There was a problem hiding this comment.
Dit volgt niet de WCAG best practice van "Label in naam". Misschien een leuke om ook "Verwijderen" en "preview.pdf" te combineren in een tabelletje of iets dergelijks? Met aria-labelledby="button-id filename-id"
3 participants
closes: #833