#4512 closed (invalid)
<ul class="errorlist"> must be inside <p> on forms generated by as_p()
| Reported by: | Owned by: | Adrian Holovaty | |
|---|---|---|---|
| Component: | Forms | Version: | dev |
| Severity: | Keywords: | ||
| Cc: | Triage Stage: | Unreviewed | |
| Has patch: | no | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
This way will be easy to keep the error next to the field, in complex form desings. Example: two columns, with fields side by side.
Change History (3)
comment:1 by , 17 years ago
comment:2 by , 17 years ago
| Resolution: | → invalid |
|---|---|
| Status: | new → closed |
Invalid, for the reasons Matt said. Just because it's already wrong doesn't mean we should introduce more wrongness. :)
Note:
See TracTickets
for help on using tickets.
I was going to say that doing this would result in invalid HTML (Paragraphs cannot contain other block-level elements), but apparently as_p() already puts the error list <ul> within a <p>. So instead, I'll say that it would result in even more invalid HTML.