Closed Bug 1813877 Opened 1 year ago Closed 1 year ago

[a11y] Page name/URL isn't announced when swipe of the navbar is performed (switching tabs)

Categories

(Fenix :: Toolbar, defect, P2)

All
Android

Tracking

(Accessibility Severity:s2, firefox110 wontfix, firefox111 verified, firefox112 verified)

VERIFIED FIXED
Accessibility Severity s2
Tracking Status
firefox110 --- wontfix
firefox111 --- verified
firefox112 --- verified

People

(Reporter: gl, Assigned: vdreghici)

References

Details

(Keywords: access, Whiteboard: [fxdroid][qa-triaged])

Attachments

(3 files)

From github: https://github.com/mozilla-mobile/fenix/issues/15631.

Steps to reproduce

  1. Enable TalkBack.
  2. Open 2-3 pages.
  3. Using 2 fingers, swipe the nav bar left and right to switch tabs.

Expected behavior

Page name + "loading" is announced.

Actual behavior

"Loading" is announced, without the name (URL) of the page.

Device information

  • Android device: LG G7 FIT (Android 8)
  • Fenix version: Nightly 10/2

┆Issue is synchronized with this Jira Task

Change performed by the Move to Bugzilla add-on.

(replying to a thread from Github)

Lorand, you had mentioned this was intermittently reproducible in the past. Does it always happen for the same web pages? Approximately how common is this bug?

Severity: -- → S2
Flags: needinfo?(lorand.janos)
Keywords: access
Priority: -- → P2

After the first page is focused either the page itself, or the address bar, the page name is announced, "loading" is only announced if it's refreshed or newly opened page (accessing a page after opening it in new tab and swiping to it from the address bar). For these pages only the "loading" part is announced and nothing else.
After that first page from which the swiping starts, I didn't get to a page that had its name announced, only if tapped on address bar again, or the page itself. (On the video I tapped a page by accident. That was announced.)

Flags: needinfo?(lorand.janos)

Oof, yeah, this is definitely at least an S2, I was able to repro this quite easily & it's a common use case. Thanks for the additional detail. Adding to Kanban board to pick up soon.

Note to engineering for what to try to accomplish with this:

  • When swiping between tabs, the URL should be automatically dictated by TalkBack (it's already shown to have the green highlight, so I'm not sure why it wouldn't be dictated)
  • Nice-to-have (though, don't spend too much effort making this happen - I think we need to review TalkBack elegance across the board): when you load a new page (after typing & entering a URL), TalkBack seems to highlight either the "site information" (🔒) button or the "tabs" button. It would be nice if it instead started with the URL, or perhaps even better, the contents of the page itself.
Whiteboard: [fxdroid]
Assignee: nobody → Vlad.DreghiciPopa
Whiteboard: [fxdroid] → [fxdroid][access-s2]
Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Flags: qe-verify+

Verified as fixed on the latest Nightly 112.0a1 (2023-02-19T16:08:33.109137) build.
The pages names are announced.
Device used: Google Pixel 7 (Android 13).
Marking this ticket as verified.

Status: RESOLVED → VERIFIED
Flags: qe-verify+

Vlad, do you think your navbar fix is safe to uplift to Beta 111?

Flags: needinfo?(Vlad.DreghiciPopa)

Comment on attachment 9319661 [details] [review]
[mozilla-mobile/firefox-android] Bug 1813877 - Announce page title with talkback when swiping on navbar (backport #790) (#981)

Beta/Release Uplift Approval Request

  • User impact if declined: Accessibility service will not announce the description of tab swiped to using gestures on navbar.
  • Is this code covered by automated tests?: No
  • Has the fix been verified in Nightly?: Yes
  • Needs manual test from QE?: Yes
  • If yes, steps to reproduce: Enable TalkBack.
    Open 2-3 pages.
    Using 2 fingers, swipe the nav bar left and right to switch tabs.
  • List of other uplifts needed: None
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): Small modification improving a11y, verified by QA.
  • String changes made/needed: no
  • Is Android affected?: Yes
Flags: needinfo?(Vlad.DreghiciPopa)
Attachment #9319661 - Flags: approval-mozilla-beta?
Flags: qe-verify+
Comment on attachment 9319661 [details] [review]
[mozilla-mobile/firefox-android] Bug 1813877 - Announce page title with talkback when swiping on navbar (backport #790) (#981)

Approved for Fenix 111.0b6
Attachment #9319661 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Whiteboard: [fxdroid][access-s2] → [fxdroid][access-s2][qa-triaged]
See Also: → 1817611

A regression has been discovered regarding this patch here: https://bugzilla.mozilla.org/show_bug.cgi?id=1817611 . It has been fixed here: https://github.com/mozilla-mobile/firefox-android/pull/1004 . This will require an uplift after it has been merged and checked by QA.

Verified as fixed on the latest Beta 111.0b6 build.
The described issue is no longer reproducible on the mentioned build.
Devices used to verify:

  • Google Pixel 7 (Android 13).
  • Google Pixel 6 ( Android 13).
  • Samsung Galaxy Tab A7 (Android 12).
  • OnePlus 5 A5000 (Android 10).
  • OnePlus A3000 (Android 6).

Marking the ticket as verified for 111 as well.

Flags: qe-verify+
Regressions: 1817611
Accessibility Severity: --- → s2
Whiteboard: [fxdroid][access-s2][qa-triaged] → [fxdroid][qa-triaged]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: