Closed Bug 1544447 Opened 5 years ago Closed 5 years ago

Some items in panels can't be opened with the numpad's Enter key

Categories

(Firefox :: Site Identity, defect, P1)

defect

Tracking

()

RESOLVED FIXED
Firefox 68
Tracking Status
firefox68 --- fixed

People

(Reporter: itiel_yn8, Assigned: Gijs)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

STR:

  1. Lastet Nightly, Windows 10
  2. Open bugzilla.mozilla.org
  3. Click the Site Identity panel
  4. Navigate to Trackers, Cookies or Report a problem
  5. Hit the Enter key in the numpad

AR:
Nothing happens

ER:
Pressing the numpad's Enter key should open the respective panels.

I'm not sure atm if this is a regression from bug 1477673 or if this was an issue even before.

Has STR: --- → yes
Summary: Some items can't be opened with the numpad Enter button in the Site Identity panel → Some items can't be opened with the numpad's Enter key in the Site Identity panel

:itiel_yn8, if you think that's a regression, then could you try to find a regression range in using for example mozregression?

I very much doubt this is a regression, though I haven't tested it. The PanelMultiView code explicitly handles Enter, but not NumpadEnter (and this isn't something I changed):
https://searchfox.org/mozilla-central/source/browser/components/customizableui/PanelMultiView.jsm#1647

This should be a trivial fix, but needs tests.

This didn't work before anyway because of all of the 3 buttons didn't open using the normal Enter or the space key (bug 1539976, among the rest).

Assignee: nobody → gijskruitbosch+bugs
Status: NEW → ASSIGNED
Summary: Some items can't be opened with the numpad's Enter key in the Site Identity panel → Some items in panels can't be opened with the numpad's Enter key
Priority: -- → P1
Pushed by gijskruitbosch@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/515cfc853f2a
add support for numpad enter key and add tests to check activation works, r=Jamie
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 68
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: