Details

    • Testing Instructions:
      Hide
      1. Create a database instance in a course, add this types of fields "date", "file", "latlong", "menu", "multimenu", "number", "picture", "radiobutton", "text", "textarea" and "url" verifying there are labels with the accesshide class in the form elements
      Show
      Create a database instance in a course, add this types of fields "date", "file", "latlong", "menu", "multimenu", "number", "picture", "radiobutton", "text", "textarea" and "url" verifying there are labels with the accesshide class in the form elements
    • Affected Branches:
      MOODLE_22_STABLE, MOODLE_23_STABLE, MOODLE_24_STABLE
    • Fixed Branches:
      MOODLE_22_STABLE, MOODLE_23_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-34563_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

          Attachments

            Issue Links

              Activity

              Hide
              nebgor Aparup Banerjee added a comment -

              Hi Rosie,

              https://github.com/rwijaya/moodle/compare/master...MDL-34563_accessibility#L1R46

              The plural strings have to be used instead. all else looks fine here.

              Show
              nebgor Aparup Banerjee added a comment - Hi Rosie, https://github.com/rwijaya/moodle/compare/master...MDL-34563_accessibility#L1R46 The plural strings have to be used instead. all else looks fine here.
              Hide
              rwijaya Rossiani Wijaya added a comment -

              Patch updated.

              Thanks Apu.

              Show
              rwijaya Rossiani Wijaya added a comment - Patch updated. Thanks Apu.
              Hide
              nebgor Aparup Banerjee added a comment -

              Rosie, MDL-34568 has shed some light so i think i need more time to review this, so its for next week.

              Please reopen if any of Tim's comments apply here too.

              Show
              nebgor Aparup Banerjee added a comment - Rosie, MDL-34568 has shed some light so i think i need more time to review this, so its for next week. Please reopen if any of Tim's comments apply here too.
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              The integration of this issue has been delayed to next week because the integration period is over (Monday, Tuesday) and testing must happen on Wednesday.

              This change to a more rigid timeframe on each integration/testing cycle aims to produce a better and clear separation and organization of tasks for everybody.

              This is a bulk-automated message, so if you want to blame somebody/thing/where, don't do it here (use git instead) :-D :-P

              Apologizes for the inconvenient, this will be integrated next week. Thanks for your collaboration & ciao

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - The integration of this issue has been delayed to next week because the integration period is over (Monday, Tuesday) and testing must happen on Wednesday. This change to a more rigid timeframe on each integration/testing cycle aims to produce a better and clear separation and organization of tasks for everybody. This is a bulk-automated message, so if you want to blame somebody/thing/where, don't do it here (use git instead) :-D :-P Apologizes for the inconvenient, this will be integrated next week. Thanks for your collaboration & ciao
              Hide
              nebgor Aparup Banerjee added a comment - - edited

              Hi Rosie,
              I 've had a closer look this time and its all good, just one quick fix needed:

              • multimenu/field.class.php line 41 : we shouldn't have a colon for accesshide labels (don't need to read out colon from reader)
              • last commit needs 'MDL'
              Show
              nebgor Aparup Banerjee added a comment - - edited Hi Rosie, I 've had a closer look this time and its all good, just one quick fix needed: multimenu/field.class.php line 41 : we shouldn't have a colon for accesshide labels (don't need to read out colon from reader) last commit needs 'MDL'
              Hide
              rwijaya Rossiani Wijaya added a comment -

              Patch updated.

              Thanks Apu.

              Show
              rwijaya Rossiani Wijaya added a comment - Patch updated. Thanks Apu.
              Hide
              nebgor Aparup Banerjee added a comment -

              cool, those labels are now integrated into 22, 23 and master.

              Show
              nebgor Aparup Banerjee added a comment - cool, those labels are now integrated into 22, 23 and master.
              Hide
              dmonllao David Monllaó added a comment -

              Commented with Rossiani, waiting for tomorrow to pass or fail it just in case it can be integred with the new patch

              • Search -> Author first name -> No label
              • Search -> Author surname
              • Search -> file field -> Wrong double quote in for attribute
              • New entry -> Date field -> Duplicate labels
              • New entry -> Number field -> No label
              • New entry -> Text field -> No label
              • Add field -> File field -> Maximum size attr -> The for has to point to "param3"
              • Add field -> Picture field -> Maximum size attr -> The for has to point to "param3"
              • Add field -> URL field -> Forced name... attr -> Label pointing to the element name instead of "forcelinkname"
              Show
              dmonllao David Monllaó added a comment - Commented with Rossiani, waiting for tomorrow to pass or fail it just in case it can be integred with the new patch Search -> Author first name -> No label Search -> Author surname Search -> file field -> Wrong double quote in for attribute New entry -> Date field -> Duplicate labels New entry -> Number field -> No label New entry -> Text field -> No label Add field -> File field -> Maximum size attr -> The for has to point to "param3" Add field -> Picture field -> Maximum size attr -> The for has to point to "param3" Add field -> URL field -> Forced name... attr -> Label pointing to the element name instead of "forcelinkname"
              Hide
              rwijaya Rossiani Wijaya added a comment -

              Thanks David.

              Patch has been updated to fix the above comments.

              Integrator please integrate the new patch.

              Thanks.

              Show
              rwijaya Rossiani Wijaya added a comment - Thanks David. Patch has been updated to fix the above comments. Integrator please integrate the new patch. Thanks.
              Hide
              nebgor Aparup Banerjee added a comment -

              wowie, good catches David! sorry i missed all that, must've been blind there!

              Show
              nebgor Aparup Banerjee added a comment - wowie, good catches David! sorry i missed all that, must've been blind there!
              Hide
              rwijaya Rossiani Wijaya added a comment -

              Updated the patch.

              Sorry for the hassle Apu.

              Show
              rwijaya Rossiani Wijaya added a comment - Updated the patch. Sorry for the hassle Apu.
              Hide
              nebgor Aparup Banerjee added a comment -

              (github is down here atm )

              Show
              nebgor Aparup Banerjee added a comment - (github is down here atm )
              Hide
              nebgor Aparup Banerjee added a comment -

              ok

              • ace963d MDL-34563 - change string firstname to lastname and add accesshide class
              • 8f46d48 MDL-34563 - removed double labeling and add label for input text
                (master)
                have been picked across to 22, 23 and master.

              This should've been well tested by now thanks to David's sharp eyes

              ps:
              we do need peer-reviewing here for the next few label ones as these are very intricate changes better seen by many eyes as possible.

              So i'd suggest to peer-review them again even if they were broken down into many issues and evidence is showing it could do with more reviews.

              Show
              nebgor Aparup Banerjee added a comment - ok ace963d MDL-34563 - change string firstname to lastname and add accesshide class 8f46d48 MDL-34563 - removed double labeling and add label for input text (master) have been picked across to 22, 23 and master. This should've been well tested by now thanks to David's sharp eyes ps: we do need peer-reviewing here for the next few label ones as these are very intricate changes better seen by many eyes as possible. So i'd suggest to peer-review them again even if they were broken down into many issues and evidence is showing it could do with more reviews.
              Hide
              dmonllao David Monllaó added a comment -

              Tested in master and 22, it passes now

              Show
              dmonllao David Monllaó added a comment - Tested in master and 22, it passes now
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              For the good and the bad... this is now part of Moodle and people around the world will start using it immediately, what a responsibility!

              Many thanks for your collaboration, yay!

              Closing, ciao

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - For the good and the bad... this is now part of Moodle and people around the world will start using it immediately, what a responsibility! Many thanks for your collaboration, yay! Closing, ciao

                People

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

                  Dates

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