[openstreetmap/openstreetmap-website] Editor color mode preference (PR #6492)

Marwin Hochfelsner notifications at github.com
Mon Nov 3 19:19:10 UTC 2025


@hlfan commented on this pull request.



> @@ -13,7 +13,7 @@
      data[:token] = token.token
    end
    data[:locale] = ID::LOCALES.preferred(preferred_languages).to_s
-   data[:theme] = (preferred_color_scheme(:site) if preferred_color_scheme(:site) != "auto")
+   data[:theme] = (preferred_color_scheme(:editor) if preferred_color_scheme(:editor) != "auto") || (preferred_color_scheme(:site) if preferred_color_scheme(:site) != "auto")

I was able to reduce quite a bit of complexity by rewriting preferred_color_scheme, but now I'm unsure where to put a test. If you have any ideas...

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6492#discussion_r2487592075
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/openstreetmap-website/pull/6492/review/3412543337 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20251103/fdec06b2/attachment.htm>


More information about the rails-dev mailing list