Uploaded image for project: 'Magnolia UI'
  1. Magnolia UI
  2. MGNLUI-7711

HorizontalLayout doesn't show validation errors

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Duplicate
    • Icon: Neutral Neutral
    • None
    • None
    • None
    • None

      Steps to reproduce

      1. Use a  field with a validator https://nightly.magnolia-cms.com/.magnolia/admincentral#app:resources:edit;/ui-framework-jcr/dialogs/rename.yaml:edit
      2. Wrap the field into horizontalLayout 
            jcrName:
              $type: compositeField
              itemProvider:
                $type: currentItemProvider
              layout:
                $type: horizontal
              properties:
                someField:
                  $type: textField         
                name:
                  $type: textField
                  required: true
                  validators:
                    - $type: nodeNameValidator
        
      1. Open https://nightly.magnolia-cms.com/.magnolia/admincentral#app:tours:browser;/magnolia-travels/Vietnam--Tradition-and-Today::
      2. Click rename action
      3. Write Hut-to-Hut-in-the-Swiss-Alps into the new name field

      Expected results

      Validation error is shown below the correct field.

      Actual results

      Validation error is shown at below the first field.

      Workaround

      Use the default stacked layout.
       

      Development notes

      https://git.magnolia-cms.com/projects/PLATFORM/repos/ui/browse/magnolia-ui-framework/src/main/java/info/magnolia/ui/framework/layout/HorizontalLayoutProducer.java?at=release%2F6.2 doesn't use the field layout producer as the default layout https://git.magnolia-cms.com/projects/PLATFORM/repos/ui/browse/magnolia-ui-framework/src/main/java/info/magnolia/ui/framework/layout/StackedLayoutProducer.java#59

        Acceptance criteria

              Unassigned Unassigned
              rkovarik Roman Kovařík
              Nucleus
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Created:
                Updated:
                Resolved:

                  Bug DoR
                  Task DoD