@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/3245453...@github.com>
_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev

Reply via email to