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

Required slider field doesn't accept min value

    XMLWordPrintable

Details

    • Bug
    • Resolution: Unresolved
    • Neutral
    • None
    • 6.2.14
    • None
    • None

    Description

      Steps to reproduce

      1. Open https://nightly.magnolia-cms.com/.magnolia/admincentral#app:resources:edit;/tours/apps/tours.yaml:edit
      2.  Configure required slider field:
          detail:
            form:
              properties:
                name:
                  i18n: true
                slider:
                  $type: sliderField 
                  required: true
        
        
      1. Save changes
      2. Open https://nightly.magnolia-cms.com/.magnolia/admincentral#app:tours:detail;/:add
      3. Fill in required fields (name, image)
      4. Touch the slider but choose the min value in the end.
      5. Click "Save changes"

      Expected results

      User can pick the min value (unless the documentation explicitly mentions that https://docs.magnolia-cms.com/product-docs/6.2/Developing/Templating/Dialog-definition/Field-definition/List-of-fields/Slider-field.html#_field_specific_properties).

      Actual results

      "This field is required" in the slider field

      Workaround

      Set the min value one step below the real accepted minum value.

      Development notes

      The  current impl considers the empty value as equal min value. https://git.magnolia-cms.com/projects/PLATFORM/repos/ui.pub/browse/magnolia-ui-framework/src/main/java/info/magnolia/ui/field/SliderField.java#153.

      Tip: How textField with com.vaadin.data.converter.StringToDoubleConverter handles the empty value?

       

      Checklists

        Acceptance criteria

        Attachments

          Activity

            People

              Unassigned Unassigned
              bsandala Brano Sandala
              Nucleus
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:

                Checklists

                  Bug DoR
                  Task DoD