diff options
author | Dirk Hohndel <dirk@hohndel.org> | 2020-11-04 13:09:44 -0800 |
---|---|---|
committer | Dirk Hohndel <dirk@hohndel.org> | 2020-11-04 14:47:10 -0800 |
commit | 6201ac34b412eb57e0ca03401da22ed9ae683c38 (patch) | |
tree | 5d41d1f3b69021918e1c74cef27f0cdd55e27c0f /desktop-widgets/tab-widgets/maintab.cpp | |
parent | 4c6b00f812757508118deee286886243485b3142 (diff) | |
download | subsurface-6201ac34b412eb57e0ca03401da22ed9ae683c38.tar.gz |
desktop: automatically pick a good theme title color
The preference setting seemed far too strange to do this. And not very user
friendly. So instead we figure out if this is a dark theme or not by looking at
text and background colors in the palette, and make sure we get notified if
that changes.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
Diffstat (limited to 'desktop-widgets/tab-widgets/maintab.cpp')
-rw-r--r-- | desktop-widgets/tab-widgets/maintab.cpp | 30 |
1 files changed, 25 insertions, 5 deletions
diff --git a/desktop-widgets/tab-widgets/maintab.cpp b/desktop-widgets/tab-widgets/maintab.cpp index ab4b4f28b..6e794f07d 100644 --- a/desktop-widgets/tab-widgets/maintab.cpp +++ b/desktop-widgets/tab-widgets/maintab.cpp @@ -47,6 +47,12 @@ struct Completers { QCompleter *tags; }; +static bool paletteIsDark(const QPalette &p) +{ + // we consider a palette dark if the text color is lighter than the windows background + return p.window().color().valueF() < p.windowText().color().valueF(); +} + MainTab::MainTab(QWidget *parent) : QTabWidget(parent), editMode(false), ignoreInput(false), @@ -72,6 +78,9 @@ MainTab::MainTab(QWidget *parent) : QTabWidget(parent), extraWidgets << new TabDiveComputer(this); ui.tabWidget->addTab(extraWidgets.last(), tr("Device names")); + // make sure we know if this is a light or dark mode + isDark = paletteIsDark(palette()); + // call colorsChanged() for the initial setup now that the extraWidgets are loaded colorsChanged(); @@ -704,13 +713,24 @@ void MainTab::clearTabs() widget->clear(); } +void MainTab::changeEvent(QEvent *ev) +{ + if (ev->type() == QEvent::PaletteChange) { + // check if this is a light or dark mode + bool dark = paletteIsDark(palette()); + if (dark != isDark) { + // things have changed, so setup the colors correctly + isDark = dark; + colorsChanged(); + } + } + QTabWidget::changeEvent(ev); +} + // setup the colors of 'header' elements in the tab widget void MainTab::colorsChanged() { - // Put together appropriate CSS stylesheets: NB: colors below in same order as the enum in prefs.h - QStringList colors = { "mediumblue", "lightblue", "black" }; // If using dark theme, set color appropriately - QString colorText = colors[prefs.headerstyle_color]; - + QString colorText = isDark ? QStringLiteral("lightblue") : QStringLiteral("mediumblue"); QString lastpart = colorText + " ;}"; // only set the color if the widget is enabled @@ -731,5 +751,5 @@ void MainTab::colorsChanged() // finally call the individual updateUi() functions so they can overwrite these style sheets for (TabBase *widget: extraWidgets) - widget->updateUi(); + widget->updateUi(colorText); } |