- Issue created by @trevorkjorlien
- 🇭🇺Hungary nagy.balint
Originally the webform component was based on the webform date component, but it had a lot of unnecessary fields and settings.
I was unaware of that feature, I will have to check what can be done. - 🇨🇦Canada trevorkjorlien
Tested the patch, and it now lets me add multiple or unlimited dates. Thanks!
However, some useful things were removed in a previous release. For instance, in the 'Advanced' tab on the Webform Flatpickr Date field, there are no longer 'Multiple settings' options. Screenshots attached.
At least for my use, it doesn't let me turn off sorting, add/remove elements, etc...
I recommend we add those settings back in.
- 🇭🇺Hungary nagy.balint
Not sure if the following patch is enough, but we will see.
- 🇮🇳India divya.sejekan
Applied add-multiple-3530184-5.patch , Patch gets applied cleanly and Multiple option is enabled.
But i could not see any changes in Advanced section tab.
Testing steps :
1. Install Webform and Datetime Flatpickr module
2. Create or edit an Webform
3. Add a Flatpickr Date element
4. Under the 'Title' field, see that there is no 'Allowed number of values' dropdown as there was beforeAs per the issue description and title the issue is resolved . RTBC++
Keeping this in review for further review on code
- 🇨🇦Canada trevorkjorlien
Applied
add-multiple-3530184-5.patch
and can now see a 'Multiple settings' section under the 'Advanced' tab.Now I can enable and disable all those options that were there before.
For me, this is approved.
- 🇭🇺Hungary nagy.balint
Thanks!
Sorry #6, but that comment was a bit confusing to me so I cannot add that as a credit.
-
nagy.balint →
committed 9bbcdc80 on 3.0.x
Issue #3530184 by nagy.balint, trevorkjorlien: Unable to add more than 1...
-
nagy.balint →
committed 9bbcdc80 on 3.0.x
Automatically closed - issue fixed for 2 weeks with no activity.