Ticket #7911 (closed New Feature: fixed)

Opened 3 years ago

Last modified 3 years ago

Unable to remove anchor

Reported by: RichardD Owned by: Saare
Priority: Normal Milestone: CKEditor 3.6.1
Component: UI : Context Menu Version: 3.6
Keywords: Cc:

Description

In the FCKeditor, there was a context-menu option to remove an anchor. In the CKEditor, this option does not exist.

Attempting to remove the anchor by clearing the anchor name displays the error, "Please type the anchor name".

As a result, the only way to remove an anchor is to edit the source.

Attachments

7911.patch (2.6 KB) - added by Saare 3 years ago.
7911_2.patch (2.6 KB) - added by Saare 3 years ago.

Change History

comment:1 Changed 3 years ago by fredck

  • Keywords remove anchor removed
  • Status changed from new to confirmed
  • Milestone set to CKEditor 3.6.1

Up to CKEditor 3.6, it was possible to create collapsed anchors only, which were represented by icons only. To delete them, it was enough to simple delete those icons.

As we've added the ability to create anchors with contents, we're now in fact missing a way to remove them. Nice catch!

Changed 3 years ago by Saare

comment:2 Changed 3 years ago by Saare

  • Status changed from confirmed to review
  • Owner set to Saare

comment:3 Changed 3 years ago by fredck

  • Status changed from review to review_failed

In Firefox, it's not possible to remove collapsed anchors.

Additionally, there is no need to use CKEDITOR.tools.extend() at line 161. It just makes the code bigger and more complicated. Simply assigning the "menu" object properties is enough.

Changed 3 years ago by Saare

comment:4 Changed 3 years ago by Saare

  • Status changed from review_failed to review

comment:5 Changed 3 years ago by fredck

  • Status changed from review to review_passed
  • Type changed from Bug to New Feature

comment:6 Changed 3 years ago by Saare

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

Fixed with [6980].

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