Opened 6 years ago

Closed 6 years ago

#7893 closed Bug (fixed)

Can't link to anchor

Reported by: Jakub Ś Owned by: Sa'ar Zac Elias
Priority: Normal Milestone: CKEditor 3.6.1
Component: General Version: 3.6
Keywords: WebKit Opera Cc: satya_minnekanti@…

Description

  1. Create an anchor
  2. Open link dialog
  3. Change Link Type to "Link to anchor in the text"

Result:No anchors are available

This issue has been reproducible in Webkit and opera from version 3.6 rev [6904]

Attachments (3)

7893.patch (858 bytes) - added by Sa'ar Zac Elias 6 years ago.
7893_2.patch (1023 bytes) - added by Sa'ar Zac Elias 6 years ago.
7893_3.patch (2.2 KB) - added by Sa'ar Zac Elias 6 years ago.

Download all attachments as: .zip

Change History (17)

comment:1 Changed 6 years ago by Jakub Ś

Status: newconfirmed

Changed 6 years ago by Sa'ar Zac Elias

Attachment: 7893.patch added

comment:2 Changed 6 years ago by Sa'ar Zac Elias

Keywords: WebKit Opera added
Milestone: CKEditor 3.6.1
Owner: set to Sa'ar Zac Elias
Status: confirmedreview

comment:3 Changed 6 years ago by Satya Minnekanti

Cc: satya_minnekanti@… added

comment:4 Changed 6 years ago by Frederico Caldeira Knabben

Status: reviewreview_passed

comment:5 Changed 6 years ago by Sa'ar Zac Elias

Resolution: fixed
Status: review_passedclosed

Fixed with [6960].

comment:6 Changed 6 years ago by Krzysztof Studnik

Resolution: fixed
Status: closedreopened

Issue still could be confirmed under IE6/7 CKEditor rev r7028

comment:7 Changed 6 years ago by Wiktor Walc

(Just in case - it starts working after switching to source mode and back).

comment:8 Changed 6 years ago by Sa'ar Zac Elias

Status: reopenedconfirmed

comment:9 Changed 6 years ago by Sa'ar Zac Elias

Status: confirmedassigned

Changed 6 years ago by Sa'ar Zac Elias

Attachment: 7893_2.patch added

comment:10 Changed 6 years ago by Sa'ar Zac Elias

Status: assignedreview

comment:11 Changed 6 years ago by Garry Yao

Status: reviewreview_failed

Does work in native IE7 which lost the "name" attribute

Changed 6 years ago by Sa'ar Zac Elias

Attachment: 7893_3.patch added

comment:12 Changed 6 years ago by Sa'ar Zac Elias

Status: review_failedreview

After finding another related bug with the patch, I'm proposing to deprecate the usage of document.anchors in those browsers.

comment:13 Changed 6 years ago by Garry Yao

Status: reviewreview_passed

comment:14 Changed 6 years ago by Sa'ar Zac Elias

Resolution: fixed
Status: review_passedclosed

Fixed with [7039].

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