@kcne commented on this pull request.
> + if (colorScheme === "auto") {
+ mediaQuery.addEventListener("change", (e) => {
+ const newTheme = e.matches ? "dark" : "light";
+ renderHeatmap(newTheme);
+ });
+ }
@hlfan Sorry about confusion here.
In the process of rebasing the commit for latest changes I forgot to pop
stashed changes so the file version didn't reflect last changes. Thanks for
catching that and pointing it out.
This is the updated version. It's working as expected in my firefox browser
locally now. Please let me know if there is something else I should modify
here.
Thank you.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5402#pullrequestreview-2549858967
You are receiving this because you are subscribed to this thread.
Message ID:
<openstreetmap/openstreetmap-website/pull/5402/review/2549858...@github.com>_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev
