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

🐛 Fix sidebar layout #943

Merged
merged 35 commits into from
Jul 3, 2023
Merged

🐛 Fix sidebar layout #943

merged 35 commits into from
Jul 3, 2023

Conversation

asim-shrestha
Copy link
Contributor

About

Previously, the sidebar would always start as closed but then open once the state was changed. This PR makes the sidebar open by default on desktop.

Notes

  • Create two sidebars, one for desktop view and one for mobile view. This is what is done in TailwindUI components
  • Refactor Sidebar.tsx to Dashboard.tsx and extract Sidebar to its own component
  • Fix scroll bar to have border radius everywhere

@vercel
Copy link

vercel bot commented Jul 3, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
agent-gpt 🛑 Canceled (Inspect) Jul 3, 2023 7:06am
docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 3, 2023 7:06am

@ergomake
Copy link

ergomake bot commented Jul 3, 2023

Hi 👋

We couldn't create a preview environment for this pull-request 😥

You can see your environment build logs here. Please double-check your docker-compose.yml file is valid.

If you need help, email us at [email protected] or join Discord.

Click here to disable Ergomake.

@ergomake
Copy link

ergomake bot commented Jul 3, 2023

Hi 👋

We couldn't create a preview environment for this pull-request 😥

You can see your environment build logs here. Please double-check your docker-compose.yml file is valid.

If you need help, email us at [email protected] or join Discord.

Click here to disable Ergomake.

@asim-shrestha asim-shrestha merged commit 5950665 into main Jul 3, 2023
4 checks passed
@vercel vercel bot temporarily deployed to Preview – docs July 3, 2023 07:06 Inactive
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.

1 participant