Skip to content

Conversation

@kylecarbs
Copy link
Member

Summary

This PR introduces shadcn UI components and refactors existing UI elements to use them for consistent styling and better accessibility.

Changes

New shadcn components:

  • context-menu.tsx - Context menu primitive styled with app colors
  • dialog.tsx - Dialog primitive with app-consistent styling
  • select.tsx - Select primitive with compact sizing

Component updates:

  • button.tsx - Updated variants to use accent colors as primary
  • components.json - Updated paths for browser/components

Refactored to use shadcn:

  • Theme selectors (ThemeSelector, GeneralSection)
  • Model management (ModelsSection, ModelRow)
  • Provider settings (ProvidersSection)
  • Modal buttons (CancelButton, PrimaryButton, DangerButton)
  • Settings modal navigation
  • Project creation modal
  • Workspace header
  • Settings button

Notes

  • The Select component wrapper maintains the existing API for easy migration
  • Many other button usages remain - this PR focuses on key visible components
  • The shadcn components have been styled to match the existing app theme

Generated with mux

- Update components.json to use browser/components paths
- Add shadcn context-menu, select, and dialog primitives
- Update dialog.tsx with app-consistent styling (bg-dark, borders)
- Update select.tsx with compact sizing and theme colors
- Update button.tsx variants to use accent colors
- Replace native <select> with shadcn Select in:
  - ThemeSelector, GeneralSection, ModelsSection
  - Create Select wrapper maintaining existing API
- Replace native <button> with shadcn Button in:
  - Modal (CancelButton, PrimaryButton, DangerButton now wrap Button)
  - SettingsModal, ProvidersSection, ModelRow
  - ProjectCreateModal, SettingsButton, WorkspaceHeader

_Generated with `mux`_
@kylecarbs kylecarbs force-pushed the refactor/shadcn-ui-components branch from ab8aed3 to 2db79af Compare December 5, 2025 20:47
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