Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix #1661: Calendar do not navigate past min and max #2771

Merged
merged 4 commits into from
Jul 1, 2022

Conversation

melloware
Copy link
Member

###Defect Fixes
Fix #1661: Calendar do not navigate past min and max

@mertsincan this actually is much easier now that you switched to hooks because I can put this code in one place when the overlay panel renders and update the navigators appropriately without and additional property.

@melloware melloware added the Status: Pending Review Issue or pull request is being reviewed by Core Team label Apr 19, 2022
@mertsincan
Copy link
Member

@melloware, Thanks a lot for this PR! You need to add a property to Calendar for this. We should not break the current default behavior. This feature is optional.

@melloware
Copy link
Member Author

I picked showOutliers as the property as I could not think of a better name for what we are allowing to be displayed.

@melloware melloware added Resolution: Needs Revision The pull request can't be merged. Conflicts need to be corrected or documentation and code updated. and removed Status: Pending Review Issue or pull request is being reviewed by Core Team labels May 9, 2022
@melloware melloware added Status: Pending Review Issue or pull request is being reviewed by Core Team and removed Resolution: Needs Revision The pull request can't be merged. Conflicts need to be corrected or documentation and code updated. labels Jun 30, 2022
@mertsincan
Copy link
Member

Maybe, its name can be 'showMinMaxRange' and the default value is 'false' to be compatible with other Prime* libraries. WDYT?

@melloware
Copy link
Member Author

OK updated to showMinMaxRange and defaulted to false.

@melloware melloware merged commit 2251a6b into primefaces:master Jul 1, 2022
@melloware melloware deleted the PR1661-2 branch July 1, 2022 12:20
@mertsincan
Copy link
Member

Thanks @melloware ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Pending Review Issue or pull request is being reviewed by Core Team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

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