[openstreetmap/openstreetmap-website] Refactor Heatmap: Manual Labels & Yearly Template (PR #5906)
Emin Kocan
notifications at github.com
Tue Apr 29 07:38:41 UTC 2025
kcne left a comment (openstreetmap/openstreetmap-website#5906)
I updated the code to manually render the labels in order to eliminate CSP warnings and remove the need for extra plugins. Please let me know if you prefer this manual approach or would rather stick with the plugin.
The main advantage of manual rendering is that it resolves the CSP issues, but it does add maintenance overhead and introduces new code that needs careful review and testing.
Using the label plugin is a lot cleaner and less code/maintenance burden, not being sure if there’s a way to address the CSP warnings, or if ignoring them would be acceptable in some cases. I’m open to feedback on direction and awaiting for reviews in either way. Thank you.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5906#issuecomment-2837810063
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/5906/c2837810063 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250429/1900beec/attachment-0001.htm>
More information about the rails-dev
mailing list