#9427 closed defect (bug) (wontfix)
Output Broken: Invalid Output (Error) on Dashboard: autocomplete and (Warning but more serious) name/id conflict
| Reported by: | hakre | Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | |
| Component: | General | Version: | |
| Severity: | normal | Keywords: | has-patch |
| Cc: | Focuses: |
Description
Dashboard output (admin backend) has at least an error. patched it.
Attachments (1)
Change History (7)
#1
follow-up:
↓ 3
@
17 years ago
- Resolution → wontfix
- Status new → closed
Perhaps the autocomplete="off" should go in the <form> element. It may not be accepted by the W3C but is the de-facto standard supported by almost all browsers.
Having name != id is not a violation of any standard.
#3
in reply to: ↑ 1
;
follow-up:
↓ 4
@
17 years ago
- Cc added
- Milestone → 2.9
- Resolution wontfix
- Status closed → reopened
Replying to azaozz:
Perhaps the autocomplete="off" should go in the <form> element. It may not be accepted by the W3C but is the de-facto standard supported by almost all browsers.
Leaving invalid (X)HTML in core WordPress code is unacceptable, IMO. It means that plugin and theme developers, among others, can no longer use the W3C validator to check that their plugin/theme/etc isn't producing invalid (X)HTML at a glance: they instead have to look or parse through the validation results to see if the reported issues are due to core WordPress files or not. What a pain. Consequently, I'm reopening this ticket.
By the way, in WordPress 2.8.6 the dashboard now additionally contains the following invalid markup:
<textarea class="comment" rows="3" cols="10">Hi, this is a comment.<br />To delete a comment, just log in and view the post's comments. There you will have the option to edit or delete them.</textarea>
That <br /> isn't permitted in <textarea>.
#4
in reply to: ↑ 3
;
follow-up:
↓ 5
@
17 years ago
- Resolution → wontfix
- Status reopened → closed
Replying to sampablokuper:
It seems it's part of the things to come: http://www.whatwg.org/specs/web-apps/current-work/multipage/forms.html#attr-form-autocomplete
#5
in reply to: ↑ 4
@
17 years ago
Replying to azaozz:
Replying to sampablokuper:
It seems it's part of the things to come: http://www.whatwg.org/specs/web-apps/current-work/multipage/forms.html#attr-form-autocomplete
If/when WordPress starts using the HTML 5 doctype by default, then fine, use @autocomplete . In the meantime, it's invalid and needs patching.
Also, what about the <br /> I mentioned above?
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
dashboard errors