[openstreetmap/openstreetmap-website] Add user profile heatmap visualization for contributions (PR #5402)

Anton Khorev notifications at github.com
Fri Jan 31 13:13:36 UTC 2025


@AntonKhorev commented on this pull request.



> +      data: {
+        source: heatmapData,
+        type: "json",
+        x: "date",
+        y: "total_changes"
+      },
+      scale: {
+        color: {
+          type: "threshold",
+          range: currentTheme === "dark" ? rangeColors : Array.from(rangeColors).reverse(),
+          domain: [10, 20, 30, 40]
+        }
+      }
+    }, [
+      [Tooltip, {
+        text: (date, value) => getTooltipText(date, value, locale)

`getTooltipText` doesn't have the `locale` parameter. And you probably don't need to pass `locale` around to other places, given that you can read it from `I18n.locale`.

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

Message ID: <openstreetmap/openstreetmap-website/pull/5402/review/2586720269 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250131/0301081e/attachment.htm>


More information about the rails-dev mailing list