Closed Bug 1315608 Opened 8 years ago Closed 8 years ago

Null PDF zoom level between Page Width and 50% displayed while navigating with keyboard

Categories

(Firefox :: PDF Viewer, defect)

defect
Not set
normal

Tracking

()

VERIFIED FIXED
Firefox 52
Tracking Status
firefox-esr45 --- wontfix
firefox50 --- wontfix
firefox51 --- fix-optional
firefox52 --- verified

People

(Reporter: bmaris, Unassigned)

References

Details

(Whiteboard: [pdfjs-f-fixed-upstream] https://github.com/mozilla/pdf.js/pull/7792)

Attachments

(1 file)

Attached image Gif showing the issue
[Note]: - This is the same behavior as in bug 1194700 but with e10s disabled. [Affected versions]: - Firefox 45.5.0ESR - Firefox 50.0 RC - Latest Nightly 52.0a1 - Developer Edition 51.0a1 [Affected platforms]: - Mac OS X 10.12 - Windows 7 64bit - Windows 10 64bit - Ubuntu 14.04 32bit [Steps to reproduce]: 1. Start Firefox 2. Open any PDF (ex: http://www.cbu.edu.zm/downloads/pdf-sample.pdf) 3. Click on predefined zoom level button 4. Hit ESC button to close the dropdown but to keep the focus on it 5. Using left/right arrow keys go through the zoom levels 6. Hit Enter when the values are changed to activate the zoom level selected [Expected result]: - All the correct zoom levels are shown while navigating using keyboard. [Actual result]: - Between Page Width and 50% zoom levels there is a null value. [Regression range]: - This is NOT a recent regression. Not sure if this is an older regression though, I was able to reproduce it on old Nightly from 42.0a1 with e10s disabled, will investigate further ASAP. [Additional notes]: - Gif showing the issue is attached.
Thanks Bogdan. Since this looks like an older and minor regression I am not going to worry about it for ESR45 or for the 50/51 release. We can take a look at it for 52 though.
I tracked down a regression using mozregression, but it's something very old, since pdf.js was not an add-on anymore. Last good revision: 1158503601be (2012-05-21) First bad revision: b038090f07c2 (2012-05-22) Pushlog: http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=1158503601be&tochange=b038090f07c2 d50e4a17308b Benjamin Smedberg — Bug 740795 part C - Don't ship pdf.js as an extension, build changes, r=ted
Flags: needinfo?(benjamin)
No comment. The PDF team should triage this with everything else.
Flags: needinfo?(benjamin)
Depends on: 1316941
Keywords: regression
Status: NEW → RESOLVED
Closed: 8 years ago
Keywords: regression
Resolution: --- → FIXED
Whiteboard: [pdfjs-f-fixed-upstream] https://github.com/mozilla/pdf.js/pull/7792
Target Milestone: --- → Firefox 52
Flags: qe-verify+
I just verified this is no longer an issue on Firefox 52 beta 8 across platforms (Windows 10 64bit, macOS 10.12.3 and Ubuntu 16.04 32bit).
Status: RESOLVED → VERIFIED
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: