mantine icon indicating copy to clipboard operation
mantine copied to clipboard

DateInput `defaultDate` prop does not work when `value={null}`

Open tofu2323 opened this issue 2 years ago • 5 comments

What package has an issue

@mantine/dates

Describe the bug

The defaultDate prop does not work on DateInput with useForm

What version of @mantine/hooks page do you have in package.json?

6.0.13

If possible, please include a link to a codesandbox with the reproduced problem

https://codesandbox.io/s/objective-lichterman-4qy3ml?file=/src/App.tsx

Do you know how to fix the issue

No

Are you willing to participate in fixing this issue and create a pull request with the fix

None

Possible fix

No response

tofu2323 avatar Jun 20 '23 02:06 tofu2323

I'd like to take a stab at this if no one else is already working on it.

AdamGEmerson avatar Jun 23 '23 21:06 AdamGEmerson

@rtivital For DateInput what's the difference between defaultDate and defaultValue and between date and value.

@tofu2323 Why do you need defaultDate? Have you tried using the date in initialValues instead? Is it so that the field falls back to a specific date in case it has no value (or its value gets cleared)?

@AdamGEmerson Feel free to do so. :) I just had a quick look into the code but had difficulties understanding the meaning of the various props (see my question to rtivital).

cyantree avatar Jun 23 '23 21:06 cyantree

Also the docs say that defaultDate and defaultValue are meant for uncontrolled use which isn't the case with useForm(). https://mantine.dev/dates/date-input/?t=props https://mantine.dev/dates/date-picker-input/?t=props

cyantree avatar Jun 23 '23 22:06 cyantree

As @cyantree stated, the most likely issue is that these props are meant for uncontrolled input and useForm is controlled.

I also noticed playing with the example sandbox that the DatePicker component isn't "working" either. The defaultDate prop will continue to overwrite the input value even after form.getInputProps() is called with a valid Date set in initialValues.

See this example: Code Sandbox

One thing that might be worth addressing here is why the DateInput and DatePickerInput behave differently in this scenario. DatePickerInput responds with "Invalid Date", and DateInput just displays as empty. Is this expected behavior?

AdamGEmerson avatar Jun 25 '23 15:06 AdamGEmerson

@tofu2323 In my approach, I directly set the default date I need either in the value prop of the DateInput component or in the initialValues object when using useForm. This avoids setting the value to null first and then assigning the default date separately, which seems more straightforward and logical to me:

<DateInput value={new Date()} />
// or
const { register, handleSubmit, setValue } = useForm({
  initialValues: {
    date: new Date()
  }
});

rajivchaulagain avatar Jun 26 '24 16:06 rajivchaulagain