Opened 10 years ago

Closed 10 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 10 years ago.
6665_3.patch (3.5 KB) - added by Garry Yao 10 years ago.

Download all attachments as: .zip

Change History (12)

comment:1 Changed 10 years ago by naresh.sivaraman

Any updates on this bug please?

comment:2 Changed 10 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 10 years ago by Wiktor Walc

Keywords: Oracle added

Changed 10 years ago by Garry Yao

Attachment: 6665.patch added

comment:4 Changed 10 years ago by Garry Yao

Owner: set to Garry Yao
Status: confirmedreview

comment:5 Changed 10 years ago by Wiktor Walc

Milestone: CKEditor 3.5.1

comment:6 Changed 10 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 10 years ago by Garry Yao

Attachment: 6665_3.patch added

comment:7 in reply to:  6 Changed 10 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 10 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 10 years ago by Sa'ar Zac Elias

Status: reviewreview_passed

comment:10 Changed 10 years ago by Garry Yao

Resolution: fixed
Status: review_passedclosed

Fixed with [6316].

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