[PAGES-699] z-index of div class="mgnlEditor mgnlPlaceholder" is only 1. Created: 08/Apr/22 Updated: 15/Sep/22 Resolved: 24/Aug/22 |
|
| Status: | Closed |
| Project: | Magnolia pages module |
| Component/s: | None |
| Affects Version/s: | 6.2.19 |
| Fix Version/s: | 6.3.0, 6.2.21 |
| Type: | Bug | Priority: | Neutral |
| Reporter: | Michiel Meeuwissen | Assignee: | Jesus Alonso |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | quickwin | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | 3.5h | ||
| Original Estimate: | Not Specified | ||
| Attachments: |
|
||||||||
| Issue Links: |
|
||||||||
| Template: |
|
||||||||
| Acceptance criteria: |
Empty
|
||||||||
| Task DoD: |
[X]*
Doc/release notes changes? Comment present?
[X]*
Downstream builds green?
[X]*
Solution information and context easily available?
[X]*
Tests
[X]*
FixVersion filled and not yet released
[ ] 
Architecture Decision Record (ADR)
|
||||||||
| Bug DoR: |
[ ]*
Steps to reproduce, expected, and actual results filled
[ ]*
Affected version filled
|
||||||||
| Date of First Response: | |||||||||
| Visible to: |
Mathijn Elhorst, Rico Jansen, Tim Molenaar
|
||||||||
| Epic Link: | AuthorX Support | ||||||||
| Sprint: | AuthX 16 | ||||||||
| Story Points: | 1 | ||||||||
| Team: | |||||||||
| Description |
|
It makes it onreachable because of some of our content elements with bigger z-index. See https://demo.magnolia-cms.com/VAADIN/themes/pages-app/page-editor.css The selector:
div.mgnlEditorBar.area ~ div.mgnlPlaceholder, div.mgnlEditorBar.area ~ * div.mgnlPlaceholder {
margin: 0 0 2px 0 !important;
z-index: 1 !important;
}
matches. I'm no frontend expert, but it seems to me that these kind of things should have a big z-index to ensure that they keep being reachable in any circonstance. |
| Comments |
| Comment by Mercedes Iruela [ 01/Aug/22 ] |
|
Hello Michiel, This issue is being difficult to reproduce. Could you please attach a template that allows us to reproduce this issue?? Thanks! Mercedes |
| Comment by Richard Gange [ 22/Aug/22 ] |
|
Hello mihxil- It would really help us to get some feedback from you. What we have decided to do is revert the z-index change we made in Thanks! |
| Comment by Jesus Alonso [ 24/Aug/22 ] |
|
The issue has been fixed changing the css. |
| Comment by Richard Gange [ 15/Sep/22 ] |
|
Hello mihxil- This issue was fixed in Magnolia 6.2.23. Cheers |