Skip to content

Re-enable editing, if necessary, when exiting PresentationMode (PR 16659 follow-up) - #16675

Merged
Snuffleupagus merged 1 commit into
mozilla:masterfrom
Snuffleupagus:pr-16659-followup
Jul 12, 2023
Merged

Re-enable editing, if necessary, when exiting PresentationMode (PR 16659 follow-up)#16675
Snuffleupagus merged 1 commit into
mozilla:masterfrom
Snuffleupagus:pr-16659-followup

Conversation

@Snuffleupagus

@SnuffleupagusSnuffleupagus commented Jul 11, 2023

Copy link
Copy Markdown
Collaborator

This regressed in PR #16659, when the signature of the PDFViewer.annotationEditorMode-setter was changed, and it currently leads to an Error being thrown when exiting PresentationMode.

@Snuffleupagus

Copy link
Copy Markdown
CollaboratorAuthor

/botio-linux preview

@moz-tools-bot

Copy link
Copy Markdown
Collaborator

From: Bot.io (Linux m4)


Received

Command cmd_preview from @Snuffleupagus received. Current queue size: 0

Live output at: http://54.241.84.105:8877/987d932e9afcf19/output.txt

…659 follow-up)
This regressed in PR 16659, when the signature of the `PDFViewer.annotationEditorMode`-setter was changed, and it currently leads to an Error being thrown when exiting PresentationMode.
@moz-tools-bot

Copy link
Copy Markdown
Collaborator

From: Bot.io (Linux m4)


Success

Full output at http://54.241.84.105:8877/987d932e9afcf19/output.txt

Total script time: 1.44 mins

Published

@calixtemancalixteman left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thank you for thinking about this.

@Snuffleupagus
Snuffleupagus merged commit 040d5bb into mozilla:masterJul 12, 2023
@Snuffleupagus
Snuffleupagus deleted the pr-16659-followup branch July 12, 2023 19:41
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@Snuffleupagus@moz-tools-bot@calixteman