Open Bug 1541376 Opened 5 years ago Updated 1 year ago

'Take a screenshot' button is not functional in PDF presentation mode

Categories

(Firefox :: Screenshots, defect, P3)

defect

Tracking

()

Tracking Status
firefox-esr68 --- wontfix
firefox-esr78 --- wontfix
firefox66 --- wontfix
firefox67 --- wontfix
firefox68 --- wontfix
firefox69 --- wontfix
firefox70 --- wontfix
firefox71 --- wontfix
firefox77 --- wontfix
firefox78 --- wontfix
firefox79 --- fix-optional

People

(Reporter: gpalko, Unassigned)

References

Details

(Keywords: regression, Whiteboard: [screenshots-extension])

Affected versions:
Nightly 68.0a1 (2019-04-02) (64-bit)
Beta 67.0b7 (64-bit)
Release 66.0.2

Affected platform:
Windows 7/10
Ubuntu 16.04
Mac OS 10.13

Steps to reproduce:

Expected result:
The user should be able to create the screenshot.

Actual Results:
Clicking the button does nothing.

Regression-range: will try to find it asap.

Regression-range: will try to find it asap.

The "Take a Screenshot" appears in contextmenu since Firefox56.
And the problem is also reproducible on Firefox56.
So, I think this is not a regression. But, this seems implementation label bug.

Running mozregression tool indicates that this issue has started to repro in the same pushlog as the one containing the actual feature: https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=44e41de60c48a94a004494dd48f57dd1d8c157d8&tochange=6a235092bda7c20cb83651d3404284fe36d04c1b

Unfortunately, I could not pinpoint the exact build that caused this, but as stated in Comment 1, this is most likely an implementation bug. Removing the regression tag.

Has STR: --- → yes

The priority flag is not set for this bug.
:ianbicking, could you have a look please?

For more information, please visit auto_nag documentation.

Flags: needinfo?(ianb)
Flags: needinfo?(ianb)
Priority: -- → P3

Presentation mode uses the Fullscreen API (content fullscreen). The Take a Screenshot overlay is visible on the PDF viewer when you exit full screen. Perhaps it isn't possible to show that overlay in content fullscreen, so it would make sense to hide the menu item in that case. Problem is, extensions cannot distinguish between content fullscreen and browser fullscreen, so some kind of pdf.js specific detection may also be needed.

Bugbug thinks this bug is a regression, but please revert this change in case of error.

Keywords: regression

Reproduced on latest Nightly build 71.0a1 (2019-09-12) on Windows 7 x64.

Hi, I would like to start working on this bug. Do you have any more info on how I can attempt to fix this?

Reproduced on latest Nightly 79.0a1 (2020-06-22), Firefox 77.0.1 and Firefox 78 beta - tested on Windows 10 x64.

Severity: normal → S3

Still reproducing on Win10 using FF 109.0b9(20230105190654).

Whiteboard: [screenshots-extension]
You need to log in before you can comment on or make changes to this bug.