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
    • Rank:
      43008

      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

        Issue Links

          Activity

          Hide
          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
          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
          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
          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
          Dan Poltawski added a comment -

          Done. Please test.

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

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

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

          Fixed STOP Closed STOP Thanks STOP

          Yay, imagination! Ciao

          Show
          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: