[openstreetmap/openstreetmap-website] Rename heatmap label to Contribution Calendar (PR #6932)
Pablo Brasero
notifications at github.com
Wed Mar 25 10:17:09 UTC 2026
pablobm left a comment (openstreetmap/openstreetmap-website#6932)
I think this is almost there. There's just the detail of the capitalisation. I think there has been both a misunderstanding and an ambiguity. Let's see if we can clarify all this.
1. At https://github.com/openstreetmap/openstreetmap-website/pull/6932#discussion_r2984771615, I think there's a misunderstanding by @Arpan200502. The comment is about how "heatmap" was replaced with "Contribution Calendar". In that specific instance, there's an inconsistency: it goes from non-capitalised to capitalised.
2. At https://github.com/openstreetmap/openstreetmap-website/pull/6932#discussion_r2984773994 however, there's an ambiguity. "Heatmap" is replaced with "Contribution Calendar", which is not necessarily wrong. We go from a single-word noun, capitalised, to a two-word noun whose capitalisation we haven't discussed: should it be capitalised as "Contribution Calendar" or as "Contribution calendar"?
Personally, I think it should be "Contribution Calendar" in all instances. This is even for the uncapitalised "heatmap" one which I think was a prior mistake as it refers to a distinct entity in the UI that should be capitalised, as opposed to a general term that could refer to many things.
Also: @Arpan200502, when offered feedback, please don't just accept it blindly. Stop for a moment, think about what you are being asked, and reason about what to do. If you think that some things should stay as they are, they do say so and explain your rationale. Part of the job of a Software Engineer is to push back when it makes sense.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6932#issuecomment-4125327195
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/6932/c4125327195 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20260325/3502fc38/attachment.htm>
More information about the rails-dev
mailing list