Skip to content

adding a rotate feature for pdfs (WIP)#133

Draft
IshDeshpa wants to merge 1 commit intoitsjunetime:mainfrom
IshDeshpa:rotate_feature
Draft

adding a rotate feature for pdfs (WIP)#133
IshDeshpa wants to merge 1 commit intoitsjunetime:mainfrom
IshDeshpa:rotate_feature

Conversation

@IshDeshpa
Copy link
Contributor

Currently a work in progress. I need to figure out how to rotate in a way such that I can scroll past the original rectangular bounds of the page when zoomed in. Also for some reason scrolling too much starts scaling the page a bit.

I had to fix a bug in mupdf involving incorrect 270 deg rotation matrix to get it to work in its current form.

Any feedback on implementation details would be much appreciated, as well as how to go forward from what I have now. I'm new to Rust so let me know if there's anything that is wrong.

@itsjunetime
Copy link
Owner

itsjunetime commented Feb 1, 2026

Yeah, I think this generally makes sense, no real issues right now. Thanks for doing this :) You'll want to run cargo fmt --all in the project root directory and then push again to get CI to pass, and I'll take a look at it again once we have CI verifying everything.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants