@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/2813235...@github.com>
_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev

Reply via email to