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.

Sign up for GitHub

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

Jump to bottom

Update docs site styles. #1039

Merged
merged 5 commits into from Dec 3, 2018
Merged

Update docs site styles. #1039

merged 5 commits into from Dec 3, 2018

Conversation

tkh44
Copy link
Member

@tkh44 tkh44 commented Nov 27, 2018

What:

  • Redirect documentation to introduction
  • Adjust fonts
  • Adjust spacing between items
  • Change Prism colors to dracula theme

Checklist:

  • Documentation
  • [n/a] Tests
  • [n/a] Code complete

@tkh44
Update docs site styles.
64cc07e 
* Redirect documentation to introduction
* Adjust fonts
* Adjust spacing between items
* Change Prism colors to dracula theme
@tkh44 tkh44 requested a review from emmatown November 27, 2018 08:05
emmatown
emmatown reviewed Nov 27, 2018
View reviewed changes
Copy link
Member

@emmatown emmatown left a comment •
edited

Choose a reason for hiding this comment

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

I really like all this!

I think the live code and regular code examples should have the same spacing, border radius, font and background color.

Also, update your global version of yarn and refresh the yarn.lock

@tkh44
Copy link
Member Author

tkh44 commented Nov 27, 2018

I'll update those and update yarn. Thanks.

@quantizor
Copy link
Contributor

I like the new syntax theme 👍

I'd probably drop the nav item size to 16px and bump up the code samples to around 16px as well so everything looks a little more proportionate

@tkh44 tkh44 merged commit caf2841 into master Dec 3, 2018
@emmatown emmatown deleted the update-docs-site-look branch December 3, 2018 20:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@emmatown emmatown emmatown left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@tkh44 @quantizor @emmatown