feat(InviteUserModal): fix modal layout and add dark mode support#61
Merged
garrity-miepub merged 2 commits intomainfrom Feb 3, 2026
Merged
feat(InviteUserModal): fix modal layout and add dark mode support#61garrity-miepub merged 2 commits intomainfrom
garrity-miepub merged 2 commits intomainfrom
Conversation
Modal Structure: - Import and use ModalBody component for proper padding/spacing - Content now properly wrapped: ModalHeader → ModalBody → ModalFooter Dark Mode Support: - Replace bg-gray-50/dark:bg-gray-800 with bg-muted token - Replace text-gray-*/dark:text-gray-* with text-foreground/text-muted-foreground - Textarea uses border-input, bg-background, text-foreground - Focus ring uses focus:ring-primary instead of hardcoded blue
Deploying ui with
|
| Latest commit: |
6432743
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://69b3290f.ui-6d0.pages.dev |
| Branch Preview URL: | https://feature-invite-user-modal-fi.ui-6d0.pages.dev |
There was a problem hiding this comment.
Pull request overview
This PR improves the InviteUserModal component by adding proper modal structure using ModalBody and implementing dark mode support through design system tokens.
Changes:
- Import and use ModalBody component for proper content padding and spacing
- Replace hardcoded dark mode color classes with design system tokens (bg-muted, text-foreground, text-muted-foreground, border-input)
- Update focus ring styling to use primary color token instead of hardcoded blue
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Stories Structure: - Convert from render-based to args-based stories for better docs integration - Add decorator with transform: translateZ(0) to contain fixed-position modal - Change layout from 'centered' to 'fullscreen' for proper modal rendering - Increase container height to 700px to show full modal content Dark Mode Support: - Modal now properly inherits theme on docs page - Works with both light and dark mode via inline rendering Storybook Improvements: - Add comprehensive argTypes with descriptions - Remove unused interactive wrapper components - Stories use args for better controls panel integration
garrity-miepub
force-pushed
the
feature/invite-user-modal-fixes
branch
from
dc384b1 to
6432743
Compare
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.
Modal Structure:
Dark Mode Support:
invite-user-modal.mov