Closed Bug 754637 Opened 10 years ago Closed 10 years ago

double tab to zoom guesses wrong

Categories

(Firefox for Android Graveyard :: General, defect)

14 Branch
x86
macOS
defect
Not set
normal

Tracking

(firefox19 verified)

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: 10 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
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.