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

          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