Closed
Bug 655056
Opened 14 years ago
Closed 14 years ago
Add alternative text to quiz progress bar
Categories
(Websites :: webifyme.org, defect, P3)
Websites
webifyme.org
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: williamr, Assigned: adam)
References
()
Details
(Keywords: access)
Per bug 653166 comment 3, the progress bar shown the below the quiz questions does not include proper alternative text.
Let's add alternative text to the progress bar that lets the user know how far into the quiz they are, either the number of questions remaining or percentage complete.
| Reporter | ||
Comment 1•14 years ago
|
||
Copy for the alternative text should match what's shown in the blue tooltip below the progress bar. As in, "XY MORE TO GO" where XY is the number of questions left to answer.
Comment 2•14 years ago
|
||
Is this string committed to git yet? We need to prepare for string freeze.
Updated•14 years ago
|
Assignee: nobody → williamr
Comment 3•14 years ago
|
||
Per williamr in IRC, this string isn't committed yet.
To remove the blocking status of Bug#650033, please add an L10n comment and this string to any template.
L10n: %s is the number of questions remaining.
"%s more to go"
We can build the functionality and close this bug later.
Note; lower case, since this is for screen readers and not a visual presentation.
Assignee: williamr → jbresnik
Comment 4•14 years ago
|
||
Stas or Gandlaf: Should ngettext be used for this string?
Comment 5•14 years ago
|
||
Yes, great catch, Austin. In some languages it might be necessary to explicitly call out "questions" in this string, e.g. "n questions left". Using ngettext will ensure that "questions" are in correct plural form in these languages.
Also, should it be %s or %d?
William, still up for it? :) I'll be happy to guide you through ngettext if you wish.
Comment 6•14 years ago
|
||
Stas can you review this commit?
https://github.com/mozilla/webifyme/commit/5953f678ee01f1125088001e2599e4969533e5b1
This fixes the L10n aspect of this bug, but not the accessibility aspect. Leaving to TBG to fix and close.
Comment 7•14 years ago
|
||
(In reply to comment #6)
I forgot to add an L10n comment.
https://github.com/mozilla/webifyme/commit/851e8d14a77a24fbc80e3bd12875ed76316eeff4
| Reporter | ||
Comment 8•14 years ago
|
||
(In reply to comment #5)
> William, still up for it? :) I'll be happy to guide you through ngettext if
> you wish.
Looks like ozten beat me to the ngettext part (thanks ozten!). I can try adding the alt text, but I can't work on it until Friday afternoon PST at the earliest.
| Assignee | ||
Updated•14 years ago
|
Assignee: jbresnik → adam
| Assignee | ||
Comment 9•14 years ago
|
||
Can someone explain how the tooltip is insufficient in providing alternative text to the progress bar graphic?
If you examine the HTML building that component, you'll see that the last child node is the text that is used to render the label, and the graphic itself is rendered as the background image of the containing div. There's no img tag to add an alt attribute to, so I can't thinking of a better way to add alternate text than what is already there.
That string is also being translated for digits 0-20.
Comment 10•14 years ago
|
||
(In reply to comment #9)
I didn't realize this was feature complete.
Marco, can you verify?
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•