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

docs: reorganize siteConfig entry (#436) #1027

Merged
merged 1 commit into from Oct 12, 2018

Conversation

tanhauhau
Copy link
Contributor

@tanhauhau tanhauhau commented Oct 10, 2018

set the setConfig fields to h4 instead of li, will create some spacing and make the docs less clustered.

Motivation

(Write your motivation here.)

Have you read the Contributing Guidelines on pull requests?

yes

Test Plan

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work. Bonus points for screenshots and videos!)

Related PRs

(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/facebook/docusaurus, and link to your PR here.)

@facebook-github-bot facebook-github-bot added the CLA Signed Signed Facebook CLA label Oct 10, 2018
@docusaurus-bot
Copy link
Contributor

docusaurus-bot commented Oct 10, 2018

Deploy preview for docusaurus-preview ready!

Built with commit a329869

https://deploy-preview-1027--docusaurus-preview.netlify.com

Copy link
Contributor

@JoelMarcey JoelMarcey left a comment

Choose a reason for hiding this comment

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

Thanks for this. I think this should work better than the basic list we have now. However, can you not add your changes to the versioned docs? Instead, make your changes to https://github.com/facebook/Docusaurus/blob/master/docs/api-site-config.md

@tanhauhau
Copy link
Contributor Author

@JoelMarcey i've fixed the commit

@JoelMarcey
Copy link
Contributor

❤️

Copy link
Contributor

@JoelMarcey JoelMarcey left a comment

Choose a reason for hiding this comment

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

This is great. Thanks!

@JoelMarcey JoelMarcey merged commit 1275005 into facebook:master Oct 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed Signed Facebook CLA
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants