moneymanagerex icon indicating copy to clipboard operation
moneymanagerex copied to clipboard

Toolbar is wrong colour in dark mode

Open tactilis opened this issue 2 years ago • 8 comments

MMEX version:

  • [x] 1.5.18 Beta
  • [x] 1.5.19 Beta

Operating System:

  • [x] Windows
  • [ ] Mac OSX
  • [ ] Linux

Description of the bug

As reported here https://forum.moneymanagerex.org/viewtopic.php?p=23191 the toolbar colour is not correct in dark mode.

image

Reproduction

Is the bug reproducible?

  • [x] Always
  • [ ] Randomly
  • [ ] Happened only once

Reproduction steps:

  1. Select dark mode in 1.5.18 (or 1.5.19 Beta)
  2. Restart MMEX

Expected result:

Toolbar should be dark.

Actual result:

See screenshot

tactilis avatar Aug 04 '22 07:08 tactilis

Whilst there don't seem to be any issues on MacOS dark mode, this change may help.

whalley avatar Aug 04 '22 12:08 whalley

Whilst there don't seem to be any issues on MacOS dark mode, this change may help.

I've tested this. It does not help.

vomikan avatar Aug 04 '22 12:08 vomikan

Agreed, it's still the same.

The question is, what changed to the handling of dark mode between 1.5.13, where it clearly worked, to 1.5.18.

I don't use dark mode, so I never noticed the release version (or beta), where this happened.

tactilis avatar Aug 04 '22 13:08 tactilis

The question is, what changed to the handling of dark mode between 1.5.13, where it clearly worked, to 1.5.18.

The toolbar was "wxToolBar" then became "wxAuiToolBar".

vomikan avatar Aug 04 '22 13:08 vomikan

Yes, was changed here... https://github.com/moneymanagerex/moneymanagerex/issues/4791

Some discussion wxAuiToolBar here: https://stackoverflow.com/questions/23366944/how-to-set-wxauitoolbar-background

whalley avatar Aug 04 '22 13:08 whalley

One more issue. The root item of the wxListCtrl has black font color. image

~~It seems was an separate bug report but I can't find it.~~

https://github.com/moneymanagerex/themes/issues/2

vomikan avatar Aug 04 '22 13:08 vomikan

Admittedly, even if you roll back the changes https://github.com/whalley/moneymanagerex/commit/b00ba2c6a38fbf1ab2d657515eba43e7c579d8df, the dark theme on Windows looks bad.

image

vomikan avatar Aug 04 '22 13:08 vomikan

Removing myself from this one as there seem to be quite a few 'Dark Mode' restrictions on Windows and I've not really got the time to evolve this on Windows.

whalley avatar Aug 05 '22 07:08 whalley