public-ui / kolibri

The accessible HTML-Standard
https://public-ui.github.io
European Union Public License 1.2
173 stars 33 forks source link

Changes CSS classes to BEM 2 #7083

Open AlexanderSchmidtCE opened 3 days ago

AlexanderSchmidtCE commented 3 days ago

Enthält ButtonLink, Button The A11y and PO reviews will only take place after all other DoD steps have been completed by the Developer:

github-actions[bot] commented 3 days ago

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

sdvg commented 3 days ago

@deleonio Richtig, das Merge-Target ist falsch. Dieses PR sollte nur in develop (V3) gemerged werden.

Lösung: Merge Target ändern, oder wenn es (wie so oft) Probleme macht, PR schließen und neu anlegen. v2-Label anschließend entfernen.

deleonio commented 3 days ago

@sdvg ich hab den Branch auf release/2 gestellt, weil es schon einen PR für develop gibt: https://github.com/public-ui/kolibri/pull/7063

sdvg commented 3 days ago

@deleonio Die beiden PRs sollten verschiedene Inhalte haben bzw. sich um verschiedene Komponenten kümmern. Es sind praktisch Part 1 und Part 2.

@AlexanderSchmidtCE Ich habe beide PRs mal zurück auf Draft gestellt, weil sie ja noch nicht ganz bereit für ein finales Review sind.