The z-index for the navigation overlay is an undefined CSS variable. That's probably why it's not clickable, and why it doesn't cover Oliver's header region.


After

CommentFileSizeAuthor
#12 Screenshot 2024-06-04 at 8.21.06 AM.png562.93 KBgauravvvv
#2 navigation-3443577.patch1.38 KBrajan kumar@2026
Home___Navigation_and_ui-suite-initiative__Channel__-_Drupal_-_6_new_items_-_Slack.png592.85 KBmherchel

Issue fork drupal-3443577

Command icon 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:

Issue fork navigation-3443577

Command icon 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:

About issue forks

Comments

mherchel created an issue. See original summary.

rajan kumar@2026’s picture

rajan kumar@2026 commented
Status: Active » Needs review
StatusFileSize
new navigation-3443577.patch1.38 KB

hii @mherchel

i defined "Navigation overlay z-index " variable.

rishabjasrotia made their first commit to this issue’s fork.

rishabjasrotia opened merge request !260

ckrina’s picture

ckrina
she/her
Primary language Catalan
Location Barcelona
commented
Project: Navigation » Drupal core
Version: 1.x-dev » 11.x-dev
Component: Code » navigation.module
Status: Needs review » Needs work

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.

ckrina’s picture

ckrina
she/her
Primary language Catalan
Location Barcelona
commented
Issue tags: +Novice, +Portland2024
anicoto’s picture

anicoto
Primary language Spanish
Location Houston, TX
commented
Novice issue reserved for the Mentored Contribution during the Contribution Day at DrupalCon Portland 2024. After the 8th of May 2024, this issue returns to being open to all. Thanks
finnsky’s picture

finnsky commented

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

finnsky’s picture

finnsky commented
Issue tags: -Novice

Probably concept of scoped css variables is out of Novice expertise area.

Gauravvvv made their first commit to this issue’s fork.

Gauravvvv opened merge request !8276

gauravvvv’s picture

gauravvvv commented
Issue summary: View changes
StatusFileSize
new Screenshot 2024-06-04 at 8.21.06 AM.png562.93 KB

After:

gauravvvv’s picture

gauravvvv commented
Status: Needs work » Needs review
finnsky’s picture

finnsky commented
Status: Needs review » Reviewed & tested by the community

Thank you! Exactly!

finnsky changed the visibility of the branch 3443577-navigation-overlay-z-index to hidden.

nod_ closed merge request !8276

  • nod_ committed ae230285 on 10.3.x 
    Issue #3443577 by Gauravvvv, rishabjasrotia, Rajan Kumar@2026, mherchel...

  • nod_ committed 7f9b562d on 10.4.x 
    Issue #3443577 by Gauravvvv, rishabjasrotia, Rajan Kumar@2026, mherchel...

  • nod_ committed 9a8693a0 on 11.0.x 
    Issue #3443577 by Gauravvvv, rishabjasrotia, Rajan Kumar@2026, mherchel...

  • nod_ committed 4ce35031 on 11.x 
    Issue #3443577 by Gauravvvv, rishabjasrotia, Rajan Kumar@2026, mherchel...
nod_’s picture

nod_
Primary language French
Location Lille
commented
Version: 11.x-dev » 10.3.x-dev
Status: Reviewed & tested by the community » Fixed

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!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.