The 2026 Drupal Business Survey is open. Your response will help shape the future of the Drupal business ecosystem.Standard profile installation
Latest 1.x navigation module
Enable Stark theme here /admin/appearance and set it as Default theme.
Visit homepage
| Comment | File | Size | Author |
|---|---|---|---|
| #8 | afterpatch_2.png | 33.39 KB | kanchan bhogade |
| #8 | afterpatch_1.png | 29.94 KB | kanchan bhogade |
| #8 | beforepatch_2.png | 31.92 KB | kanchan bhogade |
| #8 | beforepatch_1.png | 29.44 KB | kanchan bhogade |
| #6 | Screenshot_without_patch.png | 93.73 KB | kanchan bhogade |
Issue fork navigation-3421563
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
changes
Comments
Comment #2
kostyashupenkoGonna fix it quickly
Comment #4
kostyashupenkoComment #5
ckrina@kostyashupenko good catch! I'm wondering if there's a way to do this all at once and maybe this fix will solve this issue so we don't have to add this extra attribute to every component? https://www.drupal.org/project/navigation/issues/3402592#mr174-note266578
Comment #6
kanchan bhogade commentedHi @kostyashupenko
Checked on Drupal 11.x without a patch it looks good to me
if I missing something please let me know so it can be helpful to reproduce the issue
Attaching screenshot
Comment #7
kostyashupenkoRe #5
i was thinking what i can do - i decided to switch logic a bit in CSS. Now `font-family: inherit` is still placed in two difference places, but that's why:
1. I want to keep such "base" styles only in the scope of navigation module
2. Top bar is placed in the difference place in DOM tree than admin toolbar.
Looks ok now i think
Re #6
This issue can be reproduced only with some custom themes, not with core themes like Claro or like Olivero (the theme on your screenshot). These two themes have `button { font-family: inherit }` already.
Comment #8
kanchan bhogade commentedThank you @kostyashupenko able to reproduce the issue with your inputs.
I have tested MR !176 on Drupal version 10.2.2 with a custom theme
The patch was applied successfully...
Result:
The Font issue for custom themes is resolved with MR !176
RTBC++
Attaching screenshots for reference
Keeping in "needs review" for code verification
Comment #9
ckrinaWhat I meant in #5 is if it would it work to do something like this:
This way this would be defined in the
admin-initial-styles.cssfor all components without the need to specify it on each component file.If that approach works it'd mean to postpone this issue until #3402592: Reset theme css. is in and this gets integrated in that file.
Comment #10
ckrina@kostyashupenko is this still happening after merging #3402592: Reset theme css. ? And can this maybe be added in the new
admin-reset-styles.pcss.cssfile?Comment #11
ckrina@kostyashupenko is this still happening?
Comment #12
kostyashupenkoComment #13
ckrina@kostyashupenko would it be possible to have testing instructions to test it on my local? Like which theme you are using, or which changes you are recreating to test it. Thanks!
Comment #14
kostyashupenko@ckrina just enable core theme "Stark" and set it as default theme. Then visit homepage )
Comment #15
kostyashupenkoComment #19
ckrinaTested locally and it works perfect, thanks! I also like the admin-reset-styles library on its own so we can move it into its own thing in the future.