-
-
Notifications
You must be signed in to change notification settings - Fork 36
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
Some minor updates #68
Conversation
- enhance readability with slightly grey background - emphasizes code blocks - more explicit hyperref setup - make more use of fancyhdr package to help with navigation in PDF output
- add TOC to output - split list of options into subsubsections - change `rendering.num_depth` to 4 accordingly - add highlighting to code blocks of arguments chapter by declaring language
@lise-henry FYI, when testing the changes of the output, crowbook doesn't use the current templates but the templates included in the version of crowbook used. Not sure whether you'd like to change that in |
Not sure what you mean about the templates ? If you specify a file in the configuration it should use this one, else it is a bug. If you don't you have to recompile, if you change the source file because it's included in the binary so this is normal. Don't worry about travis I see where the problem is, i'll fix it when I merge. It will probably be in a few days, though, because I think the LaTeX/Crowbook integration needs to be tweaked a little (e.g. \chaptermark will cause an error if the class doesn't have chapters so it needs to be detected, and I'd like to make headers, footers and papercolor a bit more configurable). Anyway thanks for the PR, i'll merge it asap :) |
Maybe it's unclear what I meant. Let me try again. I assumed the Not an issue in any case. Maybe worth adding to the docs as an FYI, though. |
I'd like to close this for now. Running into the following issue:
with the suggested changes of the latex template. Cannot reproduce every time and every repository where I am using it. Where can I find the |
Looking at: the only possible reason might be that it breaks without a Sorry for the mess. Should've tested this more thoroughly. |
See commits for details.