Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-67869

Review form elements missing the required indicator

    XMLWordPrintable

Details

    • Bug
    • Status: Open
    • Minor
    • Resolution: Unresolved
    • 3.7.4, 3.8, 3.9
    • None
    • Forms Library
    • None
    • MOODLE_37_STABLE, MOODLE_38_STABLE, MOODLE_39_STABLE

    Description

      This is a followup / was discovered by MDL-63424, where the missing required indicator was added to the checkbox form element.

      While looking for cases, it was detected that a number of form elements don't currently support (apparently) that required indicators. Here it's the initial list:

      $ grep -Lr 'element-template' *
      editor_textarea.mustache
      element-advcheckbox-inline.mustache
      element-advcheckbox.mustache
      element-checkbox-inline.mustache
      element-duration.mustache
      element-filetypes.mustache
      element-modgrade.mustache
      element-modvisible.mustache
      element-passwordunmask-fill.mustache
      element-questioncategory-inline.mustache
      element-questioncategory.mustache
      element-radio-inline.mustache
      element-radio.mustache
      element-selectyesno-inline.mustache
      element-selectyesno.mustache
      element-template-inline.mustache
      filetypes-browser.mustache
      filetypes-descriptions.mustache
      filetypes-trigger.mustache
      

      Note that some may be ok as they are (no sense to have the required element for them) and others could have a custom implementation (like checkbox or radio do).

      But the list above needs to be reviewed and, when needed, support for the required indicator, added.

      Ideally covered with tests so we don't forget about them again.

      Ciao

      Attachments

        Issue Links

          Activity

            People

              Unassigned Unassigned
              stronk7 Eloy Lafuente (stronk7)
              David Woloszyn, Huong Nguyen, Jake Dallimore, Meirza, Michael Hawkins, Raquel Ortega, Safat Shahin, Stevani Andolo
              Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

                Created:
                Updated: