- Issue created by @Klaus Juhantalo
- ๐ฎ๐ณIndia rajneeshb New Delhi
Devel sf-dump class having z-index 99999, so to fix the issue for toolbar menu added z-index to 100000.
- Status changed to RTBC
about 2 years ago 11:53am 28 April 2023 - ๐ฌ๐งUnited Kingdom james.williams
Works for me! That said, I wonder if a fix should really go into core, since core's own toolbar gets obscured by the output from the dumper too. admin_toolbar could probably then just make use of core's fix. Up to this module's maintainer really?
- Status changed to Fixed
about 2 years ago 1:03pm 28 April 2023 -
adriancid โ
committed 33ca790b on 3.x authored by
rajneeshb โ
Issue #3335841 by rajneeshb, adriancid: Admin toolbar menu z-index sucks...
-
adriancid โ
committed 33ca790b on 3.x authored by
rajneeshb โ
-
adriancid โ
committed ac0c6477 on 3.x authored by
rajneeshb โ
Issue #3335841 by rajneeshb, adriancid: Admin toolbar menu z-index sucks...
-
adriancid โ
committed ac0c6477 on 3.x authored by
rajneeshb โ
- ๐ง๐ชBelgium joevagyok
The changed z-index causes a problem with media and other dialogues, because the toolbar z-index the closing of the dialogues is not clickable any more.
- ๐บ๐ธUnited States butterwise
The changed z-index also causes a problem with Ckeditor: When you maximize a formatted text field using Ckeditor, the admin toolbar shifts to the left-vertical position and overlays the editor window:
- ๐ฌ๐งUnited Kingdom jessebaker
This change to using an enormous z-index has also broken aspects of Acquia Site Studio which assumes the admin toolbar's z-index to be 502 which it has been for years.
Will it be reverted because otherwise we will need to resolve the issues it causes.
- ๐ง๐ชBelgium joevagyok
I tried to contact the maintainers on slack but haven't received feedback on the issue so far. I found another issue created related to this.
@jessebaker Just lock the version to 3.3.0 for now in composer until it gets resolved. - Status changed to Active
about 2 years ago 1:13pm 10 May 2023 - ๐จ๐ฆCanada adriancid Montreal, Canada
I'm reverting this issue due to the problems described at ๐ New z-index is breaking other functionalities Closed: duplicate
-
adriancid โ
committed de85d10d on 3.x authored by
rajneeshb โ
Revert "Issue #3335841 by rajneeshb, adriancid: Admin toolbar menu z-...
-
adriancid โ
committed de85d10d on 3.x authored by
rajneeshb โ
- ๐ฌ๐งUnited Kingdom jessebaker
Thanks @adriancid. However I think you also need to remove the duplicate style declaration on line 1 of the same file -
#toolbar-bar { z-index: 100000; }
was in that file twice.
- ๐จ๐ฆCanada adriancid Montreal, Canada
@jessebaker can you check the last changes? I can't see any duplicate code there.
- ๐จ๐ฆCanada adriancid Montreal, Canada
The patch here is causing this error ๐ Top search results are hidden under the toolbar. Closed: works as designed
- ๐ฌ๐งUnited Kingdom jessebaker
Hmm I might be wrong, sorry!
If you look here https://git.drupalcode.org/project/admin_toolbar/-/blob/3.3.2/css/admin.... you can see the following code twice (on lines 1 and again on 9)
#toolbar-bar { z-index: 100000; }
It looked like the commit linked in your comment only removed the second one - but looking at 3.x it looks like both are gone!
https://git.drupalcode.org/project/admin_toolbar/-/blob/3.x/css/admin.to...
- ๐ฎ๐ณIndia Harish1688 India
Hi
To address the issue, the requirement states that the toolbar menu should be positioned above the status message.
However, applying the CSS property z-index: 100000; to the #toolbar-bar class caused some unforeseen problems.In order to resolve this for admin toolbar module, creating a - patch menu-z-index-sucks-with-devel-module-3335841-17.patch. override the CSS (z-index:0;) of the devel module specifically for the admin toolbar by modifying the "admin.toolbar_search.css" file.
- Status changed to Needs review
about 2 years ago 9:43am 16 May 2023 - last update
about 2 years ago 17 pass - ๐ฎ๐ณIndia Jaspreet-Kaur
Verified! After applying patch #17 issue seems to be fixed. Thanks!
- ๐จ๐ฆCanada adriancid Montreal, Canada
I will leave open the issue until the other user reporting problems write a comment saying is working for them.
- ๐น๐ทTurkey Kartagis Istanbul
The patch at #2 works for me, I don't know about the other issues mentioned.
- Status changed to Active
4 months ago 12:09am 25 February 2025 - ๐ซ๐ทFrance dydave
I've seen the previous history on this issue and it seems patch from #2 broke the module.
However, if anybody would have a better suggestion or idea, for a CSS rule, it would be warmly welcome.
It looks like this is still an issue, see screenshot below on latest 3.x:
Although it is rather minor, since it only occurs when developing which is probably not "too" much of a problem for developers.
But indeed, if this could be fixed, it would improve developers experience, therefore I've switched this over to Feature request.We're accepting merge requests if anyone is interested in working on this.
Thanks in advance! - ๐ฎ๐ณIndia sandip
Hi @everyone i observed this issue. As mentioned in #23 and #9 it is clear that we can not increase the z-index of #toolbar-bar. But some of the sections has z-index around 999 upto 99999. Thats why we are facing this issue. Here the patch at #17 i dont think it is properly solving the issue. The same issue is occuring for CK Editor also. Here is the the issue queue of CK Editor -> https://www.drupal.org/project/admin_toolbar/issues/3426402 ๐ zindex issue between admin toolbar and ckeditor 5 Active
So i think if we cant increase z-index of #toolbar-bar so can we apply the same approach to its parent div which is #toolbar-administration ?
Like we can use position: relative and z-index around 99999. - ๐ซ๐ทFrance dydave
dydave โ changed the visibility of the branch 3.x to hidden.
- ๐ซ๐ทFrance dydave
Thanks Sandip (@sandip) for the recent feedback on this issue!
So i think if we cant increase z-index of #toolbar-bar so can we apply the same approach to its parent div which is #toolbar-administration ?
Like we can use position: relative and z-index around 99999.Could you please try translating these suggestions into actual changes in the following branch?
https://git.drupalcode.org/issue/admin_toolbar-3335841/-/tree/3335841-fi...Then try testing a bit with different types of settings for the Admin Toolbar module, browsing on various pages of the site.
That would definitely be very helpful ๐
Otherwise, I personally think this issue is probably different from the one with the CKEditor5 Toolbar ( ๐ zindex issue between admin toolbar and ckeditor 5 Active ).
But, at this point, it is still unclear whether a CSS or JS solution should be preferred.
It could still be worth exploring potential JS solutions as well:
For example, adding/removing CSS classes, altering tags styles, etc... on initialization, when the devel module is enabled and/or a Kint or Symfony vardump is found on page.Any testing, feedback and reviews would be greatly appreciated.
Thanks in advance! - ๐บ๐ธUnited States justcaldwell Austin, Texas
IMO, this is not an issue with admin_toolbar, and I wouldn't try to solve it here. The problem also occurs if you're just using the core toolbar module without admin_toolbar.
The issue stems from devel's dumper using such a high z-index (99999). Actually, it's upstream from devel, as the (default) dumper functionality comes from Symfony var-dumper. I assume there's a reason for the high z-index, but maybe an issue should be opened in one or both of those projects.
If you wanted to conditionally raise the toolbar z-index in the presence of a dump, you could use something like:
body:has(.sf-dump) .toolbar-oriented .toolbar-bar { z-index: 100000; }
Note: This might need other classes in addition to
.sf-dump
.That would still have the same conflicts with other modules mentioned above, but only when there's a dump on the page. Maybe it's worthwhile if devel is only temporarily enabled (?). As I said, I don't think this is amin_toolbar's problem to solve.