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

CSS configuration #412

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

vogella
Copy link
Contributor

@vogella vogella commented Oct 23, 2020

Implementiert ein light CSS theme zum Testen.

Hierbei wurden die hard-codierten Farben von den folgenden Klassen
entfernt:

bundles/ch.elexis.core.ui/src/ch/elexis/core/ui/text/EnhancedTextField.java
bundles/ch.elexis.core.ui/src/ch/elexis/core/ui/views/HistoryDisplay.java
bundles/ch.elexis.core.ui/src/ch/elexis/core/ui/views/KonsDetailView.java
bundles/ch.elexis.core.ui/src/ch/elexis/core/ui/util/LabeledInputField.java

Hier muss noch geprüft werden, ob diese durch CSS Ids oder CSS Klassen
ersetzen werden müssen und entsprechend gestyled werden. Klickt man
durch die gesamte Applikation findet man noch weitere Fälle die nicht
korrekt gestyled sind, hier fehlte die Zeit diese auch noch zu finden zu
fixen. Dies sollte im Nachgang noch passieren.

Implementiert ein light CSS theme zum Testen.

Hierbei wurden die hard-codierten Farben von den folgenden Klassen
entfernt:

bundles/ch.elexis.core.ui/src/ch/elexis/core/ui/text/EnhancedTextField.java
bundles/ch.elexis.core.ui/src/ch/elexis/core/ui/views/HistoryDisplay.java
bundles/ch.elexis.core.ui/src/ch/elexis/core/ui/views/KonsDetailView.java
bundles/ch.elexis.core.ui/src/ch/elexis/core/ui/util/LabeledInputField.java

Hier muss noch geprüft werden, ob diese durch CSS Ids oder CSS Klassen
ersetzen werden müssen und entsprechend gestyled werden. Klickt man
durch die gesamte Applikation findet man noch weitere Fälle die nicht
korrekt gestyled sind, hier fehlte die Zeit diese auch noch zu finden zu
fixen. Dies sollte im Nachgang noch passieren.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant