[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: PNG File Screenshot (10).png    
Issue Links:
Relates
relates to MGNLUI-4784 Implement validation bubbles for deta... Closed
relates to CONTEDIT-202 Provide solution for validation UI wi... Closed
dependency
depends upon MGNLUI-4784 Implement validation bubbles for deta... 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:
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,
Simon

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

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