Skip to content

Commit 41fa537

Browse files
Fix dispatching 2 onChange events with variant="inline" (mui#1209)
* Fix dispatching 2 onChange events with variant="inline" * Update lib/src/_shared/hooks/usePickerState.ts * Make autoOk property to have bigger prioritiy
1 parent 42b2cad commit 41fa537

File tree

1 file changed

+8
-5
lines changed

1 file changed

+8
-5
lines changed

lib/src/_shared/hooks/usePickerState.ts

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -75,16 +75,19 @@ export function usePickerState(props: BasePickerProps, options: StateHookOptions
7575
onChange: (newDate: MaterialUiPickersDate, isFinish = true) => {
7676
setPickerDate(newDate);
7777

78-
if (variant === 'inline' || variant === 'static') {
79-
onChange(newDate);
80-
}
81-
8278
if (isFinish && autoOk) {
8379
acceptDate(newDate);
80+
return;
81+
}
82+
83+
// simulate autoOk, but do not close the modal
84+
if (variant === 'inline' || variant === 'static') {
85+
onChange(newDate);
86+
onAccept && onAccept(newDate);
8487
}
8588
},
8689
}),
87-
[acceptDate, autoOk, onChange, pickerDate, variant]
90+
[acceptDate, autoOk, onAccept, onChange, pickerDate, variant]
8891
);
8992

9093
const validationError = validate(value, utils, props);

0 commit comments

Comments
 (0)