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

Markup and styling #169

Merged
merged 19 commits into from
May 28, 2024
Merged

Markup and styling #169

merged 19 commits into from
May 28, 2024

Conversation

tbaddade
Copy link
Member

@tbaddade tbaddade commented May 27, 2024

  • Einheitliches Markup für Dropdown
  • Etwas Styling

@skerbis Unbedingt testen. Weiß nicht ob das Js noch funktioniert. Du kennst dich da besser aus, wann wo da was passieren soll.

@tbaddade tbaddade requested a review from skerbis May 27, 2024 20:09
@skerbis
Copy link
Member

skerbis commented May 27, 2024

Alle Menüs sind geöffnet nach der Installation und bei jedem Seitenwechsel
Bildschirmfoto 2024-05-27 um 22 12 16

Copy link
Member

@skerbis skerbis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ich würde die Beschreibungstexte (Label) nur optional immer anzeigen. Das nimmt nun sehr viel Platz in Anspruch. Es bleibt kaum Platz für individuelle Buttons und auch mobil wird es eng

@skerbis
Copy link
Member

skerbis commented May 28, 2024

Der aktive Link in der Struktur wird nicht mehr angesprungen fixed: 87710b1

Und die Suche ist nicht mehr fixiert
@tbaddade

@skerbis skerbis merged commit f7b625b into main May 28, 2024
@skerbis skerbis deleted the markup-and-styling branch May 28, 2024 18:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants