Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
"Empty" state (also "filled" as it is derivative from "empty") doesn't work for date field with datepicker plugin, becuase "empty" state has keyup trigger only. But datepicker doesn't invoke keyup, only change only.
So I think we need to add change trigger to "empty" state as it has done for state "value".
Comment | File | Size | Author |
---|---|---|---|
#26 | 2880338-26.patch | 552 bytes | ovanes |
#25 | 2880338-nr-bot.txt | 144 bytes | needs-review-queue-bot |
#20 | 2880338-20.patch | 862 bytes | kostyashupenko |
#17 | drupal-datepicker_state_empty-2880338-17.patch | 965 bytes | SpadXIII |
#2 | drupal-datepicker_state_empty-2880338-2.patch | 343 bytes | kalistos |
Comments
Comment #2
kalistos CreditAttribution: kalistos at Adyax commentedComment #3
kalistos CreditAttribution: kalistos at Adyax commentedBackport to 8.3
Comment #4
andypostComment #5
andypostComment #6
mpdonadio@kalistos, are you seeing this with the native datepicker (eg, Chrome) or the jQuery UI polyfill (eg, Firefox). TBH, I am not sure which JSTB will use off the top of my head, but the other will require manual testing. For the time being, let's just work on the 8.4 version, and then backport when we are done (been seeing confusion on other issues where people post 8.3 and 8.4 intermixed).
Comment #7
kalistos CreditAttribution: kalistos at Adyax commented@mpdonadio, I have this issue with jQuery UI datepicker, which integrated in Drupal core:
core/assets/vendor/jquery.ui/ui/datepicker-min.js
Comment #14
nwoodland CreditAttribution: nwoodland commentedThe patch in #3 worked great resolving this for a datetime field in Drupal 8.8.1. Thanks kalistos!
Comment #15
tanubansal CreditAttribution: tanubansal at Salsa Digital commentedPatch #3 works for 9.1 as well.
Tested
Comment #17
SpadXIII CreditAttribution: SpadXIII at SIM commentedQuick re-roll for 8.9.9
Seems to work fine functionally :)
edit: looks like I forgot to change the version for the test-bot .. hmm
Comment #19
andypostComment #20
kostyashupenkoComment #21
colin.eininger CreditAttribution: colin.eininger commentedI don't know how to handle that correctly but I think this issue will fix #2445271 too
Comment #25
needs-review-queue-bot CreditAttribution: needs-review-queue-bot as a volunteer commentedThe Needs Review Queue Bot tested this issue. It either no longer applies to Drupal core, or fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".
Apart from a re-roll or rebase, this issue may need more work to address feedback in the issue or MR comments. To progress an issue, incorporate this feedback as part of the process of updating the issue. This helps other contributors to know what is outstanding.
Consult the Drupal Contributor Guide to find step-by-step guides for working with issues.
Comment #26
ovanes CreditAttribution: ovanes as a volunteer and at FFW commentedHello I have rerolled the patch to work for 9.5.7 version.