[openstreetmap/openstreetmap-website] Use secondary text color in header menu (PR #4661)
Anton Khorev
notifications at github.com
Mon Apr 8 14:56:30 UTC 2024
Uses `text-secondary` or `text-secondary-emphasis` if it's a current page.
Replaces:
- `$darkgrey`: `text-secondary` matches it
- `darken($darkgrey, 25%)`: in dark mode it shouldn't be darkened
Before:

After:

I also removed the current page color change in more/user dropdowns because it worked in the opposite direction, making the text lighter.
Before, with "Communities" as the current page:

You can view, comment on, or merge this pull request online at:
https://github.com/openstreetmap/openstreetmap-website/pull/4661
-- Commit Summary --
* Use secondary text color in header menu
-- File Changes --
M app/assets/stylesheets/common.scss (5)
M app/helpers/application_helper.rb (4)
M app/views/layouts/_header.html.erb (42)
M test/helpers/application_helper_test.rb (2)
-- Patch Links --
https://github.com/openstreetmap/openstreetmap-website/pull/4661.patch
https://github.com/openstreetmap/openstreetmap-website/pull/4661.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/4661
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/4661 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20240408/3d5e5e05/attachment-0001.htm>
More information about the rails-dev
mailing list