Thanks @AntonKhorev , I think this works well.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5362#issuecomment-2536543710
You are receiving this because you are subscribed to this thread.
Message ID:
___
Merged #5362 into master.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5362#event-15621756178
You are receiving this because you are subscribed to this thread.
Message ID:
___
rails-
This will only be configurable for logged in users, meaning that non-logged in
will still be stuck on the dark UI + dimmed tiles.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5362#issuecomment-2522096154
You are receiving this
@AntonKhorev commented on this pull request.
> @@ -516,6 +511,23 @@ body.small-nav {
}
}
+@mixin dark-map-color-scheme {
+ .leaflet-tile-container .leaflet-tile,
+ .mapkey-table-entry td:first-child > * {
+filter: brightness(.8);
Step 5 has an impossible requirement of "make it dark
@hlfan commented on this pull request.
> @@ -516,6 +511,23 @@ body.small-nav {
}
}
+@mixin dark-map-color-scheme {
+ .leaflet-tile-container .leaflet-tile,
+ .mapkey-table-entry td:first-child > * {
+filter: brightness(.8);
Doesn't the mentioned step 5 include the need for the filte
Includes #5339. Implements step 5 from
https://github.com/openstreetmap/openstreetmap-website/issues/5324#issuecomment-2504102012.
![image](https://github.com/user-attachments/assets/228d027d-2731-4c58-bf2c-0633e094660f)
![image](https://github.com/user-attachments/assets/0db27006-dd89-4dfd-8491-