Ticket #4527 (closed Bug: fixed)
Checkbox command generates invalid HTML
| Reported by: | wwalc | Owned by: | garry.yao |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.1 |
| Component: | UI : Dialogs | Version: | SVN (CKEditor) - OLD |
| Keywords: | Confirmed Review+ | Cc: |
Description
According to W3 spec: http://www.w3.org/TR/html4/interact/forms.html#edef-INPUT (value of attribute "checked" cannot be "true"; must be one of "checked"), the following code is wrong:
<input checked="true" name="sdfs" type="checkbox" value="sdf" />
Instead of checked="true" we should generate checked="checked".
Attachments
Change History
comment:1 Changed 4 years ago by garry.yao
- Status changed from new to assigned
- Keywords Review? added
- Owner set to garry.yao
comment:2 Changed 4 years ago by garry.yao
- Version set to SVN (CKEditor)
- Component changed from General to UI : Dialogs
comment:3 Changed 4 years ago by fredck
- Keywords Review- added; Review? removed
Is the "default IE values" cleanup really needed? It looks more like a bug in the dialog code. It's strange that the defaults are not there when creating a new checkbox, just when editing it.
comment:4 Changed 3 years ago by garry.yao
- Keywords Review? added; Review- removed
Is the "default IE values" cleanup really needed?
- It turns out that the cleanup for 'value' is needed, IE's nodeValue for 'value' attr is always 'on', even worse we'll find it on html output as well.
- It's proved that 'checked' value cleanup is the right place to fix, it's was not long after I read the nodeName Hell article then it landed in our editor, but with a different scene that in our case is the nodeName of a AttrNode that hurt, instead of a Element - IE always return upper-case for 'CHECKED'.
Ticket Test added at :
http://ckeditor.t/tt/4527/1.html.
http://ckeditor.t/tt/4527/2.html.
Note: See
TracTickets for help on using
tickets.
