-
Notifications
You must be signed in to change notification settings - Fork 0
Fix: Minor style adjustments (tabs & input) #2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR makes minor style adjustments to the tabs and input components, focusing on spacing and sizing defaults.
- Changes the default input size from 'large' to 'medium'
- Refactors input sizing from width-based to text-size-based styling
- Removes top margin from tabs content component
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| src/components/ui/tabs.tsx | Removes lsd:mt-4 class from TabsContent to eliminate default top margin |
| src/components/ui/input.tsx | Changes default size to 'medium', refactors sizing logic from width classes to text-size classes, adds dynamic label sizing, and consolidates horizontal padding |
| src/cosmos/fixtures/Input.fixture.tsx | Updates documentation to reflect that 'Medium' is now the default size instead of 'Large' |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
No description provided.