dheeraj12347 left a comment (openstreetmap/openstreetmap-website#6767)
Thanks for the review and the video, @pablobm – it made the problem very clear.
I’ve updated the implementation as follows:
The browse/common_details block (tags, members, etc.) is no longer sticky for
nodes, ways and relations. Only the element navigation links (the secondary
menu: History, Edit, etc.) are wrapped in sticky-top bg-body border-bottom, so
the tags and members tables now scroll normally and no longer obscure the rest
of the sidebar content on smaller viewports.
I removed the unused browse-container class.
I reverted unrelated changes in common.scss and en.yml, so this PR now only
touches the browse partials.
This keeps the navigation visible as requested in issue #6759 while avoiding
the “big sticky tags” problem from the video, and it uses Bootstrap utilities
instead of custom CSS as suggested by @tomhughes.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6767#issuecomment-3869375324
You are receiving this because you are subscribed to this thread.
Message ID:
<openstreetmap/openstreetmap-website/pull/6767/[email protected]>
_______________________________________________
rails-dev mailing list
[email protected]
https://lists.openstreetmap.org/listinfo/rails-dev
