Koala - Add new slide daily totals to top carousel#2860
Koala - Add new slide daily totals to top carousel#2860LKuemmel merged 14 commits intoopenWB:masterfrom
Conversation
packages/modules/web_themes/koala/source/src/components/models/daily-totals-model.ts
Outdated
Show resolved
Hide resolved
packages/modules/web_themes/koala/source/src/components/DailyTotals.vue
Outdated
Show resolved
Hide resolved
packages/modules/web_themes/koala/source/src/components/DailyTotals.vue
Outdated
Show resolved
Hide resolved
packages/modules/web_themes/koala/source/src/components/DailyTotals.vue
Outdated
Show resolved
Hide resolved
benderl
left a comment
There was a problem hiding this comment.
Nur ein paar kleinere Anpassungen, siehe Kommentare und Vorschläge.
Dann musst Du Dir aber nochmal die Darstellung im dunklen Theme ansehen. Die Hintergründe der einzelnen Zeilen sollten dann auch dunkler sein, damit das Erscheinungsbild einheitlich ist und der weiße Test mehr Kontrast zum Hintergrund bekommt.
packages/modules/web_themes/koala/source/src/components/DailyTotals.vue
Outdated
Show resolved
Hide resolved
packages/modules/web_themes/koala/source/src/components/DailyTotals.vue
Outdated
Show resolved
Hide resolved
packages/modules/web_themes/koala/source/src/components/DailyTotals.vue
Outdated
Show resolved
Hide resolved
benderl
left a comment
There was a problem hiding this comment.
Ich habe die Tabellenhintergrundfarben lokal in der DailyTotals-Komponente hinzugefügt und nicht global in der Quasar-SCSS-Datei. Der Grund ist, dass die Farben im Light Theme etwas heller und im Dark Theme etwas dunkler sind als die Standardfarben. Meiner Meinung nach sieht das bei größeren Farbflächen besser aus. Diese Farben gelten daher nur für diese eine Komponente.
Das sieht von den Farben her gut aus. Vom Code her solltest Du das aber anders umsetzen und die Farben im <style> Bereich definieren. Bei jeder Zeile hast Du bereits die "id" im Zugriff. Setz doch einfach eine CSS Klasse damit und definiere im Style z.B.:
.grid {
background: ...
}
.body--dark .grid {
background: ...
}
Dann sind die Farben nicht irgendwo im Code zu suchen, sondern sauber im CSS.
|
Verschiebe Tabelle CSS in den style-Bereich |


Folie zeigt die Gesamtenergie der Hauptkomponenten an. Leistungswerte werden zusätzlich auf Desktop-Größe angezeigt.

