[openstreetmap/openstreetmap-website] Add headline to heatmap (PR #5976)
Tom Hughes
notifications at github.com
Sat May 3 13:41:15 UTC 2025
@tomhughes requested changes on this pull request.
Please squash at least the first three commits together - the tests can be a separate commit if you prefer or you can just squash everything into one.
> @@ -238,6 +238,11 @@
<div class="richtext text-break"><%= @user.description.to_html %></div>
<% if @heatmap_data.present? %>
+ <div class="row">
+ <div class="col text-body-secondary fs-5">
+ <%= t(".contributions.#{@changes_count == 1 ? 'one' : 'other'}", :changes_count => @changes_count) %>
If you call the variable in the translation `count` then you won't need to work out the subkeys yourself as rails i18n will do it for you...
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5976#pullrequestreview-2813235843
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/5976/review/2813235843 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250503/26ed3948/attachment.htm>
More information about the rails-dev
mailing list