Skip to content

[TimeRangePicker] Shift popper between start and end input on multi input field#16920

Merged
LukasTy merged 3 commits into
mui:masterfrom
LukasTy:alternate-trp-trigger-ref
Mar 14, 2025
Merged

[TimeRangePicker] Shift popper between start and end input on multi input field#16920
LukasTy merged 3 commits into
mui:masterfrom
LukasTy:alternate-trp-trigger-ref

Conversation

@LukasTy

@LukasTy LukasTy commented Mar 11, 2025

Copy link
Copy Markdown
Member

Follow up on #9431.

Adds triggerRef shifting between start and end fields when using Picker with a MultiInputTimeRangeField.

Screen.Recording.2025-03-11.at.17.59.56.mov

@LukasTy LukasTy added plan: Pro Impact at least one Pro user. scope: pickers Changes related to the date/time pickers. type: enhancement It’s an improvement, but we can’t make up our mind whether it's a bug fix or a new feature. component: TimeRangePicker labels Mar 11, 2025
@LukasTy LukasTy self-assigned this Mar 11, 2025
@mui-bot

mui-bot commented Mar 11, 2025

Copy link
Copy Markdown

Comment thread packages/x-date-pickers/src/internals/components/PickerProvider.tsx Outdated
@github-actions github-actions Bot added the PR: out-of-date The pull request has merge conflicts and can't be merged. label Mar 12, 2025
@github-actions

Copy link
Copy Markdown
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions Bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged. label Mar 12, 2025
}

export interface PickerPrivateContextValue {
/*

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I just moved the attributes around to sort them alphabetically. 🙈

@flaviendelangle flaviendelangle left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I would like a behavior double check of @noraleonte
But for me it make a lot of sense to enable it only on our MUI multi input field and only for the time range picker.

@noraleonte noraleonte left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The behavior makes a lot of sense to me 👌 Thanks for the effort on this one 🚀

@LukasTy
LukasTy merged commit f4365f5 into mui:master Mar 14, 2025
@LukasTy
LukasTy deleted the alternate-trp-trigger-ref branch March 14, 2025 10:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

plan: Pro Impact at least one Pro user. scope: pickers Changes related to the date/time pickers. type: enhancement It’s an improvement, but we can’t make up our mind whether it's a bug fix or a new feature.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants