[openstreetmap/openstreetmap-website] Forward preferred color scheme to id-container (PR #6323)
Martin Raifer
notifications at github.com
Fri Sep 19 15:25:39 UTC 2025
@tyrasd commented on this pull request.
> + data[:theme] = (preferred_color_scheme(:site) if preferred_color_scheme(:site) != "auto")
data[:asset_map] = assets("iD").to_json %>
<%= tag.div "", :id => "id-container", :data => data %>
ooh, sorry, I totally overlooked that comment thread. :see_no_evil:
The missing `.theme()` method has been added with https://github.com/openstreetmap/iD/commit/f7cd173b0fb594d2a896095a65609cb604df45c7
But now this PR really has to wait for the next release, as the `.theme` is not available in 2.36. But that's fine, I'll simply include the changes here in my release PR for the upcoming 2.37. Feel free to close this PR in the meantime @hlfan
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6323#discussion_r2363287023
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/6323/review/3245453092 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250919/3b07ca05/attachment.htm>
More information about the rails-dev
mailing list