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

feat: update docs navigation #568

Merged
merged 2 commits into from
Oct 17, 2023
Merged

feat: update docs navigation #568

merged 2 commits into from
Oct 17, 2023

Conversation

abtx
Copy link
Collaborator

@abtx abtx commented Oct 16, 2023 •
edited
Loading

  • updates nav styling

Caveats:

  • menus are aligned quite well visually, but there are small differences
  • the mobile nav is implemented using react headless UI, which doesn't behave well on touch devices - it collapses open group on scroll. need to fix that.

Last build is on surge

Preview:

Screenshot 2023-10-16 at 16 36 38

Screenshot 2023-10-16 at 16 36 41
Screenshot 2023-10-16 at 16 36 44
Screenshot 2023-10-16 at 16 36 48
Screenshot 2023-10-16 at 16 36 51
Screenshot 2023-10-16 at 16 36 53

@abtx abtx changed the title Feat/update nav feat: update docs navigation Oct 16, 2023
@abtx abtx self-assigned this Oct 16, 2023
@abtx abtx marked this pull request as ready for review October 16, 2023 16:25
@abtx abtx requested a review from aliscott October 16, 2023 16:25
@aliscott aliscott merged commit 76c4ace into s3-release Oct 17, 2023
1 check passed
@aliscott aliscott deleted the feat/update-nav branch October 17, 2023 08:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aliscott aliscott aliscott approved these changes

Assignees

@abtx abtx

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@abtx @aliscott