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

Table of contents added in README.md #1545

Merged
merged 1 commit into from
Apr 7, 2016
Merged

Table of contents added in README.md #1545

merged 1 commit into from
Apr 7, 2016

Conversation

ducin
Copy link
Contributor

@ducin ducin commented Apr 2, 2016

This solves #1389. I cloned a fresh copy of master branch today.
Please merge it soon, hopefully, so that it doesn't get diverged from existing readme :)

@ducin
Copy link
Contributor Author

ducin commented Apr 2, 2016

By the way, going through the TOC I saw how many great features are here in reveal.js :)
as I was browsing through the TOC, I could see that some of the paragraphs might be reordered, some might be wrapped in bigger sections.

In general, a big overview of the TOC might be a second step.

@hakimel hakimel merged commit 8789baa into hakimel:master Apr 7, 2016
@hakimel
Copy link
Owner

hakimel commented Apr 7, 2016

Thanks! This has been merged. The TOC grew pretty huge so I might actually exclude a few sub-topics.

@ducin
Copy link
Contributor Author

ducin commented Apr 7, 2016

Thanks :) it's a good starting point to do some cleaning :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants