Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1.1, 2.2.6, 2.3.3
    • Fix Version/s: 2.2.7, 2.3.4
    • Component/s: Accessibility, Quiz
    • Labels:
    • Rank:
      33893

      Description

      The timer needs to be an ARIA live region so screen reader users know that there is a live updating section of the page and that its role is a timer.

        Issue Links

          Activity

          Hide
          Tim Hunt added a comment -

          Again, you can make it easier for me to fix this if you can tell me how to add ARIA mark-up to the page.

          Show
          Tim Hunt added a comment - Again, you can make it easier for me to fix this if you can tell me how to add ARIA mark-up to the page.
          Hide
          Tim Hunt added a comment -

          Greg, any chance you could make my life easy, and tell me the correct ARIA mark-up for this, like you did for the question flag one? Thanks.

          Show
          Tim Hunt added a comment - Greg, any chance you could make my life easy, and tell me the correct ARIA mark-up for this, like you did for the question flag one? Thanks.
          Hide
          Tim Hunt added a comment -

          So, I think this is as simple as adding a

          role="timer"

          attribute to the appropriate HTML element.

          Show
          Tim Hunt added a comment - So, I think this is as simple as adding a role="timer" attribute to the appropriate HTML element.
          Hide
          Tim Hunt added a comment -

          I think this is the right way to do it, but I am not an expert in ARIA, so please can someone peer-review this.

          Show
          Tim Hunt added a comment - I think this is the right way to do it, but I am not an expert in ARIA, so please can someone peer-review this.
          Hide
          Sam Hemelryk added a comment -

          Hi Tim,

          What you are purposing here looks spot on to me. I'm not the office expert but I have helped out a couple of the guys with issues they've worked on re: aria.
          If you would like a second vote on this I would suggest including Jason as a watcher and asking his opinion, I understand he is the in office expert.

          Gets a +1 from me.

          Cheers
          Sam

          Show
          Sam Hemelryk added a comment - Hi Tim, What you are purposing here looks spot on to me. I'm not the office expert but I have helped out a couple of the guys with issues they've worked on re: aria. If you would like a second vote on this I would suggest including Jason as a watcher and asking his opinion, I understand he is the in office expert. Gets a +1 from me. Cheers Sam
          Hide
          Sam Hemelryk added a comment -

          (wrong button sorry)

          Show
          Sam Hemelryk added a comment - (wrong button sorry)
          Hide
          Tim Hunt added a comment -

          Jason, I would be grateful if you could peer review this. Thanks.

          Show
          Tim Hunt added a comment - Jason, I would be grateful if you could peer review this. Thanks.
          Hide
          Jason Fowler added a comment -

          Perfect Tim, although the atomic attribute may be over kill. No harm in using it anyway.

          Show
          Jason Fowler added a comment - Perfect Tim, although the atomic attribute may be over kill. No harm in using it anyway.
          Hide
          Tim Hunt added a comment -

          Thanks Jason. Submitting for integration now.

          Show
          Tim Hunt added a comment - Thanks Jason. Submitting for integration now.
          Hide
          Dan Poltawski added a comment -

          Integrated, thanks Tim.

          Show
          Dan Poltawski added a comment - Integrated, thanks Tim.
          Hide
          Michael de Raadt added a comment -

          I tested this with NVDA. I wasn't sure exactly what to expect this attribute to do to the screen reader. It didn't make the screen reader read the timer unless I moused over the timer itself. I wasn't able to tab to the timer. It wasn't read as part of the normal content. If I was blind, I would still have no notion that there was a timer counting down.

          The documentation for the timer attribute doesn't really say what is supposed to happen.

          http://www.w3.org/WAI/PF/aria/roles#timer

          I'm therefore assuming that my screen reader doesn't know what to do with a timer attribute yet (perhaps other screen readers do). Alternately, I assume that this is the expected behaviour for a timer.

          I can say that this hasn't broken anything. The timer works as it did before.

          Show
          Michael de Raadt added a comment - I tested this with NVDA. I wasn't sure exactly what to expect this attribute to do to the screen reader. It didn't make the screen reader read the timer unless I moused over the timer itself. I wasn't able to tab to the timer. It wasn't read as part of the normal content. If I was blind, I would still have no notion that there was a timer counting down. The documentation for the timer attribute doesn't really say what is supposed to happen. http://www.w3.org/WAI/PF/aria/roles#timer I'm therefore assuming that my screen reader doesn't know what to do with a timer attribute yet (perhaps other screen readers do). Alternately, I assume that this is the expected behaviour for a timer. I can say that this hasn't broken anything. The timer works as it did before.
          Hide
          Tim Hunt added a comment -

          Indeed, it is not clear what should happen.

          Suppose the screen-reader read out the time every time it changed (every second!) that would be horrible.

          I suppose that, if I was designing a screen-reader, I might think a sensible design was:

          • announce that there is a timer on-screen when the page loads; and
          • provide a keyboard short-cut to easily speak the timer value at any time.
          Show
          Tim Hunt added a comment - Indeed, it is not clear what should happen. Suppose the screen-reader read out the time every time it changed (every second!) that would be horrible. I suppose that, if I was designing a screen-reader, I might think a sensible design was: announce that there is a timer on-screen when the page loads; and provide a keyboard short-cut to easily speak the timer value at any time.
          Hide
          Dan Poltawski added a comment -

          Congratulations! Another bug solved.. only another 7330 to go, thanks for contributing to contributing to 0.8% of all bugs being fixed this week!

          ciao
          Dan

          Show
          Dan Poltawski added a comment - Congratulations! Another bug solved.. only another 7330 to go, thanks for contributing to contributing to 0.8% of all bugs being fixed this week! ciao Dan

            People

            • Votes:
              1 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: