Opened 7 years ago

Closed 7 years ago

#6665 closed Bug (fixed)

NAME ATTRIBUTE IS MISSING FOR LINK ON IE

Reported by: naresh.sivaraman Owned by: Garry Yao
Priority: Normal Milestone: CKEditor 3.5.1
Component: Core : Styles Version: 3.0
Keywords: IE Oracle Cc:

Description

Happens in IE6 and IE7.

To reproduce:

  1. click "link" icon and input "Name" and other attributes in Advanced tab.

2.Then, right click the link created in step 1 and choose "Edit Link"

3.open Advanced tab in the editing link dialog,Name attribute is missing in it.

4.checking source mode, the name attribute is not set there either.

Attachments (2)

6665.patch (3.3 KB) - added by Garry Yao 7 years ago.
6665_3.patch (3.5 KB) - added by Garry Yao 7 years ago.

Download all attachments as: .zip

Change History (12)

comment:1 Changed 7 years ago by naresh.sivaraman

Any updates on this bug please?

comment:2 Changed 7 years ago by Garry Yao

Component: GeneralCore : Styles
Keywords: IE added
Status: newconfirmed
Version: 3.4.23.0

An unfixed part of #3008.

comment:3 Changed 7 years ago by Wiktor Walc

Keywords: Oracle added

Changed 7 years ago by Garry Yao

Attachment: 6665.patch added

comment:4 Changed 7 years ago by Garry Yao

Owner: set to Garry Yao
Status: confirmedreview

comment:5 Changed 7 years ago by Wiktor Walc

Milestone: CKEditor 3.5.1

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

Status: reviewreview_failed

Create a link and go to source mode. Switch back to wysiwyg mode. Note that the anchor icon is on it.
Also, the findSavedSrcRegex variable can now be removed.

Changed 7 years ago by Garry Yao

Attachment: 6665_3.patch added

comment:7 in reply to:  6 Changed 7 years ago by Garry Yao

Status: review_failedreview

Replying to Saare:

Create a link and go to source mode. Switch back to wysiwyg mode. Note that the anchor icon is on it.

This's the correct behavior. (inherited from v2)

comment:8 Changed 7 years ago by Garry Yao

The patch also fixes a bug reported at #6953 where old protected attributes were overridden by newly created ones.

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

Status: reviewreview_passed

comment:10 Changed 7 years ago by Garry Yao

Resolution: fixed
Status: review_passedclosed

Fixed with [6316].

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