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

Annotated PDF comments aren't handling tab focus correctly

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Minor Minor
    • 3.3
    • 3.3
    • Assignment
    • MOODLE_33_STABLE
    • MOODLE_33_STABLE
    • MDL-58515-master
    • Hide
      • Create an assignment in a course with at least one student.
      • As a student, submit to the assignment.
      • As a teacher, grade the submission and annotate a few comments.
      • Tab around the grading page and verify that:
        • all comments can be focused (in the correct order),
        • each comment's menu can be focused (after the comment itself),
        • all elements display a focus outline when focused,
        • all of the above also works as expected when reverse (Shift) tabbing.
      • Test with both Clean and Boost themes, and in all supported browsers.
      Show
      Create an assignment in a course with at least one student. As a student, submit to the assignment. As a teacher, grade the submission and annotate a few comments. Tab around the grading page and verify that: all comments can be focused (in the correct order), each comment's menu can be focused (after the comment itself), all elements display a focus outline when focused, all of the above also works as expected when reverse (Shift) tabbing. Test with both Clean and Boost themes, and in all supported browsers.

      The changes made in MDL-55459 have changed the behaviour of the annotated PDF comments in the tab order.

      On Chrome the comments can receive focus however they don't show the focus outline. We need all elements to show the focus outline when they have focus for accessibility reasons.

      On Safari the comments aren't focusable at all using the tab key.

      I believe both issues are caused by having elements with tabindex="0" being nested within elements that have tabindex="-1". This can cause weird behaviour for browsers.

      The focus handling needs to be cleaned up a bit.

            tonybutler Tony Butler
            ryanwyllie Ryan Wyllie
            Damyon Wiese Damyon Wiese
            Andrew Lyons Andrew Lyons
            Simey Lameze Simey Lameze
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

              Created:
              Updated:
              Resolved:

                Error rendering 'clockify-timesheets-time-tracking-reports:timer-sidebar'. Please contact your Jira administrators.