Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Option to remove empty pages? #64

Open
YbCl3 opened this issue Oct 4, 2024 · 1 comment
Open

Option to remove empty pages? #64

YbCl3 opened this issue Oct 4, 2024 · 1 comment

Comments

@YbCl3
Copy link

YbCl3 commented Oct 4, 2024

Depending on the settings cropping an empty page doesn't always really make much sense, resulting in things like tiny square pages, or even errors that leave the page uncropped because it would have 0 size.

I don't know enough about how PDFs work to know if this would be a sensible thing to add, or if it would be too different from/outside the scope of the normal cropping this tool does, but it seems to me like it would be useful to have the option to completely remove the empty pages when cropping.

@abarker
Copy link
Owner

abarker commented Oct 25, 2024

I try to keep the scope of the program to cropping. If the empty pages are causing errors, though, that seems like a bug. That case could at least have some minimum size to avoid errors. I'll consider the option to drop pages below some threshold size.

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

No branches or pull requests

3 participants
@abarker @YbCl3 and others