Details

    • Type: Sub-task
    • Status: Closed
    • Priority: 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:

      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.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              timhunt 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
              timhunt 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
              timhunt 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
              timhunt 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
              timhunt Tim Hunt added a comment -

              So, I think this is as simple as adding a

              role="timer"

              attribute to the appropriate HTML element.

              Show
              timhunt Tim Hunt added a comment - So, I think this is as simple as adding a role="timer" attribute to the appropriate HTML element.
              Hide
              timhunt 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
              timhunt 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
              samhemelryk 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
              samhemelryk 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
              samhemelryk Sam Hemelryk added a comment -

              (wrong button sorry)

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

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

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

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

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

              Thanks Jason. Submitting for integration now.

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

              Integrated, thanks Tim.

              Show
              poltawski Dan Poltawski added a comment - Integrated, thanks Tim.
              Hide
              salvetore 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
              salvetore 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
              timhunt 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
              timhunt 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
              poltawski 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
              poltawski 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:
                    Fix Release Date:
                    14/Jan/13