Closed Bug 628827 Opened 10 years ago Closed 10 years ago
Blinky double-tap zoom and pinch to zoom
No description provided.
Comment on attachment 506933 [details] [diff] [review] Blinky double-tap zoom and pinch to zoom I remember now. The animated zoom and setVisibleRect code *used* to use the underlying frameloader stuff for a reason. We don't want the displayport flashing like crazy while we set the appropriate scale and scroll to the right area. We can have a real API for this in 628799.
Attachment #506933 - Flags: review?(mbrubeck)
Assignee: nobody → ben
Status: NEW → ASSIGNED
Comment on attachment 506933 [details] [diff] [review] Blinky double-tap zoom and pinch to zoom >+++ b/chrome/content/browser.js >+ view.setScale(scale); >+ view._contentView.scrollTo(scrollX, scrollY); >+ >+ browser.scale = scale; Ouch, this is ugly. Let's get a real API in bug 628799 ASAP.
Attachment #506933 - Flags: review?(mbrubeck) → review+
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Can this be marked as verified fixed?
Verified fixed on trunk.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.