[MGNLUI-5082] Using one required field in mutlivalue composite field will hint all fields as required Created: 15/Mar/19 Updated: 11/Mar/21 Resolved: 11/Mar/21 |
|
| Status: | Closed |
| Project: | Magnolia UI |
| Component/s: | forms |
| Affects Version/s: | 6.0 |
| Fix Version/s: | None |
| Type: | Bug | Priority: | Neutral |
| Reporter: | Marty Glaubitz | Assignee: | Unassigned |
| Resolution: | Won't Fix | Votes: | 0 |
| Labels: | M6-UI-tech-bug | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Environment: |
Windows 10 |
||
| Attachments: |
|
||||||||||||||||||||
| Issue Links: |
|
||||||||||||||||||||
| 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: | |||||||||||||||||||||
| Epic Link: | M6 UI & UX Improvements | ||||||||||||||||||||
| Story Points: | 8 | ||||||||||||||||||||
| Description |
|
We using a multivalue field with a composite field inside it:
- name: assets
fieldType: multiValue
label: fields.assets
transformerClass: info.magnolia.ui.form.field.transformer.multi.MultiValueSubChildrenNodePropertiesTransformer
field:
name: asset
label: ' '
layout: vertical
fieldType: composite
transformerClass: info.magnolia.ui.form.field.transformer.composite.NoOpCompositeTransformer
fields:
- name: headline
fieldType: text
placeholder: msg.enterToOverwriteAssetDescription
- name: description
fieldType: text
rows: 2
maxLength: 160
placeholder: msg.enterToOverwriteAssetHeadline
- name: asset
buttonSelectOtherLabel: msg.selectAsset
label: ' '
fieldType: assetLink
required: true
appName: assets-app
As you can see, only the last field (asset) is marked as required. But the tooltip message "This field is required" is also shown when hovering over the other non-required fields headline and description Do not break the tooltip per field, but enhance the message within the tooltip to be more specific. |
| Comments |
| Comment by Simon Lutz [ 21/Mar/19 ] |
|
Hi martyglaubitz, thanks for reporting the issue - well spotted! We will work on it and provide a fix in an upcoming version. Cheers, |
| Comment by Roman Kovařík [ 10/Mar/21 ] |
|
asiska Could you check if this is still the case? |
| Comment by Adam Siska [ 11/Mar/21 ] |
|
still reproducible in old fw (not present in new ui fw though): |
| Comment by Adam Siska [ 11/Mar/21 ] |
|
only old ui fw |