The reader view panels should close when tapping ESC key
Categories
(Toolkit :: Reader Mode, defect, P1)
Tracking
()
People
(Reporter: danibodea, Assigned: rhn0312)
References
(Blocks 1 open bug)
Details
(Keywords: access, Whiteboard: [fidefe-quality-foundation?])
Attachments
(1 file, 1 obsolete file)
48 bytes,
text/x-phabricator-request
|
dmeehan
:
approval-mozilla-beta+
|
Details | Review |
Note
- For easy keyboard navigation, the reader view panels should close when tapping the ESC key.
Found in
- Beta v129.0b2
Affected versions
- Nightly v130.0a1
- Beta v129.0b2
Tested platforms
- Affected platforms: all
- Unaffected platforms: none
Steps to reproduce
- Load an article that supports reader view.
- Enable Reader view.
- Using keyboard navigation open any of the panels (Text and layout, Theme, Read aloud)
- Press ESC key.
Expected result
- The panels should close.
Actual result
- The panels do not close.
Regression range
- Not a regression.
Additional notes
- The panel do close when tapping the UP/DOWN Arrows (the action of scrolling the page).
Reporter | ||
Updated•2 months ago
|
Comment 1•2 months ago
|
||
Hey Irene, you've worked in this area recently. How difficult do you think it'd be to wire up a handler for Esc
to close the panels?
Reporter | ||
Comment 2•2 months ago
|
||
I suggest pairing this implementation with retaining the focus inside the opened panel.
For better understanding, I am providing you with these steps to reproduce:
- Load an article that supports reader view.
- Enable Reader view.
- Using keyboard navigation open any of the panels (Text and layout, Theme, Read aloud)
- Press the TAB key to navigate beyond the last element from the panel.
Expected: The focus should move to the first element in the panel.
Actual: The focus is moved out of the panel.
I believe this would ensure keyboard navigation consistency throughout the browser's panels.
I am not certain whether this behavior is desired over the current one, so I will leave this decision to you.
Please let me know if you prefer a new report for this suggestion. Thank you.
Comment 3•2 months ago
|
||
I'm marking this as access-s2 because the ability to dismiss the popup is important because it covers content.
I agree with Daniel's focus order suggestion too.
Comment 4•2 months ago
|
||
The severity field for this bug is set to S3. However, the accessibility severity is higher, .
:Gijs, could you consider increasing the severity?
For more information, please visit BugBot documentation.
Apologies for the delay. I've created a patch that implements the suggested escape and focus behaviour.
Updated•2 months ago
|
Updated•2 months ago
|
Updated•2 months ago
|
Updated•2 months ago
|
Comment 7•2 months ago
|
||
Irene, next week is the final week of beta for Fx129
https://whattrainisitnow.com/release/?version=129
Are you planning on completing the patch and landing it by next week?
I'm tracking this for Fx129 since it's an S2 bug and I'm wondering if we will have a beta uplift request in time.
I am on track to landing this patch before the final beta uplift date on the 26th.
Pushed by ini@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/03866d106200 The reader view panels should close when tapping ESC key. r=reader-mode-reviewers,cmkm
Comment 10•1 month ago
|
||
bugherder |
Reporter | ||
Comment 11•1 month ago
|
||
I can verify this change in Nightly v130.0a1 from 2024-07-25 in Windows 10, MacOS 11 and Ubuntu 22.
A small follow-up issue is observed and logged (bug 1909885); While the focus is retained inside the panel if the user taps TAB key, it is not retained inside the panel it the user taps SHIFT+TAB hotkey to navigate backwards.
Comment 12•1 month ago
|
||
Irene, could you add a beta uplift request on this? The final Fx129 builds tomorrow so there's not much time left.
Assignee | ||
Comment 13•1 month ago
|
||
Original Revision: https://phabricator.services.mozilla.com/D216760
Updated•1 month ago
|
Comment 15•1 month ago
|
||
beta Uplift Approval Request
- User impact if declined: keyboard accessibility would be impacted
- Code covered by automated testing: yes
- Fix verified in Nightly: yes
- Needs manual QE test: no
- Steps to reproduce for manual QE testing: n/a
- Risk associated with taking this patch: low
- Explanation of risk level: affects keyboard navigation within reader view menu panels only
- String changes made/needed: no
- Is Android affected?: no
Updated•1 month ago
|
Comment 16•1 month ago
|
||
Comment on attachment 9414851 [details]
Bug 1907280 - The reader view panels should close when tapping ESC key. r=#reader-mode-reviewers
Irene, the beta uplift request has conflicts.
I added a comment in the beta uplift request if you could take a look?
Assignee | ||
Comment 17•1 month ago
|
||
I have updated D217714 to resolve the conflicts.
Comment 18•1 month ago
|
||
Irene, not sure if you used mozphab to update the patch. I can't land it. Pleas take a look at the comment in the Phabricator revision.
Updated•1 month ago
|
Assignee | ||
Comment 19•1 month ago
|
||
Comment on attachment 9413138 [details]
Bug 1907280 - The reader view panels should close when tapping ESC key. r=#reader-mode-reviewers
Beta/Release Uplift Approval Request
- User impact if declined: keyboard accessibility would be impacted
- Is this code covered by automated tests?: Yes
- Has the fix been verified in Nightly?: Yes
- Needs manual test from QE?: No
- If yes, steps to reproduce: n/a
- List of other uplifts needed: Bug 1905097
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): affects keyboard navigation within reader view menu panels only
- String changes made/needed: n/a
- Is Android affected?: No
Comment 20•1 month ago
|
||
Comment on attachment 9413138 [details]
Bug 1907280 - The reader view panels should close when tapping ESC key. r=#reader-mode-reviewers
Approved for 129.0b9
Updated•1 month ago
|
Updated•1 month ago
|
Updated•1 month ago
|
Updated•1 month ago
|
Reporter | ||
Comment 22•1 month ago
|
||
I can confirm this implementation in Beta v129.0b9 as well. It was verified in Windows 10, Ubuntu 22 and MacOS 11. Like in Nightly v130.0a1, it is easier to navigate by keyboard and more consistent through the browser since ESC key closes the reader view panels.
Also, the focus is retained inside the panels when tapping the TAB key beyond the last panel element, but the focus is not retained inside the panel when tapping the SHIFT+TAB hotkey beyond the first element of each of the reader view panels. This follow-up issue has been logged as bug 1909885. Thank you!
Description
•