Problem/Motivation
It makes sense that when the 'All Day' checkbox is checked, the time portion of the date/time value of a Date text field is ignored. I appreciate much that a help text is shown to let the user know about this ignoration.
I feel that the current help text 'If present, the time portion of the date string will be ignored when this option is checked.' is difficult to interpret.
Steps to reproduce
- In a 'Manage fields' page of a content type, do the followings:
- Label: type "Test Date"
- Field type: select 'Date'
- Widget: select 'Text field'
- Click the Save button.
- In the 'Field settings' page that follows, accept the default settings and click the "Save field settings" button.
- In the content type settings page that follows, click the fieldset 'More settings and values' to expand it.
- Check the checkbox 'Display all day checkbox'.
- Click the 'Save settings' button.
- Go to the add content page of the content type.
- The help text 'If present, the time portion of the date string will be ignored when this option is checked.' is displayed for 'All Day' checkbox of the newly created Date field.
Proposed resolution
I propose to use the statement 'When this option is checked, the time portion of the date/time value will be ignored.' as help text.
| Comment | File | Size | Author |
|---|---|---|---|
| #12 | date-n3301664-12.patch | 682 bytes | damienmckenna |
Issue fork date-3301664
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
Comments
Comment #2
damienmckennaComment #5
immaculatexavier commentedComment #6
damienmckennaThis will need to be rerolled as #3300058 was committed.
Comment #7
joelpittetMoving to Needs review to see if I can trick the test runner to pickup the new commits. 🤞
Comment #8
franceslui commentedHi Damien,
Could you help fix the failed test issues and rerun all tests?
All immaculatexavier and I did was to replace a help text string with another string. So, we did not change the logic of any code and all tests should pass theoretically.
I appreciate your time and thank you for your support!
Frances
Comment #9
joelpittetI’m attempting updating the forks 7.x-3.x branch to see if that helps…
Comment #12
damienmckennaIn patch format.
Comment #14
damienmckennaCommitted. Thanks everyone.