kcne left a comment (openstreetmap/openstreetmap-website#5998)

Even though I spent quite a bit of time trying to make our custom configuration 
work alongside cal-heatmap, I have to say - this approach makes a lot more 
sense. Modifying the library ended up requiring more code than writing the 
config itself.

I really like that this removes the third-party dependency and results in a 
much cleaner, more maintainable, and self-contained solution.

Just a quick question: what border-radius did you end up using for the squares? 
It seems to vary slightly across the three screenshots. Also, slightly 
increasing the spacing between columns and rows could help make the heatmap 
look a bit more balanced and visually appealing.

Thank you.

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

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

Reply via email to