[MGNLUI-5634] Improve complex field styles Created: 16/Jan/20  Updated: 03/Mar/20  Resolved: 28/Feb/20

Status: Closed
Project: Magnolia UI
Component/s: None
Affects Version/s: 6.2
Fix Version/s: 6.2

Type: Task Priority: Neutral
Reporter: Rishab Dhar Assignee: Šimon Demočko
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: 0d
Time Spent: 4d 7.75h
Original Estimate: Not Specified

Attachments: PNG File new-pages-app-broken-traits-dialog.png     PNG File new-pages-app-inconsistent-trait-chooser.png     PNG File old-pages-app-trait-dialog.png     PNG File old-pages-app-traits-added.png    
Issue Links:
Relates
relates to MGNLPN-470 UX consolidation for migrated Preview... Closed
relates to MGNLPN-449 Migrate preview to new UI framework Closed
dependency
is depended upon by MGNLUI-5650 DOC: Screenshot updates of complex fi... Closed
Template:
Acceptance criteria:
Empty
Task DoR:
Empty
Date of First Response:
Sprint: UI Framework & 6.2 Ramp up 15, UI Framework 16, UI Framework 17
Story Points: 8

 Description   

Regular complex fields

By theme design the root form content has pretty styling so that it looks appropriately in the detail sub-apps and dialogs (paddings, flex-box settings, height etc). However, when those recursively apply to the complex field layouts (they're re-used from the form implementation), the UI start to get messy.* We need to prevent most of the rules from leaking into the complex fields, the attached PR attempts to do that for at least the most pressing ones.

Trait fields

Action: Open the url http://localhost:8080/magnoliaAuthor/.magnolia/admincentral#app:pages:browser;/travel/contact/variants/variant-0:treeview and trigger Chose Audience action.

Observation: Some of the fields in the dialog are missing (see screenshot). Additionally, the trait dialog in the new pages app has broken CSS and is visually different from the one in the old app. 

Expectation: The dialog view in the new pages app should be consistent to the one in the old pages app.

Screenshots have been added on how it's on the old app and in the new app.



 Comments   
Comment by Simon Lutz [ 04/Feb/20 ]

Includes feedback / open points of linked ticket MGNLPN-449.

Comment by Šimon Demočko [ 04/Feb/20 ]

Should the old-pages-app-traits-added be followed though? It seems to be broken too. The additional traits are somewhat shifted to the left.

Generated at Mon Feb 12 09:28:31 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.