- Issue created by @mherchel
- Status changed to Needs review
2 months ago 6:07am 26 April 2024 - ๐ฎ๐ณIndia Rajan Kumar@2026
hii @mherchel
i defined "Navigation overlay z-index " variable.
- First commit to issue fork.
- Merge request !260#3443577 - Fix Navigation overlay z-index is not defined โ (Open) created by rishabjasrotia
- Status changed to Needs work
2 months ago 10:19pm 26 April 2024 - ๐ช๐ธSpain ckrina Barcelona
Moving to Needs work after getting Navigation into core.
While you are making the above changes, we recommend that you convert this patch to a merge request โ . Merge requests are preferred over patches. Be sure to hide the old patch files as well.
- ๐ท๐ธSerbia finnsky
No need to write extra css here. Variable exists but scoped to `data-drupal-admin-styles` attribute. this overlay is outside of this scope.
Easiest thing here will be add this attribute `data-drupal-admin-styles` to `.admin-toolbar-overlay` element
- ๐ท๐ธSerbia finnsky
Probably concept of scoped css variables is out of Novice expertise area.
- ๐ฎ๐ณIndia Gauravvv Delhi, India
Gauravvvv โ made their first commit to this issueโs fork.
- Status changed to Needs review
22 days ago 3:05am 4 June 2024 - Status changed to RTBC
22 days ago 4:38am 4 June 2024 - ๐ท๐ธSerbia finnsky
finnsky โ changed the visibility of the branch 3443577-navigation-overlay-z-index to hidden.
- Status changed to Fixed
22 days ago 8:23am 4 June 2024 - ๐ซ๐ทFrance nod_ Lille
Committed and pushed 4ce3503191 to 11.x and 9a8693a0c1 to 11.0.x and 7f9b562d57 to 10.4.x and ae23028577 to 10.3.x. Thanks!
Automatically closed - issue fixed for 2 weeks with no activity.