960808 - upworthy.com image zooming widget does not work

Status: RESOLVED WORKSFORME

(Core :: Panning and Zooming, defect)

Description

[Botond Ballo [:botond]]

+++ This bug was initially created as a clone of Bug #960807 +++

STR:
  1. Load http://www.upworthy.com/a-few-important-things-to-remember-the-next-time-a-cop-pulls-you-over-4
  2. Click on the magnifying glass icon with the orange background in the
     top right corner of the image.
  3. Pinch or double-tap to zoom.
  4. Pan around.

Expected results:
  The overlay image is as large as the screen.

Actual results:
  The overlay image only takes up the left half, or sometimes just the top-left quarter, of the page.

Note that even if you skip step (3) you have a similar problem.

This does not work on Fennec either, but it does work in the Chrome browser on Android.

Comment 1

[Kartikaya Gupta (email:[e-mail])]

This seems to work fine for me on the latest code, although it seems to end up in a bit of overscroll at the top/bottom sometimes. Nothing really bad though.

Also just to note it looks like the page is implementing the pinch and double-tap behaviour here; I don't think this is APZC pinch code being used.

Botond, are you still able to reproduce?

Comment 2

[Botond Ballo [:botond]]

Nope, the issue I saw when I reported the bug seems to be gone.

Looking at the APZC tree I see a single APZC with zero scroll range, suggesting that it is indeed the page implementing the panning and zooming logic of that widget, and thus any bugs we still see (like the slight overscrolling you mention) are the page's bugs.

Let's close this then.