The problem is that you can't select any items in the dropdown menu because of the z-index of the dropdown. (see screenshot in attachment)

CommentFileSizeAuthor
#21 2017-09-24.png322.37 KBmpark
#15 admin_toolbar-2838636-15.patch357 byteskerby70
#14 admin_toolbar-2838636-14.patch350 bytesddrozdik
#10 admin_toolbar-2838636-10.patch358 bytesddrozdik
#10 admin+%7C+Site-Install+2017-05-25+23-05-56.png40.63 KBddrozdik
#8 Capture d’écran 2017-01-06 à 09.37.17.png50.96 KBromainj
#2 issue-with-zindex-of-dropdown-in-horizontal-tray_2838636_2.patch386 bytesf0ns
Schermafbeelding 2016-12-22 om 15.32.17.png102.94 KBf0ns

Comments

Fons Vandamme created an issue. See original summary.

f0ns’s picture

f0ns commented
StatusFileSize
new issue-with-zindex-of-dropdown-in-horizontal-tray_2838636_2.patch386 bytes

Here is the patch that fixes this problem:

f0ns’s picture

f0ns commented
Status: Active » Needs review
f0ns’s picture

f0ns commented
Version: 8.x-1.18 » 8.x-1.x-dev

Status: Needs review » Needs work

The last submitted patch, 2: issue-with-zindex-of-dropdown-in-horizontal-tray_2838636_2.patch, failed testing.

f0ns’s picture

f0ns commented
Status: Needs work » Needs review
f0ns’s picture

f0ns commented

Note sure how a line of CSS can break the tests..

romainj’s picture

romainj commented
StatusFileSize
new Capture d’écran 2017-01-06 à 09.37.17.png50.96 KB

I cannot reproduce the behavior. What browser are you using?

f0ns’s picture

f0ns commented

Chrome, maybe it has something to do with the fact that I added extra links and you are testing on the default links that are in the management menu.

ddrozdik’s picture

ddrozdik commented
StatusFileSize
new admin+%7C+Site-Install+2017-05-25+23-05-56.png40.63 KB
new admin_toolbar-2838636-10.patch358 bytes

We have faced the same situation with a theme in OpenY (see screenshot).
This is happening only in a case when there is a fixed area with z-index bigger than z-index in toolbar.

  • romainj committed f8b984b on 8.x-1.x authored by ddrozdik 
    Issue #2838636 by Fons Vandamme, ddrozdik, romainj: Issue with z-index...
romainj’s picture

romainj commented

Patch #10 works for me. I added to the latest dev version.

romainj’s picture

romainj commented
Status: Needs review » Fixed
ddrozdik’s picture

ddrozdik commented
Status: Fixed » Needs review
StatusFileSize
new admin_toolbar-2838636-14.patch350 bytes

Today working with openY I realized that in previous patch #10 I used an incorrect ID. Here is an updated patch.

kerby70’s picture

kerby70 commented
StatusFileSize
new admin_toolbar-2838636-15.patch357 bytes

Posting a patch admin_toolbar-2838636-15.patch to be applied to 8.x-1.19, to avoid issues.

f0ns’s picture

f0ns commented

Would love to see this go into the next version :)

  • romainj committed c72e1b1 on 8.x-1.x authored by kerby70 
    Issue #2838636 by ddrozdik, Fons Vandamme, kerby70, romainj: Issue with...
romainj’s picture

romainj commented

Patch #14 committed in latest dev version. Thanks to all.

romainj’s picture

romainj commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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

mpark’s picture

mpark commented
StatusFileSize
new 2017-09-24.png322.37 KB

I have to re-open this issue. After update to 8x1.20 the problem is there again. The patch is correct added:

#toolbar-bar {
  z-index: 9999;
}

but the code is diferrent, so new css code is better change to:

#toolbar-tab {
  z-index: 9999;
}
mpark’s picture

mpark commented
Version: 8.x-1.x-dev » 8.x-1.20