Opened 18 years ago

Closed 18 years ago

Last modified 18 years ago

#1102 closed defect (fixed)

[patch] a few HTML errors in the admin app

Reported by: L.Plant.98@… Owned by: Adrian Holovaty
Component: contrib.admin Version:
Severity: normal Keywords:
Cc: Triage Stage: Unreviewed
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

This patch addresses 3 HTML errors in the admin (as of rev 1764, trunk).

Attachments (1)

django_admin_html_patch_1.diff​ (1.7 KB ) - added by L.Plant.98@… 18 years ago.
patch

Download all attachments as: .zip

Change History (4)

by L.Plant.98@…, 18 years ago

Attachment: django_admin_html_patch_1.diff​ added

patch

comment:1 by Adrian Holovaty, 18 years ago

Thanks for this patch. What's the extra <div> for in the search_form.html template?

comment:2 by L.Plant.98@…, 18 years ago

Apparently <form> can only contain block or script elements:
​http://www.w3.org/TR/REC-html40/interact/forms.html#edef-FORM

so you can't put an <input> or <label> inside <form> directly, but adding a div fixes it.

comment:3 by Adrian Holovaty, 18 years ago

Resolution: → fixed
Status: new → closed

(In [1791]) Fixed #1102 -- Fixed some HTML validation errors in admin templates. Thanks, Luke Plant

Note: See TracTickets for help on using tickets.
Back to Top