Need to be able to pan fully while zoomed in

VERIFIED FIXED in fennec1.0a1

Status

defect
VERIFIED FIXED
11 years ago
10 years ago

People

(Reporter: pavlov, Assigned: Gavin)

Tracking

Trunk
fennec1.0a1
x86
Windows Vista
Bug Flags:
blocking-xul-fennec1.0 +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

_constrainPanCoords doesn't currently take in to account the zoom factor which means we overly constrain how far we can pan, which makes panning while zoomed quite painful.
Flags: blocking-fennec1.0+
Target Milestone: --- → Fennec A1
Posted patch patchSplinter Review
This makes sure we only pass page coords to constrainPanCoords, and includes some other simplifications. Kinetic is really broken with this, though, since it compares dragX/Y and pageX/Y which are screen coordinates and content (page) coordinates, respectively. I tried figuring out what the kinetic code is trying to do, but I didn't end up getting anywhere - I've just disabled it for now.
Attachment #338742 - Flags: review+
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
verified with beta3
Status: RESOLVED → VERIFIED
Component: General → Panning/Zooming
You need to log in before you can comment on or make changes to this bug.