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)
Firefox
PDF Viewer
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)
560.08 KB,
image/gif
|
Details |
[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.
Comment 1•8 years ago
|
||
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.
Reporter | ||
Comment 2•8 years ago
|
||
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)
Comment 3•8 years ago
|
||
No comment. The PDF team should triage this with everything else.
Flags: needinfo?(benjamin)
Updated•8 years ago
|
Keywords: regression
Updated•8 years ago
|
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
Updated•8 years ago
|
Flags: qe-verify+
Reporter | ||
Comment 5•8 years ago
|
||
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).
You need to log in
before you can comment on or make changes to this bug.
Description
•