Closed Bug 623313 Opened 9 years ago Closed 9 years ago

Double-tap text reflow shows old layout while zooming

Categories

(Firefox for Android Graveyard :: Panning/Zooming, defect)

defect
Not set

Tracking

(fennec2.0+)

VERIFIED FIXED
Tracking Status
fennec 2.0+ ---

People

(Reporter: mbrubeck, Assigned: mbrubeck)

References

()

Details

(Keywords: polish, Whiteboard: [has-patch])

Attachments

(1 file)

+++ This bug was initially created as a clone of Bug #621683 +++
Steps to reproduce:
0. Enable "Reformat text on zoom" in the Fennec prefs.
1. Open http://www.google.com/finance in portrait orientation.
2. Double-tap within the "Top stories / Market" section in the middle column.

Expected results: Fennec reformats the text and then zooms to the tapped section.

Actual results: Fennec zooms to a different location, *then* reformats the text so that the tapped section is on-screen.

We should make sure the page redraws with the new formatting before the zoom animation starts.
Attached patch patchSplinter Review
Throwing in a setTimeout seems to do the trick.
Attachment #503637 - Flags: review?(mark.finkle)
Whiteboard: [has-patch]
Attachment #503637 - Flags: review?(mark.finkle) → review+
http://hg.mozilla.org/mobile-browser/rev/e05ccfe2d213
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
verified FIXED on build:
Mozilla/5.0 (Android; Linux armv71; rv:2.0b9pre) Gecko/20100114 Namoroka/4.0b10pre Fennec/4.0b4pre
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.