1182222 - [AccessFu][Androd] local menu improvements

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

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

Here's a list of potential improvements we can take on:

* line granularity
* section navigation (like landmark)
* list navigation
* control navigation  -> we might be able to do that based on things being actionable
* special content navigation [??]

Comment 1

[Eitan Isaacson [:eeejay]]

Attachment: Make Layerview support accessibility HTML navigation.

This patch makes the local context menu identical to Chrome. The behavior is slightly different for the different granularity settings.
* Character/Word/Paragraph act the same as they did before.
* "Control" skips between actionable items, including links.
* "List" skips to the first item on any list on the page. This is instead of putting the cursor on the entire list.
* "Section" skips to landmarks. I think it would be smart if this also went to headings, since that is de-facto how pages are organized when they don't use other kinds of landmarks. WHat do you think?

I'll add a link to a try run for a quick doenload of a build.

Comment 2

[Eitan Isaacson [:eeejay]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=e95056ba473d

Comment 3

[Marco Zehe (:MarcoZ)]

When I try to open the build directory for this try build, I get an "artifact not found" error message. :(

Comment 4

[Marco Zehe (:MarcoZ)]

Comment on attachment 8651221 [details] [diff] [review]
Make Layerview support accessibility HTML navigation.

cancelling feedback until we have a working try build. Tried to build locally, but for some reason the patch doesn't apply cleanly, and afterwards I get build errors.

Comment 5

[Eitan Isaacson [:eeejay]]

Attachment: Make Layerview support accessibility HTML navigation.

Rebased.

Comment 6

[Marco Zehe (:MarcoZ)]

Comment on attachment 8652529 [details] [diff] [review]
Make Layerview support accessibility HTML navigation.

This works partially. The normal navigation works. The Navigation by Control Types also works, but I got neither lists, nor special items, nor sections to work.

Also, if I set TalkBack to show the context menu as a list, when returning to Fennec after dismissing that list, the new setting is not retained, but reset to the default granularity.

Comment 7

[Marco Zehe (:MarcoZ)]

Eitan, have you been able to verify my findings from the previous comment? What's the status here?

Comment 8

[Eitan Isaacson [:eeejay]]

Indeed! I have incorporated all of your input. Expect a new patch soon.

Comment 9

[Eitan Isaacson [:eeejay]]

Attachment: Make Layerview support accessibility HTML navigation.

Comment 10

[Marco Zehe (:MarcoZ)]

Comment on attachment 8659477 [details] [diff] [review]
Make Layerview support accessibility HTML navigation.

This patch works as expected! Including showing the local context menu as list.Also the different granularities work as I would expect them to.

Comment 11

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

Comment on attachment 8659477 [details] [diff] [review]
Make Layerview support accessibility HTML navigation.

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

JS part looks good, see one suggestion

::: accessible/jsat/AccessFu.jsm
@@ +311,4 @@
>        case 'Accessibility:PreviousObject':
> +      {
> +        let rule = aData ?
> +          aData.substr(0, 1).toUpperCase() + aData.substr(1).toLowerCase() :

can also do
aData.toLowerCase().replace(/^./, c => c.toUpperCase())

Comment 12

[Mark Finkle (:mfinkle) (use needinfo?)]

Comment on attachment 8659477 [details] [diff] [review]
Make Layerview support accessibility HTML navigation.

Looks OK to me.

Comment 13

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/2ed46aca6ae0

Comment 14

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

https://hg.mozilla.org/mozilla-central/rev/2ed46aca6ae0

Comment 15

[vortex]

@Eeejay: I think including headings in section navigattion is a very good idea. Few sites have landmarks, but almost all do have headings....