[openstreetmap/openstreetmap-website] Adjust vertical alignment of secondary header navigation (PR #4828)
Anton Khorev
notifications at github.com
Fri May 24 17:39:40 UTC 2024
Like #4723 + adjustments inside the user button + increased horizontal spacing between menu items (in #4723 the spacing decreased because with `display: flex` whitespace between items gets removed).
I get rid of `h1, nav, nav > ul, nav > ul > li` selector and do other changes in order to switch later to Bootstrap navbar and decrease the number of menu/sidebar visibility modes.
Before / after when not logged in:


Before / after with issues counter:


Before / after with message counter:


You can view, comment on, or merge this pull request online at:
https://github.com/openstreetmap/openstreetmap-website/pull/4828
-- Commit Summary --
* Use Bootstrap .nav in secondary header navigation
* Increase secondary nav item padding to compensate removed whitespace
* Adjust vertical alignment of user button contents
-- File Changes --
M app/assets/stylesheets/common.scss (10)
M app/views/layouts/_header.html.erb (8)
-- Patch Links --
https://github.com/openstreetmap/openstreetmap-website/pull/4828.patch
https://github.com/openstreetmap/openstreetmap-website/pull/4828.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/4828
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/4828 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20240524/5d0f758c/attachment.htm>
More information about the rails-dev
mailing list