-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
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
Update forms.rst #3548
Conversation
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'. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it's
-> its
There was a problem hiding this comment.
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
There was a problem hiding this comment.
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. |
There was a problem hiding this comment.
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)
got that, cheers |
thank you! 👍 |
Really nice note - thanks for making a PR on this after you figured it out - that's exactly what we love to see! Cheers! |
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
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.