Closed Bug 754637 Opened 9 years ago Closed 9 years ago

double tab to zoom guesses wrong

Categories

(Firefox for Android :: General, defect)

14 Branch
x86
macOS
defect
Not set
normal

Tracking

()

VERIFIED FIXED
Firefox 15
Tracking Status
firefox19 --- verified

People

(Reporter: canuckistani, Assigned: mbrubeck)

Details

(Whiteboard: [Engagement])

Attachments

(2 files)

Attached image screenshot
In some cases, double-tap to zoom guesses wrong, as seen on http://www.mvilla.it/epistle.html

to repro:

* load http://www.mvilla.it/epistle.html
* double tab to zoom in, left edge of paragraph text is no longer visible, the viewport seems to zoom in to the left edge of the unordered list.
Attached patch patchSplinter Review
This patch causes Fennec not to zoom in on <li> elements.  It will zoom to the containing element instead.  This fixes the page from comment 0.

The patch also excludes <blockquote> elements, for similar reasons.
Assignee: nobody → mbrubeck
Status: NEW → ASSIGNED
Attachment #623501 - Flags: review?(mark.finkle)
Comment on attachment 623501 [details] [diff] [review]
patch

ok, this makes sense.
Attachment #623501 - Flags: review?(mark.finkle) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/658391fe57eb
https://hg.mozilla.org/mozilla-central/rev/658391fe57eb
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 15
Double tap to zoom works correctly on the webpage from comment #0. 

Closing bug as verified fixed on:

Firefox 19.0a1 (2012-10-17)
Device: Galaxy Note
OS: Android 4.0.4
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.