Closed Bug 559764 Opened 10 years ago Closed 7 years ago

make <input type="range"> accessible

Categories

(Core :: Disability Access APIs, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla23

People

(Reporter: surkov, Assigned: surkov)

References

(Depends on 1 open bug, Blocks 1 open bug, )

Details

(Keywords: access)

Attachments

(1 file)

If it's going look like a slider then we could reuse accessible classes for XUL scale and XForms range.
what about render html5 input type=range as a slider ?
it seems nobody works on it although firefox is far behind the other browsers on smart rendering of html5 inputs.
(In reply to comment #1)
> what about render html5 input type=range as a slider ?
> it seems nobody works on it although firefox is far behind the other
> browsers on smart rendering of html5 inputs.

You should put your comment into bug 344618 where the control is supposed to be implemented, this bug is about to make accessible (make it working with screen readers) what done in bug 344618.
Note, input@type="range" got own face in bug 344618 in Firefox 22. It's time to take care about accessibility.
Blocks: 2013q2a11y
Assignee: nobody → surkov.alexander
Attached patch patchSplinter Review
1) a slider role, no underlying indicator - an accessible for a thumb, not sure if anybody needs it, however note XUL:scale has it and MSAA requires it (http://msdn.microsoft.com/en-us/library/windows/desktop/dd373645%28v=vs.85%29.aspx)
2) implements value interface
3) fires value change event
Attachment #739404 - Flags: review?(trev.saunders)
Attachment #739404 - Flags: review?(roc)
Attachment #739404 - Flags: review?(bugs)
Comment on attachment 739404 [details] [diff] [review]
patch

Not sure why you need Step() when there is GetStep().
I'd prefer having just GetStep
Attachment #739404 - Flags: review?(bugs) → review+
(In reply to Olli Pettay [:smaug] from comment #5)
> Comment on attachment 739404 [details] [diff] [review]
> patch
> 
> Not sure why you need Step() when there is GetStep().
> I'd prefer having just GetStep

it seems getters in this file aren't used to have Get prefix. GetStep is protected thus I added Step(). So should I make GetStep() public by moving it to public section?
making it public sounds ok.
(I'm surprised it is protected now.)
Comment on attachment 739404 [details] [diff] [review]
patch

>+HTMLRangeAccessible::NativeRole()
>+{
>+  return roles::SLIDER;
>+}
>+
>+////////////////////////////////////////////////////////////////////////////////
>+// HTMLRangeAccessible<Max>: Widgets

s/<max>//
Attachment #739404 - Flags: review?(trev.saunders) → review+
https://hg.mozilla.org/mozilla-central/rev/f6b9554b2b1c
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla23
You need to log in before you can comment on or make changes to this bug.