Closed Bug 1697401 Opened 4 years ago Closed 2 years ago

AppMenu FxA "Show more tabs" option does not work using keyboard only

Categories

(Firefox :: Menus, defect, P2)

defect

Tracking

()

RESOLVED WORKSFORME
Accessibility Severity s2
Tracking Status
firefox86 --- disabled
firefox87 --- disabled
firefox88 --- disabled
firefox89 --- affected

People

(Reporter: ailea, Assigned: cmkm)

References

(Blocks 2 open bugs)

Details

(Keywords: access, Whiteboard: [proton-hamburger-menu][priority:2c][fidefe-quality-foundation])

Tested with:

Nightly 88.0a1 (2021-03-10)

Tested on:

Windows 10
MacOS 11

Preconditions:

In about:config, set browser.proton.enabled = true

Steps:

  1. Launch firefox and sign in into FxA using 2 devices.
  2. Open more than 25 tabs on one device and sync tabs with another devices.
  3. Using the keyboard only navigate to the "Show More Tabs" option and press enter/space in order to expand the list of synced tabs.

Actual result:

The hamburger menu is closed/dismissed.

Expected result:

The list of all synced tabs should be expanded.

Note: The issue is reproducible both with and without Proton enabled.

Severity: -- → S3
Priority: -- → P3

This is pretty severe for those users who do encounter it (no way to access the 26th or later sync'd tabs) but I suspect few encounter the bug. Marking as an access-s2 because this is completely broken for keyboard users. This might be on the border of s3 but I could not find a workaround so leaving at access-s2 for now. The same bug occurs on the list in the account menu.

Whiteboard: [proton-hamburger-menu] → [proton-hamburger-menu][access-p2]
Whiteboard: [proton-hamburger-menu][access-p2] → [proton-hamburger-menu][access-s2]
Priority: P3 → P2
Whiteboard: [proton-hamburger-menu][access-s2] → [proton-hamburger-menu][access-s2][priority:2c]

Any update on this, One year already yet no one assigned, *sad noises

Does this need any C or Rust knowledge to implement fix? I wanna try, but I don't know where to start

The severity field for this bug is set to S3. However, the accessibility severity is higher, [access-s2].
:jaws, could you consider increasing the severity?

For more information, please visit auto_nag documentation.

Flags: needinfo?(jaws)

(In reply to Benyamin Limanto from comment #2)

Any update on this, One year already yet no one assigned, *sad noises

Does this need any C or Rust knowledge to implement fix? I wanna try, but I don't know where to start

It should be just JavaScript and DOM knowledge needed to fix this. The code for the event listener is here, https://searchfox.org/mozilla-central/rev/1c54648c082efdeb08cf6a5e3a8187e83f7549b9/browser/base/content/browser-sync.js#322-326

Would you like to work on it?

Flags: needinfo?(jaws) → needinfo?(BLBENYAMIN9)

(In reply to (Away 4/14 to 4/21) Jared Wein [:jaws] (please needinfo? me) from comment #4)

(In reply to Benyamin Limanto from comment #2)

Any update on this, One year already yet no one assigned, *sad noises

Does this need any C or Rust knowledge to implement fix? I wanna try, but I don't know where to start

It should be just JavaScript and DOM knowledge needed to fix this. The code for the event listener is here, https://searchfox.org/mozilla-central/rev/1c54648c082efdeb08cf6a5e3a8187e83f7549b9/browser/base/content/browser-sync.js#322-326

Would you like to work on it?

I will try to look into it first, and try to built the Firefox from Source

Flags: needinfo?(BLBENYAMIN9)
Whiteboard: [proton-hamburger-menu][access-s2][priority:2c] → [proton-hamburger-menu][access-s2][priority:2c][fidefe-quality-foundation]

I can't seem to reproduce this on macOS. The menu item functions as expected, and the menu doesn't close.

Assignee: nobody → cmeador

This bug does not reproduce for me on Windows either, I'm seeing the expected behavior. We may have let this one sit for so long that another change just happened to fix it. :(

Thanks so much for checking this on Windows, Molly! I can also confirm it's not reproducible in Nightly on Ubuntu, so I'm going to close this out.

Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Resolution: FIXED → WORKSFORME
Accessibility Severity: --- → s2
Whiteboard: [proton-hamburger-menu][access-s2][priority:2c][fidefe-quality-foundation] → [proton-hamburger-menu][priority:2c][fidefe-quality-foundation]
You need to log in before you can comment on or make changes to this bug.