[openstreetmap/openstreetmap-website] Fix element pagination width calculation (PR #5211)

Tom Hughes notifications at github.com
Fri Sep 13 17:07:18 UTC 2024


@tomhughes requested changes on this pull request.



> @@ -86,7 +86,7 @@ def sidebar_classic_pagination(pages, page_param)
     max_width_for_default_padding = 35
 
     width = 0
-    pagination_items(pages, {}).each do |body|
+    pagination_items(pages, {}).each do |(body)|

Just use `|body, _|` instead if you want to destructure it without an unused variable warning.

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

Message ID: <openstreetmap/openstreetmap-website/pull/5211/review/2303641221 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20240913/770e5dec/attachment.htm>


More information about the rails-dev mailing list