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

Vorschlag zur Verbesserung der CSS Wartbarkeit #5544

Open
deleonio opened this issue Nov 9, 2023 · 3 comments · May be fixed by #5589
Open

Vorschlag zur Verbesserung der CSS Wartbarkeit #5544

deleonio opened this issue Nov 9, 2023 · 3 comments · May be fixed by #5589
Assignees
Labels
v3 Here are issues that need to be resolved for version 3.

Comments

@deleonio
Copy link
Contributor

deleonio commented Nov 9, 2023

Problem: Einzelne Selektoren und nicht klar und fragil. Beispiel: :host > div > div > .close

  • Welche div-Elemente sind hier gemeint? Warum müssen sie Teil des Selektors sein?
  • Was passiert, wenn die HTML-Struktur geändert wird?

Lösung: Ein Theme aussuchen (z.B. Default) und konsequent auf Klassen-Selektoren refactoren. Benötigte Klassen dabei in den Components ergänzen.

Problem: Die Theme-Component-Styles sind in einer sehr großen TS-Datei schwer zu überblicken. Es ist schwer zu erkennen, wo eine Komponente anfängt und endet und die Editor-Navigation kann lästig sein.
Lösung: Themes nach Komponenten in Module splitten. TS oder direkt CSS (siehe auch Punkt 3).

Problem: Styles sind dupliziert. Wenn z.B. allgemeine Input-Styles angepasst werden, müssen diese in bis zu 12 Komponenten angepasst werden. Das macht es auch schwierig herauszuarbeiten, wenn Komponenten zusätzlich eigene Styles haben, welche die allgemeinen Input-Styles ergänzen oder überschreiben.
Lösung: CSS-Preprocessor wie z.B. PostCSS mit mixins verwenden.

@sdvg sdvg self-assigned this Nov 9, 2023
@sdvg sdvg linked a pull request Nov 14, 2023 that will close this issue
@sdvg
Copy link
Member

sdvg commented Nov 14, 2023

Ticket in "Review" bedeutet: Initiales Beispiel ist im PR und kann angeschaut und besprochen werden.

@deleonio
Copy link
Contributor Author

@sdvg haben wir das nicht schon komplett?

@deleonio
Copy link
Contributor Author

Hierbei sollten wir uns auch die Herangehensweise von @mfranzke mal anschauen.

Link: db-ui/mono#2152 (comment)

@laske185 laske185 added the v3 Here are issues that need to be resolved for version 3. label May 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
v3 Here are issues that need to be resolved for version 3.
Projects
Status: Backlog
Development

Successfully merging a pull request may close this issue.

3 participants