Opened 18 years ago
Closed 17 years ago
#2907 closed defect (wontfix)
[patch] Add <tbody> ... </tbody> in the table in edit_inline_tabular.html admin file
| Reported by: | anonymous | Owned by: | Adrian Holovaty |
|---|---|---|---|
| Component: | contrib.admin | Version: | dev |
| Severity: | trivial | Keywords: | html |
| Cc: | Triage Stage: | Ready for checkin | |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
in
root/django/trunk/django/contrib/admin/templates/admin/edit_inline_tabular.html
add <tbody> ... </tbody> in the table
Attachments (1)
Change History (4)
by , 17 years ago
| Attachment: | add_tbody_patch.diff added |
|---|
comment:1 by , 17 years ago
| Summary: | Add <tbody> ... </tbody> in the table in edit_inline_tabular.html admin file → [patch] Add <tbody> ... </tbody> in the table in edit_inline_tabular.html admin file |
|---|
comment:2 by , 17 years ago
| Triage Stage: | Unreviewed → Ready for checkin |
|---|
comment:3 by , 17 years ago
| Resolution: | → wontfix |
|---|---|
| Status: | new → closed |
The tbody for tables element is optional in both HTML 4 (note that it is marked as having "O" for both the start and end tag requirements in section 11.2.3 of the spec) and XHTML. So I think we can leave this patch out; it doesn't hurt anything to omit it and it makes it easier to keep newforms-admin and trunk admin in sync.
Note:
See TracTickets
for help on using tickets.
Accordingly to w3c specification the table need <tbody>, so I added.