Skip to content

Update forms.rst #3548

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 3 commits into from
Feb 16, 2014
Merged

Update forms.rst #3548

merged 3 commits into from
Feb 16, 2014

Conversation

mb3rnard
Copy link

@mb3rnard mb3rnard commented Feb 6, 2014

Hi there,
I thought it would be nice to specify that you can set a label to false to disable it. I needed it for a project, couldn't find it in the doc, and had to search through a couple of stackoverflow posts to find the solution.

Hi there, 
I thought it would be nice to specify that you can set a label to false to disable it. I needed it for a project, couldn't find it in the doc, and had to search through a couple of stackoverflow posts to find the solution.
@@ -645,7 +645,7 @@ the documentation for each type.
))

The label for a field can also be set in the template rendering the
form, see below.
form, see below. If you don't need a label associated to your input, you can disable it by setting it's value to 'false'.
Copy link
Member

Choose a reason for hiding this comment

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

it's -> its

Copy link
Member

Choose a reason for hiding this comment

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

and should put false into literals

Copy link
Member

Choose a reason for hiding this comment

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

lastly, don't forget to wrap lines after the first word that crosses the 72nd character:

form, see below. If you don't need a label associated to your input, you
can disable it by setting it's value to 'false'.

not sure what you meant about the wrapping, I applied a soft wrap after "to your input,".
Let me know how that looks
@@ -645,7 +645,7 @@ the documentation for each type.
))

The label for a field can also be set in the template rendering the
form, see below.
form, see below. If you don't need a label associated to your input, you can disable it by setting its value to false.
Copy link
Member

Choose a reason for hiding this comment

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

you forgot to wrap it after the first word that crosses the 72th character (after input,, add a line break)

And false should be in a literal (double backticks)

@mb3rnard
Copy link
Author

mb3rnard commented Feb 6, 2014

got that, cheers

@wouterj
Copy link
Member

wouterj commented Feb 6, 2014

thank you! 👍

@weaverryan
Copy link
Member

Really nice note - thanks for making a PR on this after you figured it out - that's exactly what we love to see! Cheers!

weaverryan added a commit that referenced this pull request Feb 16, 2014
This PR was merged into the 2.3 branch.

Discussion
----------

Update forms.rst

Hi there,
I thought it would be nice to specify that you can set a label to false to disable it. I needed it for a project, couldn't find it in the doc, and had to search through a couple of stackoverflow posts to find the solution.

Commits
-------

fb5f12d Update forms.rst
3116b39 Update forms.rst
95b4d40 Update forms.rst
@weaverryan weaverryan merged commit fb5f12d into symfony:2.3 Feb 16, 2014
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