Skip to content

UI rearrange#548

Closed
TheJeran wants to merge 2 commits intomainfrom
jp/new-ui
Closed

UI rearrange#548
TheJeran wants to merge 2 commits intomainfrom
jp/new-ui

Conversation

@TheJeran
Copy link
Collaborator

@TheJeran TheJeran commented Feb 5, 2026

Addressing #535 a bit. Moved the shadcn and some of my tools into a widgets folder. Moved the Navbar stuff into navbar folder.

Thinking about doing an elements folder for the little visual UI elements that are part of larger components

@lazarusA
Copy link
Member

lazarusA commented Feb 5, 2026

No, shadcn needs components/ui by convention, so all shadcn components need to be under ui/..., and possibly all future ones from shadcn.

The rest is ok!

src/
├── app/                          # Next.js app router (keep as-is)
│   ├── layout.tsx
│   ├── page.tsx
│   └── ...
│
├── components/
│   ├── ui/                       #  KEEP - Generic design system components, for all `shadcn` components
│   │   ├── button.tsx
│   │   ├── card.tsx
│   │   ├── dialog.tsx
│   │   ├── slider.tsx

@TheJeran
Copy link
Collaborator Author

TheJeran commented Feb 5, 2026

I mean it's working as is. Why does shad need to be in components/ui?

@lazarusA
Copy link
Member

lazarusA commented Feb 5, 2026

shadcn needs this by default (installing new ones will also go there), and if there is a need for updates later, it will look for them there, its a shadcn thing, not me 😄 .

@TheJeran
Copy link
Collaborator Author

TheJeran commented Feb 5, 2026

Alright I will redo this then

@TheJeran TheJeran closed this Feb 5, 2026
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