kolibri icon indicating copy to clipboard operation
kolibri copied to clipboard

Implementation of Toolbar component

Open chrisalat opened this issue 9 months ago • 1 comments

: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

chrisalat avatar May 08 '24 06:05 chrisalat

Netlify Draft Deployment URL: https://6650be90c52172171cce1d68--kolibri-public-ui.netlify.app Logs: https://app.netlify.com/sites/kolibri-public-ui/deploys/6650be90c52172171cce1d68

github-actions[bot] avatar May 08 '24 06:05 github-actions[bot]

Vielleicht noch zwei Anmerkungen:

  1. Sollen wir die Links im BMF-Theme weiß gestalten?
  2. Lassen wir aria-controls erstmal weg oder sollen wir die noch umsetzen? Falls ja, über eine id gesteuert oder direkt als Attribut _ariaControls?

chrisalat avatar May 22 '24 14:05 chrisalat

@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.

sdvg avatar May 24 '24 07:05 sdvg

@chrisalat Das disabled-Beispiel scheint mir nicht ganz richtig zu funktionieren:

image

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 avatar May 24 '24 07:05 sdvg

@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:

image 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.

chrisalat avatar May 24 '24 08:05 chrisalat