Details

    • Testing Instructions:
      Hide

      Login as admin and take a look the following page:

      1. course participant list check for label for the select input
      2. in course participant list select couple of students and add notes
      3. check for label for the select input in add note page
      4. from course participant list select couple of students and add common notes
      5. check for label for the select input in add note page
      Show
      Login as admin and take a look the following page: course participant list check for label for the select input in course participant list select couple of students and add notes check for label for the select input in add note page from course participant list select couple of students and add common notes check for label for the select input in add note page
    • Affected Branches:
      MOODLE_22_STABLE
    • Fixed Branches:
      MOODLE_22_STABLE, MOODLE_23_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-34575_accessibility

      Description

      There are several select input fields that do not have labels explicitly tied to them. Often this is because there is a visual cue as to what information is being asked for but the visual cue is not explicitly linked to the input element.

      Because this problem is sporadic it might be necessary to break this task out into smaller sub-tasks for each instance of the problem.

      Here is a tutorial for methods of labeling text input elements. http://oit.ncsu.edu/itaccess/forms#select

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            poltawski Dan Poltawski added a comment -

            I've integrated this now, thanks Rosie.

            I did one commit on top to ensure that the lang string was added in alphabetical order. Please be careful about this, by ordering alphabetically we prevent mistakes like adding duplicate strings.

            Show
            poltawski Dan Poltawski added a comment - I've integrated this now, thanks Rosie. I did one commit on top to ensure that the lang string was added in alphabetical order. Please be careful about this, by ordering alphabetically we prevent mistakes like adding duplicate strings.
            Hide
            rwijaya Rossiani Wijaya added a comment -

            Hi Dan,

            Just updated the patch for the string identifier plus label duplication.

            Please push this for integration.

            Thanks

            Show
            rwijaya Rossiani Wijaya added a comment - Hi Dan, Just updated the patch for the string identifier plus label duplication. Please push this for integration. Thanks
            Hide
            poltawski Dan Poltawski added a comment -

            Done. Please test.

            Show
            poltawski Dan Poltawski added a comment - Done. Please test.
            Hide
            abgreeve Adrian Greeve added a comment -

            Tested in 2.2, 2.3, and master.
            Looks good.
            Test passed.

            Show
            abgreeve Adrian Greeve added a comment - Tested in 2.2, 2.3, and master. Looks good. Test passed.
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Fixed STOP Closed STOP Thanks STOP

            Yay, imagination! Ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Fixed STOP Closed STOP Thanks STOP Yay, imagination! Ciao

              People

              • Votes:
                0 Vote for this issue
                Watchers:
                0 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  10/Sep/12