-
Notifications
You must be signed in to change notification settings - Fork 285
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
Date Picker Constraint Not Enforced on 'Ok' Button Click. #2508
Comments
@MJ1998 Please check if I have missed anything to add here. Thanks. |
I would say disable the calendar popup itself when there is no valid date possible + show an error that there is no valid date. |
Okay, this will be for the case when the min and max dates are overlapping, right? Also I think going with 2nd solution to solve this issue would be better as you had suggested because this will avoid increasing the number of clicks. |
Yes right. I suggested this |
Okay got it. |
personally not too keen on this - it's such a niche case and would probably mean the questionnaire author needs to do something about it - if the behaviour of our widget is consistent (i guess you would not be able to input any date ultimately) i'm happy for us to not handle this case additionally. |
Describe the bug
In the date picker if there is some constraint which does not allow to select the current date and the user still clicks on "OK" button without selecting any date it still selects the current date as the answer as it is the default to it.
Screen_recording_20240404_153456.webm
Component
SDC Gallery app
To Reproduce
Steps to reproduce the behavior:
Expected behavior
There can be two approaches to solving this:
Screenshots
If applicable, add screenshots to help explain your problem.
Smartphone (please complete the following information):
Additional context
Add any other context about the problem here.
Would you like to work on the issue?
Yes.
The text was updated successfully, but these errors were encountered: