#25481 closed Cleanup/optimization (fixed)
Documentation: Add field.help_text to looping over a form's fields
| Reported by: | Wim Feijen | Owned by: | John Moses |
|---|---|---|---|
| Component: | Documentation | Version: | dev |
| Severity: | Normal | Keywords: | |
| Cc: | Triage Stage: | Accepted | |
| Has patch: | no | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | yes | UI/UX: | no |
Description (last modified by )
On https://docs.djangoproject.com/en/dev/topics/forms/#looping-over-the-form-s-fields
The example of looping over the form's fields omits {{ field.help_text }}, though this field is outputted by default by {{ form.as_ul }}. I believe we should add {{ field.help_text }} in the example to be both consistent and complete.
From the docs:
---
Looping over the form’s fields
If you’re using the same HTML for each of your form fields, you can reduce duplicate code by looping through each field in turn using a {% for %} loop:
{% for field in form %}
<div class="fieldWrapper">
{{ field.errors }}
{{ field.label_tag }} {{ field }}
</div>
{% endfor %}
Change History (6)
comment:1 by , 10 years ago
| Description: | modified (diff) |
|---|---|
| Triage Stage: | Unreviewed → Accepted |
| Type: | Uncategorized → Cleanup/optimization |
comment:2 by , 10 years ago
| Owner: | changed from to |
|---|---|
| Status: | new → assigned |
comment:3 by , 10 years ago
Updated documentation and submitted pull request: https://github.com/django/django/pull/5383
Please let me know if am I missing anything.
Thanks.
It seems there are many other examples on that page that could also be updated similarly. Did you single this one out for any reason? I don't know that it's really necessary, but I don't have a great argument against it besides it will be a bit repetitive in all the examples, especially if the implementation is something like
{% if field.help_text %}<p>{{ field.help_text }}</p>{% endif %}so we don't output empty paragraph elements.I removed the "In addition, but unrelated" issue from the description as I don't think there's a need to complicate the example with a particular HTML structure. ("
{{ field.errors }}might need a span for the example to be usable in real life."){{ field.errors }}already includes a<ul>which can be styled, correct?