Opened 8 years ago

Closed 8 years ago

#4057 closed Bug (fixed)

Anchor disappearing

Reported by: Garry Yao Owned by: Garry Yao
Priority: Must have (possibly next milestone) Milestone: CKEditor 3.0
Component: Core : Output Data Version:
Keywords: Confirmed Review+ Cc:

Description

There were too problems with hidden fields.

Reproducing Procedures

  1. Open the "replace by class" page, empty the document;
  2. Insert an anchor;
  3. Switch to 'Source' and twice;
    • Actual Result: The anchor is disappeared.

Attachments (1)

4057.patch (1.3 KB) - added by Garry Yao 8 years ago.

Download all attachments as: .zip

Change History (5)

comment:1 Changed 8 years ago by Garry Yao

Owner: set to Garry Yao
Status: newassigned

Changed 8 years ago by Garry Yao

Attachment: 4057.patch added

comment:2 Changed 8 years ago by Garry Yao

Keywords: Review? added

comment:3 Changed 8 years ago by Frederico Caldeira Knabben

Keywords: Review+ added; IE Review? removed
Summary: [IE6] Anchor disappearingAnchor disappearing

I've confirmed it in FF too.

The patch is broken. I've applied the changes manually to review it though.

Please add the curly brackets in this if block when committing. We must "always" have curly brackets if the if condition takes more than one line of code.

comment:4 Changed 8 years ago by Garry Yao

Resolution: fixed
Status: assignedclosed

Fixed with [3951].

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