[openstreetmap/openstreetmap-website] Fix overlap between left sidebar heading and close button (PR #5790)

Anton Khorev notifications at github.com
Tue Mar 11 09:11:16 UTC 2025


Sticky close buttons have a layer that covers their shadows when sidebars are not scrolled. That layer is slightly larger than it needs to be. It overlaps with the heading and can block out letters in the heading. I'm making the cover layer smaller in this PR.

Before:
![image](https://github.com/user-attachments/assets/51aec26c-b673-4624-90dc-5ef25b87ce82)
![image](https://github.com/user-attachments/assets/caded515-8ba3-4382-8f11-eec9f3817912)

After:
![image](https://github.com/user-attachments/assets/24dd4f19-da10-4d30-8ec5-3c5001c5dde0)
![image](https://github.com/user-attachments/assets/fcb39814-069e-42f1-b593-85e404751e81)

Actually with this PR the cover layer is slightly smaller than it needs to be to fully block the shadow, but hopefully it's unnoticeable. If it becomes a problem, I'll have to set sizes directly in css, currently they are set in Bootstrap units. There's also a tradeoff between not having this overlap and the heading width, given that the heading content is variable.
You can view, comment on, or merge this pull request online at:

  https://github.com/openstreetmap/openstreetmap-website/pull/5790

-- Commit Summary --

  * Fix overlap between left sidebar heading and close button

-- File Changes --

    M app/views/layouts/_sidebar_close.html.erb (2)

-- Patch Links --

https://github.com/openstreetmap/openstreetmap-website/pull/5790.patch
https://github.com/openstreetmap/openstreetmap-website/pull/5790.diff

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

Message ID: <openstreetmap/openstreetmap-website/pull/5790 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250311/489c6d9b/attachment.htm>


More information about the rails-dev mailing list