Skip to content

[Reference][Forms] remove the label_attr option which is not available in the button type #3651

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
Mar 19, 2014

Conversation

xabbuh
Copy link
Member

@xabbuh xabbuh commented Mar 8, 2014

Q A
Doc fix? yes
New docs? no
Applies to all
Fixed tickets

The label_attr is neither defined in the ButtonType class nor in the BaseType class, but in the FormType class instead. Thus, it can't be used for the button type.

@weaverryan
Copy link
Member

👍 makes perfect sense!

weaverryan added a commit that referenced this pull request Mar 19, 2014
…not available in the button type (xabbuh)

This PR was merged into the 2.3 branch.

Discussion
----------

[Reference][Forms] remove the label_attr option which is not available in the button type

| Q             | A
| ------------- | ---
| Doc fix?      | yes
| New docs?     | no
| Applies to    | all
| Fixed tickets |

The ``label_attr`` is neither defined in the ``ButtonType`` class nor in the ``BaseType`` class, but in the ``FormType`` class instead. Thus, it can't be used for the ``button`` type.

Commits
-------

5bafc3c remove the label_attr option which is not available in the button type
@weaverryan weaverryan merged commit 5bafc3c into symfony:2.3 Mar 19, 2014
@xabbuh xabbuh deleted the remove-button-label-attr branch March 19, 2014 07:46
@happypoulp
Copy link

The label_attr is still appearing in the documentation for the button type:

http://symfony.com/fr/doc/current/reference/forms/types/button.html

@xabbuh
Copy link
Member Author

xabbuh commented Jun 27, 2014

@happypoulp You are looking at the French translation which has its own repository. Seems not to be up-to-date.

@wouterj
Copy link
Member

wouterj commented Jun 27, 2014

@happypoulp you should report it to https://github.com/symfony-fr/symfony-docs-fr/ And I think the french translator will be even happier if you submit a PR to that repo, removing the same things as @xabbuh did here.

@happypoulp
Copy link

Good to know, I'll do that, thanks!

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.

4 participants