When using the Claro theme ( have not tested with other Administration ) visual.

see screenshot

CommentFileSizeAuthor
#5 z-Index_issues_with_claro-3112965-5.patch13.33 KBdennis cohn
#2 Screenshot 2020-02-12 at 09.32.20.png80.41 KBdennis cohn
z-index_issue_with_claro.png474.83 KBphilosurfer

Comments

philosurfer created an issue. See original summary.

dennis cohn’s picture

dennis cohn commented
StatusFileSize
new Screenshot 2020-02-12 at 09.32.20.png80.41 KB

This also exist when using good old seven as a Administration Theme.
I will try to fix this..

jakubhnilicka’s picture

jakubhnilicka commented

Nice thank you Dennis.

philosurfer’s picture

philosurfer commented

This is the official guideline as it relates to z-index in Drupal theming.

https://www.drupal.org/docs/8/theming/z-indexes-in-drupal-8

dennis cohn’s picture

dennis cohn commented
StatusFileSize
new z-Index_issues_with_claro-3112965-5.patch13.33 KB

I figured it out...there is/was a bug in cssnano which converted all z-indexes to some random number.
For example:
z-index:999997 from .toolbar__top was compiled to z-index:3
z-index:1000 from . toolbar__secondary was complied to z-index: 1

After I updated cssnano to the latest version, it was compiled as it should be.
I also needed to add node-sass as a dependency in the package.json and updated grunt-sass to succesfull compile the right css.

dennis cohn’s picture

dennis cohn commented
Status: Active » Needs review
philosurfer’s picture

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

Fixed. RTBC. =)

jakubhnilicka’s picture

jakubhnilicka commented
Status: Reviewed & tested by the community » Closed (works as designed)

Fixed in alpha6 release.

jakubhnilicka’s picture

jakubhnilicka commented
Status: Closed (works as designed) » Fixed

Status: Fixed » Closed (fixed)

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