- 🇺🇸United States kthull Fort Wayne, Indiana
Patch in #3 🐛 Placeholder doesn't show Needs work no longer applies to either the 1.x-dev version or the D10-ready 1.3 version
- Status changed to Needs work
over 1 year ago 11:57am 3 May 2023 - Status changed to Fixed
10 months ago 12:15pm 28 February 2024 - 🇷🇪Réunion morgannc La Réunion
The logic is wrong. Placeholder attributes are not allowed on date fields. Today, the latest version of the module allows placeholders to be displayed on inputs, but this goes against current HTML rules :
https://html.spec.whatwg.org/multipage/input.html#date-state-(type%3Ddate)
The following content attributes must not be specified and do not apply to the element: accept, alt, checked, dirname, formaction, formenctype, formmethod, formnovalidate, formtarget, height, maxlength, minlength, multiple, pattern, placeholder, popovertarget, popovertargetaction, size, src, and width.
If you need to display pre-defined text or text on top of your element, turn to other solutions such as CSS, or even consider a text field (note that this may have repercussions in terms of ux/ui and values returned).
- 🇷🇪Réunion morgannc La Réunion
I create other issue to adding placeholder support with text field modifier : https://www.drupal.org/project/date_popup/issues/3424371 ✨ Adding placeholder support Needs review
But this is on the news version of module 2.x series. Automatically closed - issue fixed for 2 weeks with no activity.