8000 [Form] Renamed the option "empty_value" to "placeholder" by webmozart · Pull Request #12003 · symfony/symfony · GitHub
[go: up one dir, main page]

Skip to content

[Form] Renamed the option "empty_value" to "placeholder" #12003

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 1 commit into from
Sep 23, 2014

Conversation

webmozart
Copy link
Contributor
Q A
Bug fix? no
New feature? yes
BC breaks? no
Deprecations? yes
Tests pass? yes
Fixed tickets #5791
License MIT
Doc PR TODO

This PR is changing the "empty_value" option to the more understandable name "placeholder".

In a subsequent PR, the "placeholder" option should also be added to all types that support the "placeholder" HTML5 attribute.

@Tobion
Copy link
Contributor
Tobion commented Sep 23, 2014

👍

@fabpot
Copy link
Member
fabpot commented Sep 23, 2014

@webmozart Can you create an issue for the docs?

@fabpot
Copy link
Member
fabpot commented Sep 23, 2014

Thank you @webmozart.

@fabpot fabpot merged commit 2b440f3 into symfony:master Sep 23, 2014
fabpot added a commit that referenced this pull request Sep 23, 2014
…r" (webmozart)

This PR was merged into the 2.6-dev branch.

Discussion
----------

[Form] Renamed the option "empty_value" to "placeholder"

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | yes
| BC breaks?    | no
| Deprecations? | yes
| Tests pass?   | yes
| Fixed tickets | #5791
| License       | MIT
| Doc PR        | TODO

This PR is changing the "empty_value" option to the more understandable name "placeholder".

In a subsequent PR, the "placeholder" option should also be added to all types that support the "placeholder" HTML5 attribute.

Commits
-------

2b440f3 [Form] Renamed the option "empty_value" to "placeholder"
@webmozart webmozart deleted the issue6475 branch October 22, 2014 17:21
@wouterj
Copy link
Member
wouterj commented Nov 18, 2014

Why doesn't text fields use this option? (it's quite usefull)

weaverryan added a commit to symfony/symfony-docs that referenced this pull request Nov 24, 2014
This PR was merged into the 2.6 branch.

Discussion
----------

Renamed empty_value to placeholder

| Q   | A
| --- | ---
| Doc fix? | no
| New docs? | yes (symfony/symfony#12003)
| Applies to | 2.6+
| Fixed tickets | #4261

Commits
-------

54c44ea Renamed empty_value to placeholder
fabpot added a commit that referenced this pull request Feb 14, 2016
This PR was merged into the 3.0 branch.

Discussion
----------

[Form] remove deprecated empty_value_in_choices

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | tiny
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | -
| License       | MIT
| Doc PR        | -

This has been deprecated since 2.6 in #12003. One part has already been removed  (`empty_value`) but this part has been forgotten.

Commits
-------

735f92e [Form] remove deprecated empty_value_in_choices
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants
0