Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-35816 Accessibility Review issues (Deque)
  3. MDL-36882

Manual activity completion input image doesn't change title or alt text when checked as completed

    Details

    • Testing Instructions:
      Hide
      1. Enable completion tracking (Home ► Site administration ► Advanced features)
      2. Create an activity with completion tracking set to "Students can manually mark the activity as completed"
      3. As student click on the activity completion icon image (right of the activity) to mark the activity complete.
      4. Check the alt text and title change appropriately.
      5. With NVDA or JAWS check you are informed that a change has occurred.
      Show
      Enable completion tracking (Home ► Site administration ► Advanced features) Create an activity with completion tracking set to "Students can manually mark the activity as completed" As student click on the activity completion icon image (right of the activity) to mark the activity complete. Check the alt text and title change appropriately. With NVDA or JAWS check you are informed that a change has occurred.
    • Affected Branches:
      MOODLE_23_STABLE, MOODLE_24_STABLE
    • Fixed Branches:
      MOODLE_23_STABLE, MOODLE_24_STABLE
    • Pull Master Branch:
      wip-mdl-36882

      Description

      Issue
      When an activity is set so that a student can manually mark it complete, and a student clicks on the input image to the right of the activity to mark the activity complete, the alt text and title do not change. The user is also not informed that a change has occurred to the screen.

      View the video at http://screencast.com/t/BUEZjNBRu

      Standard Level
      WCAG 2 1.1.1 (A) http://www.w3.org/WAI/WCAG20/quickref/#qr-text-equiv-all
      WCAG 2 4.1.2 (A) http://www.w3.org/WAI/WCAG20/quickref/#qr-ensure-compat-rsv

      Impact
      Serious

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            rajeshtaneja Rajesh Taneja added a comment -

            Thanks for reporting this Jason,

            Alt and title is changing now. I have added aria-live attribute so that assistive technologies notify it's change.

            Show
            rajeshtaneja Rajesh Taneja added a comment - Thanks for reporting this Jason, Alt and title is changing now. I have added aria-live attribute so that assistive technologies notify it's change.
            Hide
            ankit_frenz Ankit Agarwal added a comment -

            Hi Raj,
            Changes looks good. Can you please add some testing instructions before submitting for integration?
            Thanks

            Show
            ankit_frenz Ankit Agarwal added a comment - Hi Raj, Changes looks good. Can you please add some testing instructions before submitting for integration? Thanks
            Hide
            rajeshtaneja Rajesh Taneja added a comment -

            Thanks Ankit.

            Show
            rajeshtaneja Rajesh Taneja added a comment - Thanks Ankit.
            Hide
            damyon Damyon Wiese added a comment -

            Thanks Raj, lovely work! Integrated to 23, 24 and master.

            Show
            damyon Damyon Wiese added a comment - Thanks Raj, lovely work! Integrated to 23, 24 and master.
            Hide
            abgreeve Adrian Greeve added a comment -

            Tested on the 2.3, 2.4 and master integration branches.
            The alt text and title change as expected.
            Test passed.

            Show
            abgreeve Adrian Greeve added a comment - Tested on the 2.3, 2.4 and master integration branches. The alt text and title change as expected. Test passed.
            Hide
            damyon Damyon Wiese added a comment -

            Thanks for your hard work. This issue has been integrated upstream and is now available via git (and in some hours, via mirrors and downloads).

            Show
            damyon Damyon Wiese added a comment - Thanks for your hard work. This issue has been integrated upstream and is now available via git (and in some hours, via mirrors and downloads).
            Hide
            ankit_frenz Ankit Agarwal added a comment -

            -1 for qa tests, not worth it as it was really minor change

            Show
            ankit_frenz Ankit Agarwal added a comment - -1 for qa tests, not worth it as it was really minor change
            Hide
            rajeshtaneja Rajesh Taneja added a comment -

            Thanks for the input Ankit,

            I was trying to identify few issues which can be added to QA for accessibility testing.
            As we don't want any exploratory tests for this, we were looking for specific and simple tests and this came up to be one of them.

            FYI: Damyon Wiese gave +1 to this.

            If you still think it is not required them I will remove this.

            Show
            rajeshtaneja Rajesh Taneja added a comment - Thanks for the input Ankit, I was trying to identify few issues which can be added to QA for accessibility testing. As we don't want any exploratory tests for this, we were looking for specific and simple tests and this came up to be one of them. FYI: Damyon Wiese gave +1 to this. If you still think it is not required them I will remove this.
            Hide
            ankit_frenz Ankit Agarwal added a comment -

            Well for accessibility testing I would suggest running some of the existing QAs with screen reader or something which would be much more beneficial then this minor issue. But you are the assignee, upto you to make the final call.

            cheers

            Show
            ankit_frenz Ankit Agarwal added a comment - Well for accessibility testing I would suggest running some of the existing QAs with screen reader or something which would be much more beneficial then this minor issue. But you are the assignee, upto you to make the final call. cheers

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  13/May/13