[MAGNOLIA-4890] PageEditor doesn't always render the bars on top Created: 06/Mar/13  Updated: 31/May/13  Resolved: 20/May/13

Status: Closed
Project: Magnolia
Component/s: gui, page editor
Affects Version/s: None
Fix Version/s: 4.5.9

Type: Bug Priority: Neutral
Reporter: Zdenek Skodik Assignee: Roman Kovařík
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Attachments: PNG File 5.0.after.png     PNG File 5.0before.png     PNG File after4.5.png     PNG File before4.5.png    
Issue Links:
causality
is causing MAGNOLIA-5074 Edit bars overlap smartphone/tablet p... Closed
duplicate
is duplicated by MAGNOLIA-4405 Page editor toolbar is below content ... Closed
Template:
Acceptance criteria:
Empty
Task DoD:
[ ]* Doc/release notes changes? Comment present?
[ ]* Downstream builds green?
[ ]* Solution information and context easily available?
[ ]* Tests
[ ]* 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:

 Description   

To reproduce, try to add new component to metaNavigation at demo-project. Horizontal navigation should overlap the newBar at this point.



 Comments   
Comment by Jan Haderka [ 14/May/13 ]

Possibly related selenium test: info.magnolia.integrationtests.selenium.PageEditorTest.testEditBarOfSelectedComponentIsVisibleAfterPageReload()

Comment by Jan Haderka [ 16/May/13 ]

Problem is cause by the fact that <ul/> element in nav-global/nav-global-box has z-index set explicitly to 100.
Still to overcome such problems we can force z-index even higher (65535 sounds like nice rounded value ) for all
mgnlEditorBarButtons
mgnlEditorBarLabel
mgnlEditor
mgnlEditorBar
mgnlEditorPlaceholder
elements (hope I didn't miss any).

Please note that z-index works only on elements with explicitly set position, so we would need to force that one as well to make it work, position:relative seems to be reasonably safe here.

Comment by Espen Jervidalo [ 17/May/13 ]

It should be enough to set the z-index on the mgnlEditor class. Thats a general class used for all bars.

Comment by Jan Haderka [ 17/May/13 ]

Even better. Thx.

Comment by Roman Kovařík [ 20/May/13 ]

Screenshots before and after the fix added.

Generated at Mon Feb 12 04:00:00 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.