[openstreetmap/openstreetmap-website] Change colors of changeset bboxes when they enter/exit the viewport (PR #5924)
Tom Hughes
notifications at github.com
Sun Apr 13 10:12:22 UTC 2025
@tomhughes commented on this pull request.
Technically this looks so I think the main questions here will be whether (a) people think it's a good idea and (b) whether these are the best colour choices.
> @@ -1,15 +1,40 @@
OSM.HistoryChangesetsLayer = L.FeatureGroup.extend({
- _changesets: [],
+ newerColor: "#AA8866",
+ olderColor: "#8888AA",
If we're going to use named constants for these colours shouldn't we do it for the others as well?
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5924#pullrequestreview-2762728126
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/5924/review/2762728126 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250413/f0539fa1/attachment.htm>
More information about the rails-dev
mailing list