"View Background Image" in PDF presentation mode results in a unusable browser window
Categories
(Firefox :: PDF Viewer, defect, P2)
Tracking
()
Tracking | Status | |
---|---|---|
firefox-esr68 | --- | unaffected |
firefox70 | --- | unaffected |
firefox71 | --- | wontfix |
firefox72 | --- | wontfix |
firefox73 | --- | wontfix |
firefox74 | --- | wontfix |
firefox75 | --- | fixed |
People
(Reporter: cbadau, Assigned: Snuffleupagus)
References
Details
(Keywords: regression)
Attachments
(3 files)
Affected versions
- Firefox 71 Beta 6
- latest Nightly 72.0a1 (2019-10-31)
- 71.0b1 DevEdition
Affected platforms
- Windows 10 x64
- Windows 8.1 x64
- Ubuntu 18.04 x64
- macOS 10.15
Steps to reproduce
- Launch Firefox and open a pdf page (e.g.: http://www23.statcan.gc.ca/imdb-bmdi/pub/instrument/3901_Q2_V3-eng.pdf).
- Click the Presentation Mode button to enter in full screen mode.
- Right click to open context menu and select "View Background Image" option.
Expected result
- The browser switch back to Normal mode (exits the Presentation Mode) and the page with background image is displayed.
Actual result
- The page with background image is displayed, but you don't have any option to exit from here or to switch back to normal mode. Please see attachment "issue.gif". The only actions you can do here are to Restart browser or to Close browser.
Regression range
- This is a recent regression. The issue is NOT reproducible on Firefox 70.0.1, but it is reproducible on Developer Edition 71.0b1. I will come back with the complete regression range ASAP.
Additional notes
- The issue is NOT reproducible on Windows 7 x64.
Reporter | ||
Updated•4 years ago
|
Comment 1•4 years ago
|
||
Bugbug thinks this bug is a regression, but please revert this change in case of error.
Comment 2•4 years ago
|
||
We should probably just morph this into a "remove the context menu option".
Comment 3•4 years ago
|
||
Regression range
- First bad: 20191011215012
- Last good: 20191010214019
- Pushlog: https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=15ff5c7542067cfa94da1ce0c896141be4afea78&tochange=c7f931c95c789a1b997941f16ee53eb5a942f3f1
- Potential regressor: Bug 1505916
Assignee | ||
Comment 4•4 years ago
|
||
(In reply to Jim Mathies [:jimm] from comment #2)
We should probably just morph this into a "remove the context menu option".
In that case, this bug is probably a duplicate of bug 1498555.
Given that the "View Background Image" never makes sense in the PDF Viewer, as far as I'm concerned, always hiding the context menu entry seems reasonable. Assuming that's the direction you want to go here, I can see about submitting a proper patch later.
Comment 5•4 years ago
|
||
The patch above seems reasonable.
Jonas, want to open a phabricator review request?
Assignee | ||
Comment 6•4 years ago
|
||
Given that the "View Background Image" option never makes sense in the PDF Viewer, it's simply always removed from the context menu.
Assignee | ||
Comment 7•4 years ago
|
||
(In reply to Brendan Dahl [:bdahl] from comment #5)
The patch above seems reasonable.
Jonas, want to open a phabricator review request?
Thanks; I've submitted a patch the correct way now :-)
I also tried, to the best of my ability, to add a new test for this and hopefully it actually makes sense.
Comment 8•4 years ago
|
||
This is an edge case and our last beta goes to build this Thursday. If the patch lands on nightly before I would be happy to consider an uplift but I am marking it as fix-optional for 71 given that the beta cycle is coming to an end.
Updated•4 years ago
|
Comment 9•4 years ago
|
||
Too late for a fix in 72, happy to take a patch for 74. I'm marking the bug fix-optional to remove it from weekly regression triage.
Updated•4 years ago
|
Comment 10•4 years ago
|
||
Pushed by bdahl@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/832da9d56b03 Hide the "View Background Image" context menu option in the PDF Viewer. r=bdahl
Comment 11•4 years ago
|
||
bugherder |
Updated•4 years ago
|
Updated•4 years ago
|
Updated•4 years ago
|
Description
•