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 ability to present multiple files #121

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

contrun
Copy link

@contrun contrun commented Sep 4, 2022

No description provided.

@jaspervdj
Copy link
Owner

This seems reasonable and code looks good -- but I'd like to understand better what the use case is.

As presentations get lengthier, it can be useful to split them up over multiple files. But then I would also expect to be able to navigate back to earlier slides (e.g. when there's a question from the audience). Implementing this is trickier, since theoretically each file could have different settings, which doesn't really work well with the existing code.

Another thing that's a bit weird is that pressing Q does not quit immediately, but rather jumps to the next file.

@contrun
Copy link
Author

contrun commented Sep 6, 2022

Your concern is vaild. I think there are two different use cases.

One is that the same person splits his/her presentation into different files and he/she want to presents them at once. In this case, he/she naturally wants a fluent transition (the features as you mentioned, no pressing Q and easily going back to previous file, are indeed desirable).

Another theoretical usage is that the many people present with patat in the same computer, or the same person gives a few lectures in the same day, and there are some breaks between the lectures. If so, they may want some kind of isolation between the sessions. I must admit the chance of this use case is slim. What I had in mind is the first use case.

I just thought giving people hints on the length of one file and requiring them to press Q to present a new file is acceptable. And it is quite easy to implement.

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.

None yet

2 participants