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

moodleforms unique id on field div wrappers

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide

      note to tester: you could test the related MDL-31469 at the same time

      For a variety of forms in Moodle (I tested on the quiz settings form, so I would suggest you test on some other complex forms like
      a. add forum,
      b. create multichoice question,
      c. ...)

      1. Go to the form, and make sure it looks/works OK>
      2. Use a tool like firebug to instpect the HTML, and marvel at the beauty of the new id and class attributes on the outer wrapper div for each form field.
      3. Validate the HTML of the page, and make sure there are no problems relating to the ids. (Note that there are other problems, which may include MDL-31469.)

      Show
      note to tester: you could test the related MDL-31469 at the same time For a variety of forms in Moodle (I tested on the quiz settings form, so I would suggest you test on some other complex forms like a. add forum, b. create multichoice question, c. ...) 1. Go to the form, and make sure it looks/works OK> 2. Use a tool like firebug to instpect the HTML, and marvel at the beauty of the new id and class attributes on the outer wrapper div for each form field. 3. Validate the HTML of the page, and make sure there are no problems relating to the ids. (Note that there are other problems, which may include MDL-31469 .)
    • Affected Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE, MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE
    • Pull from Repository:
    • Pull Master Branch:

      Description

      It would be useful, when authoring CSS, if MoodleForms gave a unique id to the top level div wrapping each label and field.

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              anthonyforth Anthony Forth
              Reporter:
              anthonyforth Anthony Forth
              Peer reviewer:
              Tim Hunt
              Integrator:
              Aparup Banerjee
              Tester:
              Adrian Greeve
              Participants:
              Component watchers:
              Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias, Bas Brands
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                12/Mar/12