Closed Bug 1469674 Opened 6 years ago Closed 6 years ago

screen reader access: popup notifications keyboard access is broken

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
Tracking Status
firefox-esr52 --- unaffected
firefox-esr60 --- unaffected
firefox60 --- unaffected
firefox61 --- unaffected
firefox62 --- fixed
firefox63 + fixed

People

(Reporter: derek.riemer, Unassigned)

References

Details

(Keywords: access, regression, Whiteboard: fixed by bug 1466998)

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:62.0) Gecko/20100101 Firefox/62.0
Build ID: 20180617100056

Steps to reproduce:

1. visit https://davidwalsh.name/demo/notifications-api.php
2. request a notification from it.
3. try pressing f6 until you land on the notification.
4. now, turn on NVDA. press f6 to land on the location area.
5. press nvda+numpad 8 twice to go up a couple of level in the accessibility tree, and then move left until you are on the alert.
6. drill into the alert.
7. note that you can focus the notification allow and deny button.
8. press NVDA+shift+numpad - on the deny button. Focus lands on the deny button.
9. Try pressing tab. Nothing happens.


Actual results:

keyboard focus can't be placed anywhere of importance. pressing f6 does not focus the notification allow dialog, it in fact just stops working at the location bar. Once focus is forced in the dialog, pressing tab and arrow keys does nothing. This dialog is not accessible.


Expected results:

full keyboard access should be provided. I think this regressed.
Component: Untriaged → Disability Access
Confirmed.
Status: UNCONFIRMED → NEW
Component: Disability Access → Keyboard Navigation
Ever confirmed: true
Keywords: regression
OS: Unspecified → All
Hardware: Unspecified → All
Summary: screen reader access: notifications keyboard access is broken → screen reader access: popup notifications keyboard access is broken
21:15.82 INFO: Last good revision: 48a5d87cf9bdb59e39653ab331df04ea2a04267d
21:15.82 INFO: First bad revision: 20d536fd0f2a02bd4527044d367cf98bebbb358d
21:15.83 INFO: Pushlog:
https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=48a5d87cf9bdb59e39653ab331df04ea2a04267d&tocha
nge=20d536fd0f2a02bd4527044d367cf98bebbb358d
Another regression from enabling shadow DOM in Bug 1460069. This is worth re-testing once bug 1466998 lands.
Blocks: 1460069
Has Regression Range: --- → yes
Has STR: --- → yes
Component: Keyboard Navigation → DOM
Product: Firefox → Core
derek, would you mind re-testing in Nightly now that the work from bug 1466998 landed?
Also noting the issue should only exist in Nightly, since the shadow DOM feature is disabled except in Nightly right now.
Flags: needinfo?(derek.riemer)
yep, fixed!
Flags: needinfo?(derek.riemer)
Fixed by bug 466998, as suspected. Closing WORKSFORME.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WORKSFORME
Resolution: WORKSFORME → FIXED
Whiteboard: fixed by bug 1466998
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.