Opened 16 years ago
Closed 16 years ago
#3581 closed Bug (fixed)
name attribute on anchors is duplicated
Reported by: | Alfonso Martínez de Lizarrondo | Owned by: | Artur Formella |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 3.0 |
Component: | General | Version: | SVN (CKEditor) - OLD |
Keywords: | Confirmed Review+ | Cc: |
Description (last modified by )
Load CKEditor and insert an anchor. Put any name on it.
Switch to source mode, everything looks OK.
<p> This<a name="test"></a> is some <strong>sample text</strong>. You are using <a href="http://www.fckeditor.net/">CKEditor</a>.</p>
Switch to design and back to source, the anchor now has two name attributes
<p> This<a name="test" name="test"></a> is some <strong>sample text</strong>. You are using <a href="http://www.fckeditor.net/">CKEditor</a>.</p>
Attachments (1)
Change History (9)
comment:1 Changed 16 years ago by
Description: | modified (diff) |
---|
comment:2 Changed 16 years ago by
Keywords: | Confirmed added |
---|---|
Milestone: | → CKEditor 3.0 |
comment:3 Changed 16 years ago by
Owner: | set to Artur Formella |
---|
Changed 16 years ago by
Attachment: | 3581.patch added |
---|
comment:4 Changed 16 years ago by
Keywords: | Review? added |
---|
comment:5 follow-up: 6 Changed 16 years ago by
Keywords: | Review- added; Review? removed |
---|
The fix for links is ok, but I don't see how it's possible to add a name attribute to the images.
I don't even remember if it's valid or people should use only IDs for images, and use name only for form elements and anchors as legacy (they should be IDs for compatibility with xhtml 1.1)
comment:6 Changed 16 years ago by
I have found "name" on http://www.w3.org/TR/REC-html40/struct/objects.html
Please insert in source mode:
<p> <img name="test" /> </p>
Switch do WYSIWYG and back to Source. Result:
<p> <img name="test" name="test" /> </p>
It is not the expected behavior.
comment:7 Changed 16 years ago by
Keywords: | Review+ added; Review- removed |
---|
Right, even if the interface doesn't allow to change it, the code should work correctly
The same problem was with images.