@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/[email protected]>
_______________________________________________
rails-dev mailing list
[email protected]
https://lists.openstreetmap.org/listinfo/rails-dev

Reply via email to