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

Disable dark-mode basemap #813

Closed
upintheairsheep opened this issue Sep 18, 2024 · 7 comments
Closed

Disable dark-mode basemap #813

upintheairsheep opened this issue Sep 18, 2024 · 7 comments

Comments

@upintheairsheep
Copy link

Hello, can you add an option to disable the altered dark mode base maps, for example Mapnik? It may be helpful for some users, but the low-quality dithering makes it unfavorable for others.

@bryceco
Copy link
Owner

bryceco commented Sep 18, 2024

What do you mean by low-quality dithering? It shouldn't be changing the resolution.

@upintheairsheep
Copy link
Author

What do you mean by low-quality dithering? It shouldn't be changing the resolution.
Not dithering, but it just looks a bit unclean compared to normal Mapnik/etc.
86039B92-A673-4019-B8D9-6480F68EA5EB

@bryceco
Copy link
Owner

bryceco commented Sep 18, 2024

Long-time commentators know I don't like configuration options, so if the brightness level is inappropriate we should fix it, rather than provide an option to disable it.

I don't use DarkMode myself so I'd prefer the community decide on a level of dimming that is agreeable to everyone. Currently it is set at -40% (or 60%) brightness. There are various online tools that will show you what other brightness levels look like (you can upload screenshots). Could you post examples of what you like and we can get comments from other users?

@bryceco
Copy link
Owner

bryceco commented Sep 18, 2024

#789

@verhovsky
Copy link
Contributor

It's not the amount of brightness, but that whatever you're doing to invert the colors is making text look bad, probably because it affects the blurriness around letters ("dithering") somehow, though admittedly it's already bad because open street map renders the map at half resolution to save money and looks terribly blurry on modern high pixel density displays.

image

image

@bryceco
Copy link
Owner

bryceco commented Sep 19, 2024

It's using a CoreImage filter that adjusts the brightness and nothing else, but that exposes the graininess of the underlying image.

filter.brightness = -0.4

@bryceco
Copy link
Owner

bryceco commented Sep 25, 2024

I've reduced the change in brightness. Hopefully that is sufficient.

@bryceco bryceco closed this as completed Sep 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants