Closed Bug 524526 Opened 11 years ago Closed 11 years ago
Zooming sometimes zooms to incorrect positions
After doubletapping around for a while, zooming occasionally gets into a funk where double tapping on anything goes to the wrong place. Restarting Fennec reverts to normal behavior. Discovered on an article at cnn.com.
Only begins happening after a link opens something in a new window. I assume this is because the content scrollarea has changed to something besides 0,0.
When loading, beginBatchOperation has been called, so scrollTo will not scroll to the correct place unless the container size has been resized. In order to have zoom look correct by the time the zoom operation is over, this forces a viewport refresh.
Assignee: nobody → webapps
Attachment #408676 - Flags: review?(mark.finkle)
Zooming still wasn't working because zoomToFit was being called every couple of seconds. I fixed the zoomChanged flag and zooming during load is looking much better. I think I've seen these changes in one of your patches that never landed Mark, whatever happened to it?
Comment on attachment 409147 [details] [diff] [review] Fix zoomChanged flag I'm starting to worry about the overall affect on performance our "correctness" patches are causing. I'd rather be "correct" first and "fast" second. We need to start profiling again.
Attachment #409147 - Flags: review?(mark.finkle) → review+
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → B5
verified FIXED on build (opened a couple of tabs, waited for each to load and then performed a couple of zooms on the original tab): Mozilla/5.0 (X11; U; Linux armv7l; Nokia N900; en-US; rv:1.9.2b2pre) Gecko/20091102 Namoroka/3.6b2pre Fennec/1.0b5
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.