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

Add dark mode to status bar #432

Merged
merged 3 commits into from
Jan 24, 2022
Merged

Add dark mode to status bar #432

merged 3 commits into from
Jan 24, 2022

Conversation

reupen
Copy link
Owner

@reupen reupen commented Jan 23, 2022

This:

  • adds a dark mode to the status bar
  • reworks much of the status bar layout logic
  • replaces the status bar playlist lock icon with the 🔒 character

The status bar is, unfortunately, another control that has no native dark mode, and therefore some workarounds are used to get something acceptable working.

Note also that currently, the 🔒 character doesn't seem to work out of the box on Windows 7 (Segoe UI Symbol isn't linked to Segoe UI there, at least in the VM I'm using).

This:

- adds a dark mode to the status bar
- reworks much of the status bar layout logic
- replaces the status bar playlist lock icon with the 🔒 character

(Currently, the 🔒 character doesn't work out of the box on Windows 7.)
@reupen reupen marked this pull request as ready for review January 24, 2022 20:38
@reupen reupen merged commit b093cb9 into master Jan 24, 2022
@reupen reupen deleted the status-bar-dark-mode branch January 24, 2022 21:39
@marc2k3
Copy link
Contributor

marc2k3 commented Mar 5, 2022

(Segoe UI Symbol isn't linked to Segoe UI there, at least in the VM I'm using).

Amusingly (or not!) it does work when using DirectWrite.

image

I was disappointed to see it was still monochrome in Windows 8.1. I thought that was the minimum version required for colour fonts but it seems support is rather limited.

It needs Windows 10 to be properly coloured...

image

@reupen
Copy link
Owner Author

reupen commented Mar 5, 2022

Amusingly (or not!) it does work when using DirectWrite.

Now that's interesting. When I wrote that comment, I also tried editing the GDI font linking registry key but even that didn't work. (Perhaps it doesn't handle code points that are more than one code unit correctly 🤷‍♂️ )

Now there is a long-standing issue about migrating to DirectWrite, though I was minded to see if I can get Uniscribe to fall back to Segoe UI Symbol to fix that character (if it's easy).

I was disappointed to see it was still monochrome in Windows 8.1. I thought that was the minimum version required for colour fonts but it seems support is rather limited.

Sounds like the problem is the font (Segoe UI Emoji) there.

@marc2k3
Copy link
Contributor

marc2k3 commented Mar 5, 2022

Seems like this particular icon being monochrome in 8.1 (and earlier versions of 10) was deliberate. I found this reference which shows the history and it's different again in 11....

image

https://emojipedia.org/locked/

@reupen
Copy link
Owner Author

reupen commented Mar 7, 2022

Hmm, that is a lot of change – not sure I'm keen on the Windows 11 one to be honest, looks like an odd shopping bag or something to me 🤷

@marc2k3
Copy link
Contributor

marc2k3 commented Mar 8, 2022

And here it is...

image

Also see the white blobs in the bottom corners of your nice dark menu. I guess that's the price to pay with 11 applying rounded corners to everything. /sad face

Even more fun, notification area tooltips getting squeezed in to an insanely low maximum width...

image

@reupen
Copy link
Owner Author

reupen commented Mar 8, 2022

And here it is...

Ugh, that also looks like they've ruined the Segoe UI Symbol version (l mean it looks unrecognisable to me). I thought the emoji would be better and easier to maintain but I may have to go back to using an icon (which means creating one, or finding one with an appropriate licence, as the old one was a 16x16 bitmap...)

Also see the white blobs in the bottom corners of your nice dark menu. I guess that's the price to pay with 11 applying rounded corners to everything. /sad face

Hmm, that's odd. My laptop doesn't seem to be doing it but I saw it in a VM for a little while (is now gone there as well). No idea why that would happen, but I'll have to investigate as the dark menu stuff is all undocumented...

Also noted that the Windows Explorer dark mode colours are different on Windows 11 (compared to Windows 10) so I'll probably need to do something about that too...

Edit: Looks like the menu problem has also been reported for Notepad++: notepad-plus-plus/notepad-plus-plus#10697

Even more fun, notification area tooltips getting squeezed in to an insanely low maximum width...

image

Not just that, ampersands are completely messed up too... Needs to go into the Feedback Hub (if someone hasn't already reported it...)

@reupen reupen mentioned this pull request Mar 28, 2022
43 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants