Harden various editor implementations/incarnations
(MGNLUI-5407)
|
|
| Status: | Closed |
| Project: | Magnolia UI |
| Component/s: | None |
| Affects Version/s: | 6.0 |
| Fix Version/s: | 6.2 |
| Type: | Sub-task | Priority: | Neutral |
| Reporter: | Aleksandr Pchelintcev | Assignee: | Šimon Demočko |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | 0.5h | ||
| Time Spent: | 2.65h | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Template: |
|
||||||||
| Date of First Response: | |||||||||
| Sprint: | UI Framework 8, UI framework 9, UI Framework 10, UI Framework 11 | ||||||||
| Comments |
| Comment by Šimon Demočko [ 19/Nov/19 ] |
|
rkovarik we will need to define the describeByProperty for Magnolia-included apps. Otherwise tab caption defaults to illegible full-paths (pages, resources, etc.). Some apps (e.g. Contacts) need an implementation of ItemDescriber. Do we need issues in respective projects for these? |
| Comment by Roman Kovařík [ 20/Nov/19 ] |
|
New pages app detail is using advanced logic (changing the caption according to navigation in the edited page). Resources app not critical for me ((the old app is suffering from the same issue, probably no complains since the first impl https://demo.magnolia-cms.com/.magnolia/admincentral#app:resources:hotfix;/admincentral/apps/notifications.yaml:edit) but you can still create a ticket for it. Contacts app is just a sample app, we are updating the config on the fly during 6.x development, feel free to amend a QA commit. |
| Comment by Šimon Demočko [ 20/Nov/19 ] |
|
Thanks! Hovering over tab caption shows full caption for me only on Safari on the demo server, not in Chrome. Same behaviour after 5440 merge in 6.2. |