Skip to content
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

The TinyMCE bullet menu stays on top #3733

Closed
addison74 opened this issue Jan 14, 2024 · 2 comments · Fixed by #3851
Closed

The TinyMCE bullet menu stays on top #3733

addison74 opened this issue Jan 14, 2024 · 2 comments · Fixed by #3851

Comments

@addison74
Copy link
Contributor

Steps to reproduce this issue:

1 - Go to Backend > CMS > Pages and select a page for editing
2 - Activate TinyMCE editor by pressing [Show / Hide Editor]
3 - Click on the three horizontal bullets icon [...] at the right to show hidden menu and let it visible
4 - Click on [Variables] and [Widgets] buttons

Both "Insert Variable..." and "Insert Widget..." windows are behind the menu instead of being in front of it. In addition, that menu, even if it remains visible, should be covered by the gray background, which is not.

Please see the image bellow

overlay

@addison74 addison74 added the bug label Jan 14, 2024
@sreichel
Copy link
Contributor

Note: when you hit "Insert widget" window z-index increases with 2 with every click ... (?!?!)

... needs some investigation.

@fballiano
Copy link
Contributor

Note: when you hit "Insert widget" window z-index increases with 2 with every click ... (?!?!)

... needs some investigation.

I've analyzed the code and it was done that way to manage situations when multiple overlays are opened one after each other. This way the last opened overlay stays always on top of the previous one. Having a fixed zindex wouldn't have worked.

Also, it's increased by 2 because 1 is for the backdrop and 1 is for the actual modal "window".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants