kolibri
kolibri copied to clipboard
Implementation of Toolbar component
:ballot_box_with_check: Definition of Done checklist
- [ ] Meaningful title for the release notes
- [ ] Pull request is linked to a problem
- [ ] All changes relate to the problem
- [ ] A11y tests performed successfully or not relevant
- [ ] Manual test performed successfully (by reviewer) or not relevant
Netlify Draft Deployment URL: https://6650be90c52172171cce1d68--kolibri-public-ui.netlify.app Logs: https://app.netlify.com/sites/kolibri-public-ui/deploys/6650be90c52172171cce1d68
Vielleicht noch zwei Anmerkungen:
- Sollen wir die Links im BMF-Theme weiß gestalten?
- Lassen wir
aria-controls
erstmal weg oder sollen wir die noch umsetzen? Falls ja, über eineid
gesteuert oder direkt als Attribut_ariaControls
?
@chrisalat Ich würde die neuen Test-Routen gerne noch für das visuelle Testing aufnehmen. Siehe packages/tools/visual-tests/tests/sample-app.routes.js
. Dort eintragen und anschließend Snapshots erstellen.
@chrisalat Das disabled-Beispiel scheint mir nicht ganz richtig zu funktionieren:
Hier sind die Items 1 und 3 aktiv. Wenn ich 1 aktiviert habe, würde ich davon ausgehen, dass ich mit einmal Pfeiltaste rechts
drücken zu Item 3 komme. Aktuell muss ich zwei mal drücken.
@sdvg Laut dem Beispiel von W3C müssen die disabled Elemente trotzdem fokussiert werden. Meine erste Logik war tatsächlich dafür gedacht die Element zu überspringen. Hab ich aber dann wieder entfernt :)
@chrisalat Das disabled-Beispiel scheint mir nicht ganz richtig zu funktionieren:
Hier sind die Items 1 und 3 aktiv. Wenn ich 1 aktiviert habe, würde ich davon ausgehen, dass ich mit einmal `Pfeiltaste rechts` drücken zu Item 3 komme. Aktuell muss ich zwei mal drücken.