Calendar: shows future and/or past dates even when minDate and/or maxDate props are set

[acc-cassio]

Describe the bug

The Calendar component shows past and future dates even when minDate and maxDate are set.
Case in point: the "MinMax" example in the documentation. You can go past minDate and maxDate.

Apparently, this issue has been solved before, see issue #1661. However, the behaviour is still present in primereact's current version.

Reproducer

No response

System Information

PrimeReact version: 10.9.2

Steps to reproduce the behavior

1. Go to https://primereact.org/calendar/#minmax
2. Try to navigate to dates outside the min and max range
3. Dates are displayed

Expected behavior

Dates before minDate and after maxDate (whichever is present, if any) should not be shown.

[melloware]

There is a another property: https://primereact.org/calendar/#api.Calendar.props.showMinMaxRange

Some users wanted people to allow you to see older and future invalid dates but the showMinMaxRange property blocks you from going backwards or forwards.

[melloware]

Here is a stackblitz: https://stackblitz.com/edit/uvwu3ybx?file=src%2FApp.jsx

[acc-cassio]

Oh, I had tried this property and it didn't work, so I had assumed it had a different purpose. It works now!

Just one little thing, I think the documentation and/or name of the property should be adjusted:

"showMinMaxRange: Whether to allow navigation past min/max dates."

Because when you set the property to true, the navigation is NOT allowed, contrary to what indicated above

Any way, thank you a lot!

[melloware]

@acc-cassio do you have a recommendation for what the docs should say? I can make the change?

[melloware]

how about this

Controls whether navigation beyond minimum and maximum dates is restricted. When true, navigation is limited to the min/max date range.

[acc-cassio]

The suggestion sounds quite clear to me! Thank you!

[acc-cassio]

Ideally the name of the property should be changed too, right? But I guess this is more complicated due to backwards compatibility?

[acc-cassio]

I'd name it 'restrictMinMaxNav', to match the suggested description

[melloware]

yeah i hate renaming props after they exist though and PR uses a lot of showXXX so i proabbly named it poorly to begin with 😄