feat: Add dark mode theming system with case study migration#196
Draft
feat: Add dark mode theming system with case study migration#196
Conversation
- Added semantic color CSS variables that adapt to theme (light/dark) - Created ThemeProvider component for theme management - Added useTheme hook for programmatic theme control - Migrated Dialog component to use semantic color tokens - Added dark mode story to Dialog Storybook examples - Created comprehensive theming documentation The theming system uses CSS custom properties that automatically adjust based on the active theme. This provides a foundation for full dark mode support and compatibility with shadcn/ui components. Breaking Changes: None - fully backward compatible 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Updated Button component to use semantic color tokens for secondary, minimal, and danger-secondary variants - Updated Alert component to remove hardcoded text colors, using opacity for subtlety - Added DarkMode stories for Button and Alert components to showcase theme switching - Fixed ThemeProvider context value memoization for performance This serves as a case study for migrating the remaining components to the new theming system. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🔍 Summary
Type: Feature Enhancement
Status: Draft - Awaiting Senior Developer Review
Breaking Changes: None
Introducing dark mode theming system to Hailstorm with case study migration of 3 components.
📋 Changes Overview
Infrastructure Added
docs/THEMING.mdComponents Migrated (Case Study)
hasCloseButtonprop, migrated to semantic tokens🧪 Testing Evidence
📊 Impact Analysis
✅ Checklist
🎯 Next Steps (Pending Approval)
❓ Review Questions
background,foreground,border)📸 Visual Testing
Run Storybook to see dark mode in action:
Navigate to:
Note: This is a draft PR for architectural review. No production deployment intended until approved.