Skip to content

Clean up sidebar #35

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

Merged
merged 7 commits into from
Feb 13, 2019
Merged

Clean up sidebar #35

merged 7 commits into from
Feb 13, 2019

Conversation

alexkrolick
Copy link
Collaborator

@alexkrolick alexkrolick commented Feb 13, 2019

These are mostly formatting changes except for condensing the Reason docs into one section.

  • condense sidebar elements with only one child to a single nav entry
    • tweak nav alignment to account for subheadings so they are aligned the same as other elements
  • add full titles for wrapper library docs
  • condense reason docs and add more links back to language documentation
    • @wyze is this OK with you?
  • fix some broken/wonky layout items
    • vertically center the octopii 🐙 😄
    • broken svg
    • lighten link color
    • use 3 instead of 4 columns for the ecosystem logos since there are 6

screen shot 2019-02-13 at 2 08 21 am

@wyze
Copy link
Contributor

wyze commented Feb 13, 2019

I'm okay with the layout structure. I had one comment about the example that was brought to my attention yesterday in a PR.

@alexkrolick alexkrolick merged commit 60d19f3 into master Feb 13, 2019
@alexkrolick alexkrolick deleted the organize-reason branch February 13, 2019 18:02
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.

2 participants