Skip to content
This repository has been archived by the owner on Feb 13, 2025. It is now read-only.

Sidebar, Toggle & Wrapper #8

Merged
merged 16 commits into from
Dec 16, 2023
Merged

Sidebar, Toggle & Wrapper #8

merged 16 commits into from
Dec 16, 2023

Conversation

Zaid-maker
Copy link
Owner

@Zaid-maker Zaid-maker commented Dec 15, 2023


Generated summary (powered by Graphite)

TL;DR

This pull request adds a new sidebar component and its related files to the project. It also includes changes to the layout and package dependencies.

What changed

  • Added a new sidebar component (sidebar/index.tsx) that includes a toggle button (sidebar/toggle.tsx) to expand or collapse the sidebar.
  • Created a wrapper component (sidebar/wrapper.tsx) for the sidebar that handles its styling and behavior.
  • Modified the layout.tsx file to include the new sidebar component alongside the existing navbar component.
  • Updated the package.json and package-lock.json files to include the zustand dependency.

How to test

  1. Checkout this branch and run the project.
  2. Navigate to the browse section of the app.
  3. Verify that the sidebar is displayed on the left side of the screen.
  4. Click on the toggle button to expand or collapse the sidebar.
  5. Ensure that the sidebar expands or collapses accordingly.

Why make this change

  • The new sidebar component provides additional functionality and improves the user experience by allowing users to expand or collapse the sidebar as needed.
  • The addition of the zustand dependency enables the use of a state management solution for the sidebar component, making it easier to manage its state and interactions.

Copy link

semanticdiff-com bot commented Dec 15, 2023

Review changes with SemanticDiff.

Analyzed 10 of 10 files.

Overall, the semantic diff is 3% smaller than the GitHub diff.

Filename Status
✔️ app/(browse)/_components/container.tsx Analyzed
✔️ app/(browse)/_components/sidebar/index.tsx Analyzed
✔️ app/(browse)/_components/sidebar/toggle.tsx Analyzed
✔️ app/(browse)/_components/sidebar/wrapper.tsx Analyzed
✔️ app/(browse)/layout.tsx 39.2% smaller
✔️ components/hint.tsx Analyzed
✔️ components/ui/tooltip.tsx Analyzed
✔️ package-lock.json 0.39% smaller
✔️ package.json 40.0% smaller
✔️ store/use-sidebar.ts Analyzed

Copy link

vercel bot commented Dec 15, 2023

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

Name Status Preview Comments Updated (UTC)
gamehub ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 16, 2023 1:52pm

@Zaid-maker Zaid-maker changed the title Sidebar Sidebar, Toggle & Wrapper Dec 15, 2023
@Zaid-maker Zaid-maker marked this pull request as ready for review December 15, 2023 22:10
Copy link

gitstream-cm bot commented Dec 15, 2023

🥷 Code experts: no user but you matched threshold 10

Zaid-maker has most 👩‍💻 activity in the files.
Zaid-maker has most 🧠 knowledge in the files.

See details

app/(browse)/layout.tsx

Activity based on git-commit:

Zaid-maker
DEC 14 additions & 1 deletions
NOV
OCT
SEP
AUG
JUL

Knowledge based on git-blame:
Zaid-maker: 100%

package-lock.json

Activity based on git-commit:

Zaid-maker
DEC 5310 additions & 113 deletions
NOV
OCT
SEP
AUG
JUL

Knowledge based on git-blame:
Zaid-maker: 100%

package.json

Activity based on git-commit:

Zaid-maker
DEC 44 additions & 4 deletions
NOV
OCT
SEP
AUG
JUL

Knowledge based on git-blame:
Zaid-maker: 100%

To learn more about /:\ gitStream - Visit our Docs

@Zaid-maker Zaid-maker marked this pull request as draft December 15, 2023 22:11
@Zaid-maker Zaid-maker marked this pull request as ready for review December 16, 2023 10:44
@Zaid-maker
Copy link
Owner Author

Merge activity

# for free to subscribe to this conversation on GitHub. Already have an account? #.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant