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

Moved images and demos to separate folders, cleaned up examples, updated documentation. #21

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

Conversation

torsteinnh
Copy link

I updated the latex templates to suit my own workflow a bit better, feel free to merge if it is useful for you.
The underlying goal is to make it more usable in overleaf, now the whole folder can be zipped and uploaded when creating a new slide without cluttering up the workspace too much.
It would be ideal if the .sty files could be moved to a subfolder as well, but I didn't have time to figure that out as of now.

@kellertuer
Copy link
Contributor

Thanks for the ideas!

For the demos that looks good, for the other parts – one original idea (probably even before I took over) was to have all In one folder without subfolders so it is “move these files around and it works”.
And for sure that is a tradeoff between cluttering and non-working-after-copying (because one forgot to adapt folders).

So even for the images this might be dangerous, I have to try. I personally have that installed in my LaTeX path anyways and do never copy it around, which is the more advanced version – I am usually just too lazy to explain that for example for Windows users, where I always have to check where the LaTeX path is.
So, I will have to check for the images being in a subfolder, but the demos is a good idea, since they are just used in the docs, and there we can make sure that works.

@torsteinnh
Copy link
Author

Thank you for the quick reply, let me know if the image part don't work, and I'll revert that part on my branch.

@kellertuer
Copy link
Contributor

I will check, I think there was some good reason we did not move those – folder indication on windows, something like that, I have to find time to check.

@kellertuer
Copy link
Contributor

I checked, and I do not feel 100% safe with moving the images;
moving the example PDFs is fine since they are included in the anyways pre-rendered docs, but the images might act different on different systems, so I would prefer to provide them in the same folder as the presentation main file.

Besides that we originally had a “start in English” and “start in Norwegian” example. Any reason why you unified that to one example, that is just the English one?

@torsteinnh
Copy link
Author

I see, I moved the images back out now.

The main reason for merging the examples was to avoid mistakes where forget to delete the unused ones and start working on both. This recently happened to me with a shared overleaf project, and I had to manually merge the documents together.

@kellertuer
Copy link
Contributor

Well, on the other hand, a person not so experienced with LaTeX and options in loading themes, but not find the right option to switch the language. That was the reason to provide 2 examples.

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