Closed Bug 1799212 Opened 2 years ago Closed 2 years ago

Escape key does not exit screenshots tool when focused on Save visible/Save full page buttons (component)

Categories

(Firefox :: Screenshots, defect, P2)

defect

Tracking

()

VERIFIED FIXED
109 Branch
Tracking Status
firefox109 --- verified
firefox110 --- verified

People

(Reporter: tgiles, Assigned: niklas)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

STR

  • Set screenshots.browser.component.enabled to true
  • Navigate to https://bugzilla.mozilla.org/home (or any webpage)
  • Use keyboard shortcut (Ctrl + Shift + S on Windows, not sure on Mac or Linux) to take a screenshot
  • Use F6 to cycle to the Save visible/Save full page buttons
  • Hit Escape key to cancel screenshot

Expected:

  • Screenshot is cancelled when ESC key is pressed

Actual:

  • Nothing happens

This is based on the expectation from the previous extension implementation

The extension's UI does get dismissed using Escape when the focus is on either the Save visible or Save full page button, so this would represent a regression if we flip the pref.

Severity: -- → S3
Priority: -- → P2
Assignee: nobody → nbaumgardner
Status: NEW → ASSIGNED
Pushed by nbaumgardner@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/6b4aaee5d13c Escape screenshots when panel is focused. r=sfoster
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 109 Branch
Flags: qe-verify+

Reproducible on a 2022-11-04 Nightly build on Windows 10.
Verified as fixed on Firefox 109.0b4(build ID: 20221218190303) and Nightly 110.0a1(build ID: 20221218193142) on Windows 10, macOS 12, Ubuntu 22.

Status: RESOLVED → VERIFIED
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: