Since JS trims the heatmap data to align with the calendar-dependent start of 
the week, the point where monotonic month indices snap back a year can differ 
between calendars.

This misalignment desynchronizes month label indices and can cause month labels 
to be skipped.

The fix makes the label handling logic agnostic to yearly offsets.
You can view, comment on, or merge this pull request online at:

  https://github.com/openstreetmap/openstreetmap-website/pull/6725

-- Commit Summary --

  * Fix month label index desync causing missing labels

-- File Changes --

    M app/assets/javascripts/heatmap.js (4)

-- Patch Links --

https://github.com/openstreetmap/openstreetmap-website/pull/6725.patch
https://github.com/openstreetmap/openstreetmap-website/pull/6725.diff

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

Message ID: <openstreetmap/openstreetmap-website/pull/[email protected]>
_______________________________________________
rails-dev mailing list
[email protected]
https://lists.openstreetmap.org/listinfo/rails-dev

Reply via email to