[MGNLUI-5684] Extend MultiFormView to use LayoutProducer Created: 25/Feb/20 Updated: 11/Mar/21 Resolved: 11/Mar/21 |
|
| Status: | Closed |
| Project: | Magnolia UI |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Improvement | Priority: | Neutral |
| Reporter: | Šimon Demočko | Assignee: | Roman Kovařík |
| Resolution: | Won't Do | Votes: | 0 |
| Labels: | multifield | ||
| Remaining Estimate: | 0d | ||
| Time Spent: | 5h 20m | ||
| Original Estimate: | Not Specified | ||
| Attachments: |
|
||||||||||||||||||||||||||||
| Issue Links: |
|
||||||||||||||||||||||||||||
| Template: |
|
||||||||||||||||||||||||||||
| Acceptance criteria: |
[ ]*
Take into consideration the changes stories app
[ ]*
Use FormLayoutProducer for detail subapp in new personas app multifield to have vertical alignment
[ ]*
Use stackedLayoutProducer in dialogs to have captions on top in narrow layouts (dialogs, e.g. choose audience dialog)
[ ]*
provide reasonable defaults - a simple multifield with just one type of field provided should not have captions at the top - those would be redundant.
[ ]*
NarrowVerticalLayout will be likely superseded by this solution, consider removal
|
||||||||||||||||||||||||||||
| 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)
[ ]*
https://jira.magnolia-cms.com/browse/MGNLUI-5684?focusedCommentId=252702&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-252702
|
||||||||||||||||||||||||||||
| Date of First Response: | |||||||||||||||||||||||||||||
| Epic Link: | M6 MultiFormView | ||||||||||||||||||||||||||||
| Sprint: | UI FW 22, UI FW 23 | ||||||||||||||||||||||||||||
| Story Points: | 8 | ||||||||||||||||||||||||||||
| Description |
| Comments |
| Comment by Roman Kovařík [ 02/Mar/21 ] |
|
For QA:
Fields are wrapped in containers and the wrappers inherit the caption from the fields. When both show the caption, this is the result. The container label is preferred here. |
| Comment by Roman Kovařík [ 03/Mar/21 ] |
|
The indentation problems on the screenshot don't seem to be present anymore (http://nightly.magnolia-cms.com)
This field is a composite, not multi field. Not related?
Usage of different layouts doesn't seems to do any difference.
Does this mean field captions at the left side instead of top?
This probably still makes sense (if only for Here you can see how would be affected. The changes are quite minimal, at least compared to my expectations. It's because the API requirements for sdemocko As you was involved in the multi form efforts, could you help me to reevaluate the need/expectation from this ticket? |
| Comment by Šimon Demočko [ 05/Mar/21 ] |
|
Moved responses to # |