Note: There are a few cases of duplicates in user autocompletion which are being worked on.

When two-finger Scrolling with the trackpad (with momentum), pressing Ctrl while scrolling finishes causes page zoom if pixel scrolling is disabled

RESOLVED FIXED in mozilla8

Status

()

Core
DOM: Events
P1
normal
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: bz, Assigned: bz)

Tracking

(Blocks: 1 bug)

Trunk
mozilla8
All
Mac OS X
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(blocking2.0 final+)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

6 years ago
+++ This bug was initially created as a clone of Bug #574663 +++

This is just like bug 574663, but first you need to flip "mousewheel.enable_pixel_scrolling" to false in about:config.  It looks like the patches in bug 574663 only modified the pixel-scrolling codepaths....

I finally figured out today that this is what causes sites to randomly zoom for me.
(Assignee)

Comment 1

6 years ago
Created attachment 549815 [details] [diff] [review]
Don't do zoom or history actions for momentum scrolls, no matter what's going on with pixel scrolling.
Attachment #549815 - Flags: review?(masayuki)
Attachment #549815 - Flags: review?(Olli.Pettay)
(Assignee)

Updated

6 years ago
Assignee: nobody → bzbarsky
Component: Widget: Cocoa → DOM: Events
Priority: -- → P1
QA Contact: cocoa → events
Whiteboard: [need review]
You could extend content/events/test/test_bug574663.html to test this, for example by adding another flag to sendTouchpadScrollMotion that makes it send one line scroll with hasPixels = false instead of what it currently does.
(Assignee)

Comment 3

6 years ago
Hmm.   Let me give that a shot!

Updated

6 years ago
Attachment #549815 - Flags: review?(Olli.Pettay) → review+
(Assignee)

Comment 4

6 years ago
Created attachment 549820 [details] [diff] [review]
With the test change

Verified that the changed test fails without the code change and passes with it.
Attachment #549820 - Flags: review?(masayuki)
(Assignee)

Updated

6 years ago
Attachment #549815 - Attachment is obsolete: true
Attachment #549815 - Flags: review?(masayuki)
Comment on attachment 549820 [details] [diff] [review]
With the test change

r=me.

# Inside the if's parentheses are very messy, though... I'll sort out them ASAP.
Attachment #549820 - Flags: review?(masayuki) → review+
(Assignee)

Updated

6 years ago
Whiteboard: [need review] → [need landing]
(Assignee)

Comment 6

6 years ago
http://hg.mozilla.org/integration/mozilla-inbound/rev/908e0c628083
Flags: in-testsuite+
Whiteboard: [need landing]
Target Milestone: --- → mozilla8
http://hg.mozilla.org/mozilla-central/rev/908e0c628083
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.