Skip to content

feat(react): create TimePickerField component#1657

Merged
gcornut merged 2 commits into
masterfrom
feat/react_create_time_picker_field_component
May 25, 2026
Merged

feat(react): create TimePickerField component#1657
gcornut merged 2 commits into
masterfrom
feat/react_create_time_picker_field_component

Conversation

@gcornut
Copy link
Copy Markdown
Member

@gcornut gcornut commented May 7, 2026

CF-1403

General summary

Create a time picker field based on a SelectTextField with generated time suggestions, with min/max times and with parsing of user entered time

StoryBook lumx-vue: https://eed80a613--697a023f84e832e23544fb3c.chromatic.com/

StoryBook lumx-react: https://eed80a613--5fbfb1d508c0520021560f10.chromatic.com/

@gcornut gcornut force-pushed the feat/react_create_time_picker_field_component branch from 8b3d8a1 to 77a7822 Compare May 7, 2026 15:06
@gcornut gcornut marked this pull request as ready for review May 7, 2026 15:54
@gcornut gcornut requested review from a team and lumautomation as code owners May 7, 2026 15:54
@gcornut gcornut force-pushed the feat/react_create_time_picker_field_component branch from 77a7822 to 40debd4 Compare May 20, 2026 09:25
@gcornut gcornut force-pushed the feat/react_create_time_picker_field_component branch 3 times, most recently from a3fb964 to 304da32 Compare May 21, 2026 14:05
@gcornut gcornut force-pushed the feat/react_create_time_picker_field_component branch from 304da32 to 7e55f87 Compare May 21, 2026 15:12
@gcornut gcornut force-pushed the feat/react_create_time_picker_field_component branch from 7e55f87 to 6e4efb4 Compare May 22, 2026 09:23
Copy link
Copy Markdown
Contributor

@juanigalan91 juanigalan91 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

I've also tested it and we can select times as well as entering a custom one

@gcornut gcornut force-pushed the feat/react_create_time_picker_field_component branch from 6e4efb4 to 9cb13db Compare May 25, 2026 10:54
@github-actions
Copy link
Copy Markdown

Visual Regression Report

@lumx/react — 3 difference(s), 7 new

@lumx/vue — 1 difference(s)

@lumx/react vs @lumx/vue — 1 difference(s), 451 matched, 130 React-only, 3 Vue-only


View full report · Download artifacts

@gcornut
Copy link
Copy Markdown
Member Author

gcornut commented May 25, 2026

Test OK ✅

@gcornut gcornut merged commit 5524428 into master May 25, 2026
18 checks passed
@gcornut gcornut deleted the feat/react_create_time_picker_field_component branch May 25, 2026 11:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants