1066639 - Fix several regressions in cards view accessibility.

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::System, defect)

Description

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

After bug 1061324 landed cards view accessibility regressed. Things that need to be done:

* Enable pointer events for cards buttons
* Enable navigation to the app when the preview is activated
* Make sure statusbar is actually invisible
* Make sure notification is actually invisible

Comment 1

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

Attachment: Github pull request.

Comment 2

[(inactive after 6/18) Alive Kuo [:alive]]

Comment on attachment 8488772 [details] [review]
Github pull request.

Don't think there's problem but still f?=sfoster

Comment 3

[Zac C (:zac)]

Comment on attachment 8488772 [details] [review]
Github pull request.

The test is unusually long and could be prone to intermittents but it runs OK on 
device too so I effectively r+ it, but....

r- on the smallest nit: it needs to also be added to tbpl-manifest.ini so that this test will Gaia-Try run it.  (expect this file will be deprecated before the end of the week but in the meantime we need to do it this way).

Comment 4

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

Comment on attachment 8488772 [details] [review]
Github pull request.

Added a test to tbpl manifest.

Comment 5

[Zac C (:zac)]

Comment on attachment 8488772 [details] [review]
Github pull request.

r+ looks good now.

Comment 6

[Sam Foster [:sfoster] (he/him)]

Comment on attachment 8488772 [details] [review]
Github pull request.

Looks good to me also

Comment 7

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

https://github.com/mozilla-b2g/gaia/commit/f4e4858733e19ccfc278b7fe7e01dd7a16fb456b

Comment 8

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

Comment on attachment 8488772 [details] [review]
Github pull request.

Cards view accessibility(a11y) was worked on during 2.1 however some new features landed late enough and broke some of the a11y.
[Bug caused by] (feature/regressing bug #): bug 1061324
[User impact] if declined: Cards view would be significantly inaccessible. The users will have several unlabelled buttons, also they won't be able to select apps by clicking on the preview screen.
[Testing completed]: Added substantial gaia ui tests.
[Risk to taking this patch] (and alternatives if risky): Relatively low. Just minor changes to what already been worked on accessibility wise.
[String changes made]: None

Comment 9

[[:fabrice] Fabrice Desré]

Comment on attachment 8488772 [details] [review]
Github pull request.

As far as I can tell bug 1061324 didn't land on 2.1, so we don't have any regression fix to uplift on 2.1 either.

Comment 10

[Sam Foster [:sfoster] (he/him)]

(In reply to Fabrice Desré [:fabrice] from comment #9)
> Comment on attachment 8488772 [details] [review]
> Github pull request.
> 
> As far as I can tell bug 1061324 didn't land on 2.1, so we don't have any
> regression fix to uplift on 2.1 either.

That's right, the decision was to not uplift, so this patch doesnt need to be there either (in fact wouldn't merge at all). Despite any misgivings I might have about this decision, I'm not going to go ahead and uplift anyway :)