1625462 - PDF Zooming zooms entire page with apz.allow_zooming=true

Status: RESOLVED FIXED

(Core :: Panning and Zooming, enhancement, P3)

Description

[ciprian.tomoiaga]

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:74.0) Gecko/20100101 Firefox/74.0

Steps to reproduce:

Go to about:config and set apz.allow_zooming to true.
Then opened a pdf (e.g. https://arxiv.org/pdf/2001.01469v1.pdf ) and try zooming in.

(MacOS, if it matters. Pinch to zoom on trackpad.)

Actual results:

The whole page zooms and the controls go outside.
Also, cannot zoom out to smaller doc.

Expected results:

Only the PDF content should have zoomed it

Comment 1

[Michel Le Bihan]

Hello,
Why did you enable apz.allow_zooming? Zooming should work without it. If I press ctrl and start scrolling my mouse wheel, the document zooms in without any issues.

Comment 2

[BugBot [:suhaib / :marco/ :calixte]]

Bugbug thinks this bug should belong to this component, but please revert this change in case of error.

Comment 3

[ciprian.tomoiaga]

(In reply to Michel Le Bihan from comment #1)

Hello,
Why did you enable apz.allow_zooming? Zooming should work without it. If I press ctrl and start scrolling my mouse wheel, the document zooms in without any issues.

Two reasons:

• I do not use a mouse for my laptop. I want to use pinch gesture to zoom. While it does work without apz.allow_zooming, it is unusable, at least on a Mac. The zoom jumps in very big increments (>10%) and doesn't stop once you no longer move the fingers. It is almost impossible to get a decent zoom level once you start.
• I want smooth zoom, not prefixed increments.

Thanks !

Comment 4

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

This will probably be helped by bug 1619187 but it may need additional changes on the pdf.js side to listen for the ctrl+wheel events and do content zooming.

Comment 5

[Timothy Nikkel (:tnikkel)]

Bug 1619187 "fixed" this bug and "created" bug 1643097.

I'll mark this bug fixed and we'll track the new issue in the new bug.