1616157 - PDF.js UI elements are affected by global default zoom

Status: VERIFIED FIXED

(Firefox :: PDF Viewer, defect, P2)

Description

[Ryan Hendrickson [rhendric on GitLab, GitHub]]

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:74.0) Gecko/20100101 Firefox/74.0

Steps to reproduce:

1. Set the default zoom preference to 150%
2. Open a PDF in Firefox

Actual results:

The PDF.js UI (toolbar, sidebar) is scaled up to 150%. There's no way to scale it down to 100%, because all ways to zoom in/out are handled specially by PDF.js.

Expected results:

The PDF.js UI should stay scaled to 100%, because the size of other Firefox UI elements is not affected by the default zoom. Other special viewers like the image viewer and the media player also default to 100% regardless of the default zoom.

Comment 1

[Gingerbread Man]

Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:75.0) Gecko/20100101 Firefox/75.0
20200217214434

Comment 2

[Jonas Jenwald [:Snuffleupagus]]

Attachment: Bug 1616157 - Ensure that the UI elements of the PDF viewer won't be affected by a non-standard global default zoom level. r=bdahl

This patch ensures that the PDF viewer UI elements won't be zoomed in/out, on either initial loading or when resetting the zoom level, when a non-standard global default zoom level is set. Also hides the zoom indicator, in the Address Bar, in this case to avoid inconsistent state when the PDF viewer is used.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e62a98a8ac04
Ensure that the UI elements of the PDF viewer won't be affected by a non-standard global default zoom level. r=bdahl

Comment 4

[Alexandru Michis [:malexandru]]

https://hg.mozilla.org/mozilla-central/rev/e62a98a8ac04

Comment 5

[Camelia Badau [:cbadau], Desktop Test Engineering]

Reproduced the initial issue on Windows 10 x64 using Firefox 74.0b4.
Verified fixed on Windows 10 x64, Ubuntu 18.04 x64 and macOS 10.14.6 using Firefox 75.0b6.