Skip to content

[Reference][Form Types] Add missing "format" option to "datetime" field #3466

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Feb 1, 2014
Merged

[Reference][Form Types] Add missing "format" option to "datetime" field #3466

merged 2 commits into from
Feb 1, 2014

Conversation

bicpi
Copy link
Contributor

@bicpi bicpi commented Jan 11, 2014

Q A
Doc fix? no
New docs? yes
Applies to 2.3+
Fixed tickets #3410

the format of the input, i.e. how Symfony will interpret the given input
as a datetime string. It defaults to the `RFC 3339`_ format which is used
by the HTML5 ``datetime`` field. Keeping the default will render a field of
type ``datetime``.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is an important note, and I think we can be even more specific on what we mean:

Keeping the default value will cause the field to be rendered as an ``input`` field with ``type="datetime"``.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@weaverryan Adopted your proposal, sounds good, thanks. And it's really important to know when exactly you get thedatetime field type :-)

weaverryan added a commit that referenced this pull request Feb 1, 2014
…options

[Reference][Form Types] Add missing "format" option to "datetime" field
@weaverryan weaverryan merged commit 579215f into symfony:2.3 Feb 1, 2014
@weaverryan
Copy link
Member

👍 great work on this Philipp - I think it's tricky to describe just right :).

Thanks!

@bicpi bicpi deleted the add_missing_datetime_format_form_options branch February 1, 2014 17:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants