WordPress.org

Make WordPress Core

Opened 3 years ago

Last modified 5 weeks ago

#39384 new defect (bug)

TwentySeventeen Navbar z-index

Reported by: halgatewood Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version:
Component: Bundled Theme Keywords: has-patch
Focuses: css, template Cc:

Description

The current z-index for the fixed navigation is 7. This seems a bit low as it should be above all other items on the page. For instance this widget getting mixed up with the navigation items:

https://halgatewood.com/wp-content/uploads/2016/12/twentyseventeen-zindex.jpg

Something much higher may be appropriate:

.site-navigation-fixed.navigation-top { z-index: 9999; }

Thanks,
Hal Gatewood

Attachments (1)

39384.patch (418 bytes) - added by poena 5 weeks ago.
Updates z-index for the fixed menu.

Download all attachments as: .zip

Change History (3)

#1 @justnorris
3 years ago

  • Type changed from enhancement to defect (bug)

I just ran into the same issue.

I think z-index: 9999 can be an overkill, and maybe even override some lightboxes, but z-index:7 is definitely way too low for a sticky menu.

Maybe it should be set to around 500 - 1000 ?

Last edited 3 years ago by justnorris (previous) (diff)

@poena
5 weeks ago

Updates z-index for the fixed menu.

#2 @poena
5 weeks ago

  • Component changed from Themes to Bundled Theme
  • Focuses css added
  • Keywords has-patch added

Tested and added the small patch.
Changed the component to Bundled Theme.

Note: See TracTickets for help on using tickets.