1843303 - Unable to resize an image added to the PDF if the PDF is rotated and the image spills over a second page

Status: VERIFIED FIXED

(Firefox :: PDF Viewer, defect, P1)

Description

[Rares Doghi, Desktop QA]

Attachment: 2023-07-13_15h52_38.mp4

Found in

• Nightly 117.0a1 (2023-07-12)

Affected versions

• Nightly 117.0a1 (2023-07-12)

Tested platforms

• Affected platforms: ALL

Steps to reproduce

1. Open a PDF file with multiple pages in Firefox.
2. Rotate the PDF.
3. Click the Add an image button from the PDF toolbar.
4. Place the Image so that its bottom right corner is on a second page.
5. Try to resize the image.

Expected result

• The user should be able to resize the image even if the bottom right corner falls on a second page.

Actual result

• The part of the Image that spilled on the second page is not recognized by the Tool and the user cannot resize it.
  The user would have to move the added image up and then resize it, but if the image is larger than the pdf page he will not be able to resize the image.

Regression range

• N/A

Comment 1

[BugBot [:suhaib / :marco/ :calixte]]

:rdoghi, if you think that's a regression, could you try to find a regression range using for example mozregression?

Comment 2

[Calixte Denizet (:calixte)]

I can reproduce this issue with others tools.
The problem is due to the fact we add a z-index to each editor, hence it creates a stacking context which make editors lower than the different annotationEditorLayer.
I think it doesn't really make sense to have an editor which has a part of it outside the current page.

Comment 3

[BMO Github Automation]

Attachment: [mozilla/pdf.js] [Editor] Avoid to have some part of an editor outside its page (bug 1843303) (#16689)

Comment 4

[Rares Doghi, Desktop QA]

Verified as fixed in our latest Nightly build 117.0a1 (2023-07-23).