@tomhughes commented on this pull request.
Is there some easy way to reproduce this problem? or maybe you could add before
after images of what it fixes or something? It's hard to visualise what's going
on and how this fixes it at the moment...
> if (previousMonth === null) {
+ currentMonth += Math.round((monthLabelStartIndex -
getMonthOfThisWeek(date)) / 12) * 12;
Can we not use `currentMonth` here rather than calling the function a second
time?
```suggestion
currentMonth += Math.round((monthLabelStartIndex - currentMonth) / 12)
* 12;
```
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6725#pullrequestreview-3679488968
You are receiving this because you are subscribed to this thread.
Message ID:
<openstreetmap/openstreetmap-website/pull/6725/review/[email protected]>_______________________________________________
rails-dev mailing list
[email protected]
https://lists.openstreetmap.org/listinfo/rails-dev
