- Issue created by @jrockowitz
- last update
over 1 year ago 534 pass - @jrockowitz opened merge request.
- Status changed to Needs review
over 1 year ago 4:30pm 3 July 2023 - 🇮🇳India kasli_harshit
Hi @jrockowitz , I tried to reproduce this issue :
Steps to reproduce :
1.Add a text field (any field) to webform.
2.Add a date list field to webform ( chose the option that you want to enable i.e: d ,m ,y).
3.make the date list field required on the conditional basis ( i made the date list field required if the text field has a value).
4 Now fill the value in the name , don't fill anything in the date-time field even if it is required and submit the form.issue : instead of showing the inline the error(form validation ) for the required date time field the page will load on submission and than we will see the error for the date field required.
I have applied the patch , applied successfully now after submitting the form with the the date-field empty, we are getting the inline error and preventing the page load which is the ideal behaviour similar to what we have incase of normal date-time required field.
Attaching the screenshot for the same. we can move this to RTBC+1.
Thanks. - Status changed to RTBC
over 1 year ago 3:59pm 12 September 2023 - Status changed to Fixed
over 1 year ago 1:47pm 14 September 2023 -
jrockowitz →
authored 19b2bc92 on 6.1.x
Issue #3372030 by jrockowitz: Datelist element does not use clientside...
-
jrockowitz →
authored 19b2bc92 on 6.1.x
-
jrockowitz →
authored 19b2bc92 on 6.x
Issue #3372030 by jrockowitz: Datelist element does not use clientside...
-
jrockowitz →
authored 19b2bc92 on 6.x
-
jrockowitz →
authored 19b2bc92 on 6.2.x
Issue #3372030 by jrockowitz: Datelist element does not use clientside...
-
jrockowitz →
authored 19b2bc92 on 6.2.x
Automatically closed - issue fixed for 2 weeks with no activity.