Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

🐞 Bug: nav: Default Theme muss valide gestyled werden. #5810

Closed
fohi17 opened this issue Dec 20, 2023 · 1 comment · Fixed by #5833
Closed

🐞 Bug: nav: Default Theme muss valide gestyled werden. #5810

fohi17 opened this issue Dec 20, 2023 · 1 comment · Fixed by #5833
Assignees
Labels
bug Something isn't working feature New feature or request

Comments

@fohi17
Copy link

fohi17 commented Dec 20, 2023

Note

  • Ich achte beim Erstellen des Issues darauf, dass ich keine fachlichen und unternehmensinternen Details preisgebe.

Fehlermeldung

Beschreibung des Fehlers

Die nav-Komponente hat noch mehrere Baustellen. Zunächst muss einmal das default Theme valide gestyled werden. Es treten bspw. folgende Probleme auf:

  • Es gibt keine visuelle Unterscheidung zwischen normalen, aktiven und ausklappbaren Einträgen.
  • Navigation mit reinem Text belegen sollte nicht möglich sein, wie es im horizontal Beispiel gemacht ist.
  • Active: Ist Evtl. hinfällig. Außerdem sind alle Einträge Buttons bis auf einen Link. Außerdem ist die Nav-Region mit "..." beschriftet. Daher ist das Beispiel nicht so gut.
  • Nice to have: Wenn ein Ausklappbarer Eintrag nicht selbst anklickbar ist, sollte der ganze Eintrag zum ausklappen verwendbar sein, wie es in der Kolibri-Dokus Navigation gemacht ist.

Reproduktion

Komponente: nav (https://public-ui.github.io/v2/sample-react/#/nav/basic)
Version: v2.0.0

@fohi17 fohi17 added the bug Something isn't working label Dec 20, 2023
@deleonio
Copy link
Contributor

deleonio commented Dec 20, 2023

  • Vollständig valides Nav-Styling muss für Default-Theme umgesetzt werden.
  • Es gibt keine Text-Menüpunkte mehr
  • Aufklappmenüs ohne eigene Seite - sind ein Schalter aus Label und Aufklapp-Icon
  • Hat der Menüpunkt eine eigene Seite sind Link und Expand-Button getrennt.
  • Wenn man sich auf einem Menüpunkt befindet (aktiv), dann ist das Untermenü, falls vorhanden, aufgeklappt.

image

@deleonio deleonio moved this to 📄 Ready in KoliBri Board Dec 20, 2023
@deleonio deleonio added the feature New feature or request label Dec 20, 2023
@sdvg sdvg moved this from 📄 Ready to 🏗 In progress in KoliBri Board Dec 21, 2023
@sdvg sdvg self-assigned this Dec 21, 2023
@sdvg sdvg linked a pull request Dec 22, 2023 that will close this issue
@deleonio deleonio added this to the BIK BITV-Test '23 milestone Jan 5, 2024
@sdvg sdvg moved this from 🏗 In progress to 🔍Review in KoliBri Board Jan 5, 2024
@github-project-automation github-project-automation bot moved this from 🔍Review to ✅ Done in KoliBri Board Jan 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working feature New feature or request
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants