Closed Bug 2068610 Opened 24 days ago Closed 4 days ago

Search Mode Switcher menu items should not have explicit access keys (first letter shouldn't be underlined) so that multiple panel items can share the same implied access key

Categories

(Firefox :: Address Bar, defect, P1)

Desktop
All
defect

Tracking

()

VERIFIED FIXED
158 Branch
Tracking Status
firefox-esr140 --- unaffected
firefox-esr153 --- wontfix
firefox155 --- wontfix
firefox156 --- wontfix
firefox157 --- wontfix
firefox158 --- verified
firefox159 --- verified

People

(Reporter: oardelean, Assigned: dao)

References

(Blocks 2 open bugs, Regression)

Details

(Keywords: regression, Whiteboard: [sng][mcab-hnt-followon])

Attachments

(3 files)

Attached image image.png —

Notes

  • Please see the attached screenshot for more details.
  • Reproducible with both Proton and Nova.
  • Could not reproduce on macOS so far.

Found in

  • Nightly 157.0a1;

Affected versions

  • Nightly 157.0a1;

Tested platforms

  • Windows 11;
  • Ubuntu 22;
  • macOS 13;

Affected platforms

  • Windows 11;
  • Ubuntu 22;

Unaffected platforms

  • macOS 13;

Steps to reproduce

  1. Launch Firefox.
  2. Open the Search Mode Switcher.

Expected result

  • Search engines are displayed properly.

Actual result

  • Search engines listed have the first letter underscored.

Regression range

Set release status flags based on info from the regressing bug 2051148

:mbeier, since you are the author of the regressor, bug 2051148, could you take a look? Also, could you set the severity field?

For more information, please visit BugBot documentation.

This is a mnemonic for the key that can be pressed to select each item (also called its accesskey). It can be disabled by setting ui.key.menuAccessKey to 0 but it's intentionally on by default on Linux.

One thing I just realized is that imnemonics aren't visible in the newtab searchbar because panel-list uses XUL labels to render them (the accesskey still works). Dao do you think this inconsistency should be addressed?

Status: NEW → RESOLVED
Closed: 22 days ago
Flags: needinfo?(mbeier) → needinfo?(dao+bmo)
Resolution: --- → INVALID

I believe these are implied first-letter access keys, and therefore shouldn't get the underline.

Assignee: nobody → dao+bmo
Blocks: 2067642
Status: RESOLVED → REOPENED
Flags: needinfo?(dao+bmo)
Resolution: INVALID → ---
Whiteboard: [sng][mcab-hnt-followon]

Oh, these aren't auto accesskeys, but they should be. Same reasoning as in bug 2071753.

See Also: → 2071753
Depends on: 2071756
Summary: Search Engines in the Search Mode Switcher have the first letter underscored → Search Mode Switcher menu items should not have the first letter underlined
Summary: Search Mode Switcher menu items should not have the first letter underlined → Search Mode Switcher menu items should not have explicit access keys (first letter shouldn't be underlined) so that multiple panel items can share the same implied access key

The Smartbar's mention panel moves focus to its editor before this runs, which is why the check asks what kind of element has focus and not where it sits; a XUL panel opened by mouse leaves focus on the body and keeps the letters.

See Also: → 2073853
Blocks: 2073853
See Also: 2073853 →
Severity: -- → S3
Priority: -- → P1
Pushed by dgottwald@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/70b104f0bc81 https://hg.mozilla.org/integration/autoland/rev/7e4a59fede40 Let a panel-list select an item by its first letter while focus is outside the panel. r=tgiles,reusable-components-reviewers https://github.com/mozilla-firefox/firefox/commit/8a4cce3a21f2 https://hg.mozilla.org/integration/autoland/rev/bb67d5fdf7ce Drop the search mode switcher items' accesskeys. r=daleharvey,fluent-reviewers,bolsson
Status: REOPENED → RESOLVED
Closed: 22 days ago → 4 days ago
Resolution: --- → FIXED
Target Milestone: --- → 158 Branch
QA Whiteboard: [S4][qa-found-in-c157] → [S4][qa-found-in-c157][search][qa-triage-done-c158/b157][qa-ver-needed-c158/b157]
QA Contact: oardelean

Reproducible on a 2026-09-02 Firefox Nightly build on Windows 10.

Verified as fixed on Firefox Nightly 159.0a1 and Firefox 158.0b1 on Windows 10, Ubuntu 22, macOS 13.

Status: RESOLVED → VERIFIED
QA Whiteboard: [S4][qa-found-in-c157][search][qa-triage-done-c158/b157][qa-ver-needed-c158/b157] → [S4][qa-found-in-c157][search][qa-triage-done-c158/b157][qa-ver-done-c159/b158]

The patch landed in nightly and ESR is affected.

:dao, please make an uplift decision for ESR:

  • For ESR, nominate the patch for the appropriate ESR uplift approval if the fix should be included in ESR, or set the ESR status flag(s) to wontfix if an uplift isn't needed. Each affected ESR release needs a decision.

See Requesting an Uplift for documentation on how to request an uplift.

For more information, please visit BugBot documentation.

Flags: needinfo?(dao+bmo)
Flags: needinfo?(dao+bmo)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: