Ticket #3581 (closed Bug: fixed)

Opened 3 years ago

Last modified 3 years ago

name attribute on anchors is duplicated

Reported by: alfonsoml Owned by: arczi
Priority: Normal Milestone: CKEditor 3.0
Component: General Version: SVN (CKEditor) - OLD
Keywords: Confirmed Review+ Cc:

Description (last modified by alfonsoml) (diff)

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

3581.patch Download (676 bytes) - added by arczi 3 years ago.

Change History

comment:1 Changed 3 years ago by alfonsoml

  • Description modified (diff)

comment:2 Changed 3 years ago by arczi

  • Keywords Confirmed added
  • Milestone set to CKEditor 3.0

comment:3 Changed 3 years ago by fredck

  • Owner set to arczi

Changed 3 years ago by arczi

comment:4 Changed 3 years ago by arczi

  • Keywords Review? added

The same problem was with images.

comment:5 follow-up: ↓ 6 Changed 3 years ago by alfonsoml

  • 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 in reply to: ↑ 5 Changed 3 years ago by arczi

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 3 years ago by alfonsoml

  • Keywords Review+ added; Review- removed

Right, even if the interface doesn't allow to change it, the code should work correctly

comment:8 Changed 3 years ago by arczi

  • Status changed from new to closed
  • Resolution set to fixed

Fixed with [3598]

Note: See TracTickets for help on using tickets.
© 2003 – 2011 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy