Closed
Bug 1659670
Opened 5 years ago
Closed 5 years ago
dmanip pinch gestures don't smooth zoom with keyboard modifiers since bug 1658647
Categories
(Core :: Panning and Zooming, defect, P3)
Core
Panning and Zooming
Tracking
()
RESOLVED
FIXED
81 Branch
| Tracking | Status | |
|---|---|---|
| firefox-esr68 | --- | unaffected |
| firefox-esr78 | --- | unaffected |
| firefox79 | --- | unaffected |
| firefox80 | --- | unaffected |
| firefox81 | --- | fixed |
People
(Reporter: tnikkel, Assigned: tnikkel)
References
(Regression)
Details
(Keywords: regression)
Attachments
(1 file)
We discard pinch gestures if they have any modifiers here
| Assignee | ||
Updated•5 years ago
|
Summary: dmanip pinch gestures don't smooth zoom since bug 1658647 → dmanip pinch gestures don't smooth zoom with keyboard modifiers since bug 1658647
| Assignee | ||
Comment 1•5 years ago
|
||
If you happen to have a full size keyboard with numlock defaulting to on this is very noticeable.
| Assignee | ||
Comment 2•5 years ago
|
||
Comment 3•5 years ago
|
||
Set release status flags based on info from the regressing bug 1658647
status-firefox79:
--- → unaffected
status-firefox80:
--- → unaffected
status-firefox81:
--- → affected
status-firefox-esr68:
--- → unaffected
status-firefox-esr78:
--- → unaffected
Updated•5 years ago
|
Severity: -- → S3
Priority: -- → P3
Updated•5 years ago
|
Attachment #9170621 -
Attachment description: Bug 1659670. Don't add keyboard modifiers to direct manipulation pinch gestures. r?kats → Bug 1659670. Only consider non-lock keyboard modifiers when the APZC decides whether to ignore modifiers on pinch gestures. r?kats
Pushed by tnikkel@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/d9ee9a2a6fec
Only consider non-lock keyboard modifiers when the APZC decides whether to ignore modifiers on pinch gestures. r=kats
Comment 5•5 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → 81 Branch
Updated•5 years ago
|
Keywords: regression
Updated•5 years ago
|
Has Regression Range: --- → yes
You need to log in
before you can comment on or make changes to this bug.
Description
•