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

Add sele mode so that each input file has an even number of pages #21

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

cykerway
Copy link

@cykerway cykerway commented Dec 7, 2015

Hello,

I find this tool very useful and I just added a new select/concat mode sele to it in case people want to read the generated output in double-page layout or they want to print it out.

Basically it inserts an empty page after each input if that input ends with an odd page number. For example if input files have 3, 5, 7 pages, resp. then in sele mode the output will be (3 + 1) + (5 + 1) + (7 + 1) = 18 pages.

The empty page is generated with reportlab and is set to have the same size as the page previous to it.

Thank you for consideration.

These two modes are added to facilitate concatenation of slides and
lecture notes.
These two modes are added to facilitate concatenation of slides and
lecture notes.
Add mode ``sele``. It inserts an empty page at the end of each input
file if necessary to make sure each input file has an even number of
pages in the generated output file. This is good for people reading in
double-page layout or if they want to print the generated pdf file.
@hellerbarde
Copy link
Owner

Oh, this is very clever. That's a great idea! I'm a little wary about adding another dependency, but this might be worth it. I'll have to think about it some more. Also, I don't really like the sele. I think it should be part of sel/cat, but maybe an option like "--add-empty-page" or "-e". Or maybe this is the moment where a .config/stapler/staplerrc configuration file should come in...

Ideas?

@hellerbarde
Copy link
Owner

sorry for leaving you hanging for so long. I will tackle this over the weekend.

@corollari
Copy link
Collaborator

See cykerway#1

@hellerbarde
Copy link
Owner

So, I am back, developing this application. I have started rewriting the command parsing, to enable options to commands. After some consideration, I did not find adding a letter to a command to be very intuitive and I decided to make it possible to extend commands with some sort of arguments/options.
At that point, your changes will become easy to incorporate, as you have structured your changes quite well. 👍
Stay tuned :) And sorry for dragging my feet. Life happened.

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.

3 participants