Closed
Bug 587814
Opened 14 years ago
Closed 14 years ago
Smoother panning using mozAnimationFrame
Categories
(Firefox for Android Graveyard :: Panning/Zooming, defect)
Firefox for Android Graveyard
Panning/Zooming
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: stechz, Assigned: stechz)
Details
Attachments
(1 file)
5.83 KB,
patch
|
mfinkle
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Updated•14 years ago
|
Attachment #466441 -
Attachment is patch: true
Attachment #466441 -
Attachment mime type: application/octet-stream → text/plain
Attachment #466441 -
Flags: review?(mark.finkle)
Comment 1•14 years ago
|
||
Comment on attachment 466441 [details] [diff] [review] Patch > AnimatedZoom.prototype.finish = function() { > try { >+ Elements.viewBuffer.style.display = "none"; >+ What is this doing? > // resume live rendering > this.bv.resumeRendering(true); > > // if we actually zoomed somewhere, clean up the UI to normal > if (this.zoomRect) > Browser.setVisibleRect(this.zoomRect); >+ > } No blank line r+ with question answered
Attachment #466441 -
Flags: review?(mark.finkle) → review+
Updated•14 years ago
|
OS: Mac OS X → All
Hardware: x86 → All
Assignee | ||
Comment 2•14 years ago
|
||
Sorry, both of those were from another experiment. They can be removed.
Assignee | ||
Comment 3•14 years ago
|
||
Pushed http://hg.mozilla.org/mobile-browser/rev/3e452a03bf0f
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Comment 4•14 years ago
|
||
Looks like part of this was lost in a merge. Pushed: http://hg.mozilla.org/mobile-browser/rev/9e68083047aa
Updated•14 years ago
|
Flags: in-testsuite?
You need to log in
before you can comment on or make changes to this bug.
Description
•