1166321 - [AccessFu] Add support for role="switch" accessible elements.

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

[Yura Zenevich [:yzen] (please use "needinfo")]

Right now role="switch" is not supported though it would be super useful for gaia-components and in Firefox OS in general.

Comment 1

[Yura Zenevich [:yzen] (please use "needinfo")]

Attachment: 1166321 patch v1

Comment 2

[Eitan Isaacson [:eeejay]]

Comment on attachment 8608727 [details] [diff] [review]
1166321 patch v1

Review of attachment 8608727 [details] [diff] [review]:
-----------------------------------------------------------------

::: accessible/jsat/OutputGenerator.jsm
@@ +812,4 @@
>        aBraille.push({
> +        string: aRoleStr === 'switch' ?
> +          this._getOutputName(checked ? 'stateOn' : 'stateOff') :
> +          this._getOutputName(checked ? 'stateChecked' : 'stateUnchecked')

The state displayed for both roles is identical. Do we really need this distinction?

::: dom/locales/en-US/chrome/accessibility/AccessFu.properties
@@ +127,5 @@
>  pressAction    =      pressed
>  checkAction    =      checked
>  uncheckAction  =      unchecked
> +onAction       =      on
> +offAction      =      off

Ideally, this would be a past tense verb like the rest of the actions. But that might be a bit too chatty: "turned on"

@@ +216,5 @@
>  outlineitemAbbr    =       outline item
>  pagetabAbbr        =       tab
>  propertypageAbbr   =       property page
>  graphicAbbr        =       graphic
> +switchAbbr         =       switch

You could maybe remove the vowel and just have swtch, but someone who uses a braille display would know better. Also, I don't think this is ever displayed anyway, because the state is used.

Comment 3

[Yura Zenevich [:yzen] (please use "needinfo")]

https://hg.mozilla.org/integration/mozilla-inbound/rev/7526d40755d0

Comment 4

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/7526d40755d0