924284 - [AccessFu] Utter the value of input widgets.

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

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

In cases such as <input type="range"> and its value changing, we need to utter the value changed.

Comment 1

[Eitan Isaacson [:eeejay]]

.. and the current value when you land on it.

Comment 2

[Eitan Isaacson [:eeejay]]

Attachment: Output accessible values

Comment 3

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

Comment on attachment 816041 [details] [diff] [review]
Output accessible values

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

While running all jsat test I got 1 failure:

63 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/a11y/accessible/tests/mochitest/jsat/test_explicit_names.html | Context output is correct for textarea1 (output: text area, This is the text area text., Test Text Area) == (expected: text area, Test Text Area, This is the text area text.)

Since the value is now included before the name and the sub-tree is ignored the new output is actually correct.

Other than the above, looks good so r=me.

::: accessible/src/jsat/OutputGenerator.jsm
@@ +12,5 @@
>  const INCLUDE_DESC = 0x01;
>  const INCLUDE_NAME = 0x02;
> +const INCLUDE_VALUE = 0x04;
> +const INCLUDE_CUSTOM = 0x08;
> +const NAME_FROM_SUBTREE_RULE = 0x10;

Perhaps we should use 0x16 to make sure there's never overlap between NAME_FROM_SUBTREE_RULE and INCLUDE_CUSTOM or NAME_FROM_SUBTREE_RULE and INCLUDE_NAME.

::: accessible/tests/mochitest/jsat/output.js
@@ +24,5 @@
>    var context = new PivotContext(accessible, oldAccessible);
>    var output = aGenerator.genForContext(context).output;
>  
>    isDeeply(output, expected,
> +    "Context output is correct for " + aAccOrElmOrID + " (output: " + output.join(', ') + ') == (expected: ' + expected.join(', ') + ')');

NIT: line too long, mixed single and double quotes.

Comment 4

[Eitan Isaacson [:eeejay]]

(In reply to Yura Zenevich [:yzen] from comment #3)
> Comment on attachment 816041 [details] [diff] [review]
> Output accessible values
> 
> Review of attachment 816041 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> While running all jsat test I got 1 failure:
> 
> 63 ERROR TEST-UNEXPECTED-FAIL |
> chrome://mochitests/content/a11y/accessible/tests/mochitest/jsat/
> test_explicit_names.html | Context output is correct for textarea1 (output:
> text area, This is the text area text., Test Text Area) == (expected: text
> area, Test Text Area, This is the text area text.)
> 
> Since the value is now included before the name and the sub-tree is ignored
> the new output is actually correct.
> 

Oops, fixed.

> Other than the above, looks good so r=me.
> 
> ::: accessible/src/jsat/OutputGenerator.jsm
> @@ +12,5 @@
> >  const INCLUDE_DESC = 0x01;
> >  const INCLUDE_NAME = 0x02;
> > +const INCLUDE_VALUE = 0x04;
> > +const INCLUDE_CUSTOM = 0x08;
> > +const NAME_FROM_SUBTREE_RULE = 0x10;
> 
> Perhaps we should use 0x16 to make sure there's never overlap between
> NAME_FROM_SUBTREE_RULE and INCLUDE_CUSTOM or NAME_FROM_SUBTREE_RULE and
> INCLUDE_NAME.
> 

I think you are thinking in decimal :)

> ::: accessible/tests/mochitest/jsat/output.js
> @@ +24,5 @@
> >    var context = new PivotContext(accessible, oldAccessible);
> >    var output = aGenerator.genForContext(context).output;
> >  
> >    isDeeply(output, expected,
> > +    "Context output is correct for " + aAccOrElmOrID + " (output: " + output.join(', ') + ') == (expected: ' + expected.join(', ') + ')');
> 
> NIT: line too long, mixed single and double quotes.

Fixed.

Comment 5

[Eitan Isaacson [:eeejay]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/0e82e8f1c85f

Landed. Yura, if you are doing any work on value changed events, lets keep it in this bug.

Comment 6

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/0e82e8f1c85f