Skip to content
This repository has been archived by the owner on Feb 18, 2024. It is now read-only.

Reorganize and nest docs navigation for readability #1115

Merged
merged 4 commits into from
Sep 20, 2018

Conversation

eliperelman
Copy link
Member

Fixes #1112.

@eliperelman eliperelman added this to the v9 milestone Sep 19, 2018
@eliperelman eliperelman self-assigned this Sep 19, 2018
Copy link
Member

@edmorley edmorley left a comment

Choose a reason for hiding this comment

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

Thank you for this!

What do you think about these further tweaks?

https://screenshots.firefox.com/JvAySSrao1eEJifi/deploy-preview-1115--neutrinojs.netlify.com

@eliperelman
Copy link
Member Author

Love it, I'll revise.

@edmorley
Copy link
Member

(The yellow highlights are things that are confusing since they have words in common)

@eliperelman
Copy link
Member Author

@edmorley how's this?

screen shot 2018-09-19 at 4 55 19 pm

mkdocs.yml Outdated
- API: './api.md'
- Configuration API: './webpack-chain.md'
- Configuration: './webpack-chain.md'
- Customization: './customization.md'
- Creating Presets: './creating-presets.md'
Copy link
Member

Choose a reason for hiding this comment

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

Should we move "API" and "Creating Presets" slighly lower down (eg below middleware)?
And maybe put "Customization" above "Configuration"?

Copy link
Member Author

Choose a reason for hiding this comment

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

++

@edmorley
Copy link
Member

Sorry meant to add to the review overview comment: This is awesome - much cleaner :-)

@eliperelman eliperelman merged commit 807c8d2 into neutrinojs:master Sep 20, 2018
@eliperelman
Copy link
Member Author

w00t

@timkelty
Copy link
Contributor

Great job, guys!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Development

Successfully merging this pull request may close these issues.

3 participants