Ticket #2411 (closed Bug: fixed)

Opened 11 months ago

Last modified 11 months ago

Anchors are not created after #2263

Reported by: alfonsoml Owned by: martinkou
Priority: High Milestone: FCKeditor 2.6.3
Component: General Version: FCKeditor 2.6.3 Beta
Keywords: Confirmed Review+ Cc:

Description

Reported in http://www.fckeditor.net/forums/viewtopic.php?f=6&t=10656

I've checked that Firefox, Opera and Safari fail to create anchors since #2263 was fixed http://rev.fckeditor.net/fckeditor/trunk/2138/_samples/ works and http://rev.fckeditor.net/fckeditor/trunk/2139/_samples/ fails

No javascript errors are generated, the <a> is created but it lacks the name attribute.

Attachments

2411.patch (1.3 KB) - added by martinkou 11 months ago.
2411_2.patch (2.1 KB) - added by martinkou 11 months ago.

Change History

Changed 11 months ago by martinkou

  • owner set to martinkou
  • status changed from new to assigned

Changed 11 months ago by martinkou

Changed 11 months ago by martinkou

  • keywords Review? added

Please test for #2263 in IE with non-empty anchors as well during review.

Changed 11 months ago by fredck

  • keywords Review- added; Review? removed

The fix looks correct, but it seems that reverting the first part of [2139] (the additions from line 124 to 130) is also correct.

Changed 11 months ago by alfonsoml

I haven't tested if it's really a bug , but maybe it's related (due to the "name" attribute). Check this report: http://www.fckeditor.net/forums/viewtopic.php?f=6&t=10674

Changed 11 months ago by martinkou

Changed 11 months ago by martinkou

  • keywords Review? added; Review- removed

Changed 11 months ago by fredck

  • keywords Review+ added; Review? removed

Changed 11 months ago by martinkou

  • status changed from assigned to closed
  • resolution set to fixed

Fixed with [2318].

Click here for more info about our SVN system.

Note: See TracTickets for help on using tickets.