-
-
Notifications
You must be signed in to change notification settings - Fork 48
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
CMS z-index
is too low
#252
Comments
This makes sense, I think I was just trying to avoid an ever escalating z-index battle. The general plan is to have the least specific CSS defaults as possible, so the CMS takes on the character of the project and is easy to override. My hope is that all you would have to do is drop in your own |
@jimafisk If you don't have time for it, I could try to implement it. Where should I look for the index definitions? |
Hi @notramo, I just took a look, it seems like our plenti/defaults/core/cms/admin_menu.svelte Line 202 in c585291
The plenti/defaults/core/cms/admin_menu.svelte Line 226 in c585291
And our plenti/defaults/core/cms/admin_menu.svelte Line 262 in c585291
Those values generally seem appropriate to me, although I'd honestly prefer to lower the modal wrapper to In cases where site elements are appearing above the Plenti admin menus undesirably, I would attempt to do the following:
I do think that we should add an
I also usually wrap my custom HTML/CSS in a Are you pulling in a style framework like Bootstrap or Foundation? Those can often be quite heavy-handed with their implementations of these things. |
@notramo were you able to get a z-index override working for your project? |
If the page uses a
z-index
greater than 0, it overlaps with the CMS. The CMS should use the highest possiblez-index
.The text was updated successfully, but these errors were encountered: