Skip to content

Comments

feat(viewer): handle keyboard shortcuts for print, save, undo, etc.#5748

Open
balazs-szucs wants to merge 8 commits intoStirling-Tools:mainfrom
balazs-szucs:viewer-shortcuts
Open

feat(viewer): handle keyboard shortcuts for print, save, undo, etc.#5748
balazs-szucs wants to merge 8 commits intoStirling-Tools:mainfrom
balazs-szucs:viewer-shortcuts

Conversation

@balazs-szucs
Copy link
Collaborator

Description of Changes

Shortcut Action
Ctrl+P Print document
Ctrl+S Save/apply changes (annotations, redactions)
Ctrl+Z Undo (annotation history)
Ctrl+Shift+Z Redo (annotation history)
Ctrl+Y Redo (annotation history)
Ctrl+0 Reset zoom to fit-width
Ctrl+R Rotate clockwise
Ctrl+Shift+R Rotate counter-clockwise
Home Go to first page
End Go to last page
PageUp Previous page
PageDown Next page
Escape Close search interface

Checklist

General

Documentation

Translations (if applicable)

UI Changes (if applicable)

  • Screenshots or videos demonstrating the UI changes are attached (e.g., as comments or direct attachments in the PR)

Testing (if applicable)

  • I have tested my changes locally. Refer to the Testing Guide for more details.

Signed-off-by: Balázs Szücs <bszucs1209@gmail.com>
Signed-off-by: Balázs Szücs <bszucs1209@gmail.com>
Signed-off-by: Balázs Szücs <bszucs1209@gmail.com>
Signed-off-by: Balázs Szücs <bszucs1209@gmail.com>
@dosubot dosubot bot added size:L This PR changes 100-499 lines ignoring generated files. enhancement New feature or request labels Feb 17, 2026
@stirlingbot stirlingbot bot added the Front End Issues or pull requests related to front-end development label Feb 17, 2026
Copilot AI review requested due to automatic review settings February 20, 2026 11:46
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review is ineligible. To be eligible to request a review, you need a paid Copilot license, or your organization must enable Copilot code review.

@jbrunton96
Copy link
Contributor

/deploypr

@stirlingbot stirlingbot bot added the pr-deployed Pull request has been deployed to a test environment label Feb 20, 2026
@stirlingbot
Copy link
Contributor

stirlingbot bot commented Feb 20, 2026

🚀 PR Test Deployment

Your PR has been deployed for testing!

🔗 Test URL: http://23.22.230.180:5748
Security Disabled

This deployment will be automatically cleaned up when the PR is closed.

@ConnorYoh
Copy link
Contributor

Hi @balazs-szucs these are awesome!
I am having some issues with print and rotate. If I have clicked not the pdf before running print I get this
image

Also it runs reload unless I have clicked the pdf for rotate.

It feels like there is some context logic going on that needs tightening up.

These a great features! Improves the UX loads thanks

@balazs-szucs
Copy link
Collaborator Author

I’m having some issues with printing and rotation. If I haven’t clicked on the PDF before running print, I get this:

Yes, it uses the same shortcuts as the browser, so if the site isn’t in focus, we can’t override the browser shortcut. Hence, the browser prints the website instead of our print command overriding it to print the PDF. I'm not quite certain what we can do about that, tbh. But I'll see what I can do.

@balazs-szucs
Copy link
Collaborator Author

I was thinking about this for a while, realized it was actually my bad. Thanks for the review.

Should be fixed

@balazs-szucs
Copy link
Collaborator Author

/deploypr

@stirlingbot
Copy link
Contributor

stirlingbot bot commented Feb 21, 2026

🚀 PR Test Deployment

Your PR has been deployed for testing!

🔗 Test URL: http://23.22.230.180:5748
Security Disabled

This deployment will be automatically cleaned up when the PR is closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request Front End Issues or pull requests related to front-end development pr-deployed Pull request has been deployed to a test environment size:L This PR changes 100-499 lines ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants