Skip to content

Add theming system and shared UI components - #27

Merged
tayloraj10 merged 4 commits into
mainfrom
pr/theming-infra
Jul 21, 2026
Merged

Add theming system and shared UI components#27
tayloraj10 merged 4 commits into
mainfrom
pr/theming-infra

Conversation

@tayloraj10

Copy link
Copy Markdown
Contributor

Summary

  • Adds a centralized MUI theme (theme/theme.ts) and a dark/light mode provider
  • Adds shared components used across auth and dashboard layouts: ThemeToggle, WaveDivider
  • Updates app.css and root.tsx to wire up the theme provider

This is a foundational PR split out of the review-dash branch so subsequent feature PRs (auth pages, dashboard, reviews) can build on top of it independently.

Test plan

  • npm run typecheck passes
  • App boots and theme/dark-mode toggle renders correctly

Introduces a centralized MUI theme, dark/light mode provider, and
shared components (ThemeToggle, WaveDivider, chip color helpers)
used across the authenticated and unauthenticated layouts.
Comment thread app/root.tsx Outdated
Comment thread app/components/WaveDivider.tsx
Comment thread app/components/ThemeToggle.tsx
Updated the theme mode handling in the application by removing the inline script for theme initialization and implementing a cookie-based approach. The ThemeModeProvider now accepts an initial mode, and the theme mode is stored in a cookie for better SSR support. Additionally, updated .gitignore to exclude Docker build artifacts.
…d to MUI

Replaces className-based Tailwind styling with MUI Box/sx in the components
that live on this branch. Tailwind itself stays installed for now since
downstream layout files (auth-pages, dashboard-layout) still use it.
All components now use MUI's sx prop / theme system; no Tailwind
utility classes remain anywhere in the app, so the plugin, deps, and
directives are no longer needed. Custom properties in app.css are
trimmed to only the tokens still referenced by plain CSS (keyframes,
scrollbar colors); component-level design tokens live in theme.ts.
@tayloraj10
tayloraj10 merged commit 047a497 into main Jul 21, 2026
1 check failed
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.

3 participants