[openstreetmap/openstreetmap-website] Change heatmap color to linear from 0 to max changes (PR #5833)

Holger Jeromin notifications at github.com
Fri Mar 21 10:06:52 UTC 2025


@HolgerJeromin commented on this pull request.



> +          domain: [0, heatmapData.reduce((max, item) => {
+            return item.total_changes > max ? item.total_changes : max;
+          }, 0)]

Faster for 365 entries? I am not sure.
Definitely not worth the [ugly readability](https://medium.com/better-programming/think-again-before-you-use-array-reduce-28f785b5aea9).

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

Message ID: <openstreetmap/openstreetmap-website/pull/5833/review/2705389922 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250321/7f6c6e81/attachment.htm>


More information about the rails-dev mailing list