Closed Bug 1387057 Opened 2 years ago Closed 2 years ago

[a11y] improve accessibility of onboarding notification bar.


(Firefox :: General, defect, P3)




Firefox 57
Tracking Status
firefox56 --- verified
firefox57 --- verified
firefox58 --- verified


(Reporter: yzen, Assigned: yzen)



(Keywords: access, Whiteboard: [photon-onboarding])


(1 file)

No description provided.
Summary: [a11y] improve accessibility of onboarding notification popup. → [a11y] improve accessibility of onboarding notification bar.
Attachment #8893463 - Flags: review?(dtownsend)
Attachment #8893463 - Flags: review?(dtownsend) → review+
Pushed by
improved accessibliity of onboarding notification bar. r=mossop
Flags: qe-verify+
Priority: -- → P3
QA Contact: jwilliams
Backed out for for linting failure at onboarding.js:984 and failing browser-chrome's browser_onboarding_accessibility.js:

Bug 1377283

Bug 1377298

Bug 1377276

Bug 1387057

Push with failures:

Failure log eslint:
> TEST-UNEXPECTED-ERROR | /home/worker/checkouts/gecko/browser/extensions/onboarding/content/onboarding.js:984:5 | Unsafe assignment to innerHTML (no-unsanitized/property) 

Failure log browser-chrome:

[task 2017-08-04T05:53:12.472863Z] 05:53:12     INFO - TEST-PASS | browser/extensions/onboarding/test/browser/browser_onboarding_accessibility.js | Content should be visible to screen reader - "true" == "true" - 
[task 2017-08-04T05:53:12.475728Z] 05:53:12     INFO - Buffered messages finished
[task 2017-08-04T05:53:12.478768Z] 05:53:12     INFO - TEST-UNEXPECTED-FAIL | browser/extensions/onboarding/test/browser/browser_onboarding_accessibility.js | Focus should be on the first tour item - {} == {} - 
[task 2017-08-04T05:53:12.481486Z] 05:53:12     INFO - Stack trace:
[task 2017-08-04T05:53:12.488560Z] 05:53:12     INFO - resource://testing-common/content-task.js line 52 > eval:null:11
[task 2017-08-04T05:53:12.490251Z] 05:53:12     INFO - resource://testing-common/content-task.js:null:53
[task 2017-08-04T05:53:12.493159Z] 05:53:12     INFO - Not taking screenshot here: see the one that was previously logged
[task 2017-08-04T05:53:12.496281Z] 05:53:12     INFO - TEST-UNEXPECTED-FAIL | browser/extensions/onboarding/test/browser/browser_onboarding_accessibility.js | Overlay button focus state is saved correctly - "undefined" == "true" - 
[task 2017-08-04T05:53:12.499231Z] 05:53:12     INFO - Stack trace:
[task 2017-08-04T05:53:12.504917Z] 05:53:12     INFO - resource://testing-common/content-task.js line 52 > eval:null:13
[task 2017-08-04T05:53:12.508518Z] 05:53:12     INFO - resource://testing-common/content-task.js:null:53
[task 2017-08-04T05:53:12.510420Z] 05:53:12     INFO - Close the dialog and check modal dialog state
Flags: needinfo?(yzenevich)
Pushed by
improved accessibliity of onboarding notification bar. r=mossop
I believe this one will now stick so removing ni?
Flags: needinfo?(yzenevich)
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 57
Comment on attachment 8893463 [details] [diff] [review]
1387057 notification bar a11y v1

This is one of several bugs that make onboarding accessible to keyboard and screen reader users.
[Feature/Bug causing the regression]: None
[User impact if declined]: Users who use accessibility services or keyboard would not be able to use onboarding.
[Is this code covered by automated tests?]: Yes
[Has the fix been verified in Nightly?]: No
[Needs manual test from QE? If yes, steps to reproduce]:
* Keyboard focus should be visible for all focusable-with-keyboard controls within the notification footer.
* If using a screen reader (such as narrator/NVDA in windows or voice over in OSX that support live regions) when the notification is shown, the user should get a verbal notification that the footer is visible.
[List of other uplifts needed for the feature/fix]: not for this bug, but all onboarding accessibility bugs are listed in bug 1377300
[Is the change risky?]: No
[Why is the change risky/not risky?]: Only affects users that use keyboard
[String changes made/needed]: none
Attachment #8893463 - Flags: approval-mozilla-beta?
Comment on attachment 8893463 [details] [diff] [review]
1387057 notification bar a11y v1

Improve a11y for the onboarding tour in 56. Please uplift to beta.
Attachment #8893463 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Not sure if it was this or bug 1387057, but browser_onboarding_tours.js started to permafail after they were uplifted to Beta. Backed out.
Flags: needinfo?(yzenevich)
posted try with rebased commits in bug 1377300
Flags: needinfo?(yzenevich)
I can confirm this issue is fixed, I reproduced it with Fx 57.0a1 (build ID: 20170802100302) on Windows 10 x64.
I verified using Fx 57.0a1 (2017-09-01) and Fx 56.0b8, on Windows 10 x64, mac OS X 10.12.6 and Ubuntu 14.04 LTS.

Flags: qe-verify+
I have verified that this issue works as expected on Win 10 x64, Win 7 x86, Mac 10.13, & Ubuntu 16.04 x32 with Firefox 58.
You need to log in before you can comment on or make changes to this bug.