Closed
Bug 583828
Opened 13 years ago
Closed 13 years ago
Wrong zoom level calculation in _getZoomRectForRect
Categories
(Firefox for Android Graveyard :: Panning/Zooming, defect)
Firefox for Android Graveyard
Panning/Zooming
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: mbrubeck, Assigned: mbrubeck)
Details
Attachments
(1 file, 2 obsolete files)
2.98 KB,
patch
|
stechz
:
review+
|
Details | Diff | Splinter Review |
The old version of this code (_getZoomLevelForElement) called browserToViewportRect on the element's bounding rect, so it needed to multiply vis.width by bv.getZoomLevel() too. The new code no longer calls browserToViewportRect, so it should no longer apply the getZoomLevel factor.
Attachment #462141 -
Flags: review?(webapps)
Assignee | ||
Comment 1•13 years ago
|
||
Also clean up some nearby whitespace.
Assignee: nobody → mbrubeck
Attachment #462141 -
Attachment is obsolete: true
Status: NEW → ASSIGNED
Attachment #462142 -
Flags: review?(webapps)
Attachment #462141 -
Flags: review?(webapps)
Comment 2•13 years ago
|
||
There are several places where getZoomLevelForRect are called: http://mxr.mozilla.org/mobile-browser/search?string=getZoomLevelForRect It looks like FormHelper still seems to do things in viewport coordinates?
Assignee | ||
Comment 3•13 years ago
|
||
(In reply to comment #2) > There are several places where getZoomLevelForRect are called: > http://mxr.mozilla.org/mobile-browser/search?string=getZoomLevelForRect > > It looks like FormHelper still seems to do things in viewport coordinates? Good catch. Fixed.
Attachment #462142 -
Attachment is obsolete: true
Attachment #462167 -
Flags: review?(webapps)
Attachment #462142 -
Flags: review?(webapps)
Updated•13 years ago
|
Attachment #462167 -
Flags: review?(webapps) → review+
Assignee | ||
Comment 4•13 years ago
|
||
http://hg.mozilla.org/mobile-browser/rev/7b7e45d2918d
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•