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 guidelines for nested TOC #24

Merged
merged 1 commit into from Oct 21, 2019
Merged

Add guidelines for nested TOC #24

merged 1 commit into from Oct 21, 2019

Conversation

jenweber
Copy link
Contributor

@jenweber jenweber commented Oct 10, 2019

Right now, in the Ember Table of Contents, there can be just one topic grouping, which has an index page and subpages. This PR adds guidelines for adding one more level of nesting.

The original gist can be found here: https://gist.github.com/jenweber/2db1fdaa39f31f45490f9126fcfa8215

Here's an example from the ancient guides Cookbook:

Screen Shot 2019-10-10 at 12 34 56 PM

@MelSumner MelSumner self-requested a review October 10, 2019 16:35
Copy link
Member

@MelSumner MelSumner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💯 💯 💯

@jenweber jenweber requested a review from a team October 10, 2019 16:35
Copy link

@pzuraq pzuraq left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@jenweber jenweber merged commit dff5ad3 into master Oct 21, 2019
@jenweber
Copy link
Contributor Author

Thanks everyone for the feedback! I will share progress updates on the Discord channel for the learning team.

@jenweber jenweber deleted the nesting branch October 21, 2019 18:46
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

6 participants