[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: PNG File Screenshot 2022-04-08 at 15.57.50.png     PNG File clear-floats.png     PNG File image-2022-08-19-17-31-41-618.png     PNG File image-2022-08-19-17-45-28-898.png    
Issue Links:
relation
is related to PAGES-412 Page Editor Bars: Responsive/horizont... Closed
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: AuthorX

 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 PAGES-412. Instead we are going to clear floats as shown in this PR. Would it be possible for you to use your browser console to be sure this CSS change helps your issue?

Thanks!
Rich

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
Rich

Generated at Mon Feb 12 06:21:26 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.