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

Update Docs Styling & Nav #912

Merged
merged 5 commits into from
Jan 2, 2025
Merged

Update Docs Styling & Nav #912

merged 5 commits into from
Jan 2, 2025

Conversation

yeldarby
Copy link
Contributor

@yeldarby yeldarby commented Jan 1, 2025

Description

This is a first pass at updating the nav architecture and styling of the docs in preparation for a major revamp.

Changes:

  • Updating logo, favicon, fonts & colors to match our brand
  • Updates homepage to lie outside of nav hierarchy
    • Fixes mobile nav (before it was completely unclear you landed in a sub-nav & had to click the back arrow)
    • Hides sidebar to set the index up to be a better landing page
  • Nav hierarchy update (renames the legacy "Roboflow Inference" section to "Models" to more closely match what it is & moves some pages around to support that)
  • Updated styling (font sizes, margin, padding), simplified header w/o giant dark-mode toggle button

Homepage Before:
image

Homepage After:
image

Inner Page Before:
image

Inner Page After:
image

Mobile Nav (From Homepage) Before:
image

Mobile Nav (From Homepage) After:
image

Type of change

  • New feature (non-breaking change which adds functionality)

How has this change been tested, please provide a testcase or example of how you tested the change?

Non-functional change; previewed via local docs server.

Any specific deployment considerations

Just docs.

Docs

  • This is a docs change.

Copy link
Contributor

@hansent hansent left a comment

Choose a reason for hiding this comment

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

lgtm

@yeldarby yeldarby merged commit bd6e98e into main Jan 2, 2025
71 checks passed
yeldarby added a commit that referenced this pull request Jan 2, 2025
@yeldarby yeldarby deleted the docs/styling branch January 2, 2025 21:57
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