-
Notifications
You must be signed in to change notification settings - Fork 189
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
resolves #473 adds extra documentation on document structure and header #474
Conversation
Co-authored-by: Guillaume Grossetie <ggrossetie@yuzutech.fr>
I've applied your suggestions. Please let me know if anything else requires editing. |
Mostly capitalisation channges. Co-authored-by: Guillaume Grossetie <ggrossetie@yuzutech.fr>
Co-authored-by: Guillaume Grossetie <ggrossetie@yuzutech.fr> Co-authored-by: Dan Allen <dan.j.allen@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's all good!
I will let @mojavelinux a chance to review one last time before merging 😉
Thanks @jhkuperus for your contribution and @mojavelinux for your review 👍🏻 |
You're welcome and thanks for helping me help out the next person to get a smoother experience with this awesome tool :) |
I've added the suggested extra notes and links. The sample-presentation now has an
IMPORTANT
block pointing out the need for proper document structure. It links to the page about the Title Slide. On the Title Slide page, I've added both a good and a bad example of a Document Header and a link to the AsciiDoc page on the Document Header.I couldn't find how to generate the site to test if my changes looked good. I installed antora, but I might have a newer version as it complains about incorrect configuration. If someone could tell me how to generate the site to verify if links work, I'll gladly do that before finalizing this PR.
I'm looking forward to your review.