-
Notifications
You must be signed in to change notification settings - Fork 71
[LG-5504] feat(input-box): add InputSegment
#3293
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
base: LG-5504/input-box-context
Are you sure you want to change the base?
Conversation
…egment component references
|
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.
I think i should break these out into smaller files
|
Size Change: 0 B Total Size: 1.61 MB ℹ️ View Unchanged
|
| * 1 | ||
| * 1970 | ||
| */ | ||
| min: number; |
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.
Update to minSegmentValue
| * 12 | ||
| * 2038 | ||
| */ | ||
| max: number; |
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.
Update to maxSegmentValue
| step?: number; | ||
|
|
||
| /** | ||
| * Whether the segment should wrap at min/max boundaries |
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.
need to update this: should only be at the max boundary
✍️ Proposed changes
This PR is the second PR in a series of PRs that adds the new
InputBoxpackage and integrates it into the existingDatePickerpackage:InputBoxContext#3290InputBox#3285This PR adds the generic
InputSegmentcomponent to the new @leafygreen-ui/input-box package, which provides reusable input segment functionality for date and time components. The component extracts common logic and UI elements previously embedded in the date-picker package, making it available for use across DatePicker, TimeInput, and other similar components.🎟 Jira ticket: LG-5504
✅ Checklist
For new components
🧪 How to test changes
InputSegmentcomponent renders correctly in their respective stories