@AntonKhorev commented on this pull request.


> +    <div class="col overflow-auto">
+      <div id="cal-heatmap-wrapper" class="heatmap-wrapper">
+        <!-- Labels -->
+        <ul class="list-unstyled d-flex flex-column justify-content-between 
ch-domain-text heatmap-labels">
+          <li>&nbsp;</li>
+          <li><%= t("date.abbr_day_names")[1] %></li>
+          <li>&nbsp;</li>
+          <li><%= t("date.abbr_day_names")[3] %></li>
+          <li>&nbsp;</li>
+          <li><%= t("date.abbr_day_names")[5] %></li>
+          <li>&nbsp;</li>
+        </ul>
+        <!-- Heatmap -->
+        <div id="cal-heatmap" class="heatmap"
+          data-heatmap="<%= @heatmap_data.to_json %>"
+          data-site_color_scheme="<%= preferred_color_scheme(:site) %>">

The color scheme is already present in `data-bs-theme` on `<html>` ("auto" if 
that attribute is missing).

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

Message ID: 
<openstreetmap/openstreetmap-website/pull/5402/review/2593626...@github.com>
_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev

Reply via email to