Skip to content

Koala - Add new slide daily totals to top carousel#2860

Merged
LKuemmel merged 14 commits intoopenWB:masterfrom
Brett-S-OWB:daily-totals
Nov 6, 2025
Merged

Koala - Add new slide daily totals to top carousel#2860
LKuemmel merged 14 commits intoopenWB:masterfrom
Brett-S-OWB:daily-totals

Conversation

@Brett-S-OWB
Copy link
Contributor

Folie zeigt die Gesamtenergie der Hauptkomponenten an. Leistungswerte werden zusätzlich auf Desktop-Größe angezeigt.
Bildschirmfoto 2025-10-27 um 16 10 57
Bildschirmfoto 2025-10-27 um 16 11 19

Copy link
Contributor

@benderl benderl left a comment

Choose a reason for hiding this comment

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

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.

Image

@Brett-S-OWB
Copy link
Contributor Author

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.

Bildschirmfoto 2025-11-04 um 11 30 39 Bildschirmfoto 2025-11-04 um 11 30 51

Copy link
Contributor

@benderl benderl left a comment

Choose a reason for hiding this comment

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

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.

@Brett-S-OWB
Copy link
Contributor Author

Verschiebe Tabelle CSS in den style-Bereich
commit : Move table css to style section of component

@benderl benderl self-assigned this Nov 6, 2025
@benderl benderl added the enhancement New feature or request label Nov 6, 2025
@LKuemmel LKuemmel merged commit 930652a into openWB:master Nov 6, 2025
3 checks passed
@Brett-S-OWB Brett-S-OWB deleted the daily-totals branch December 2, 2025 07:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants