Opened 7 years ago

Closed 7 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 7 years ago.

Download all attachments as: .zip

Change History (5)

comment:1 Changed 7 years ago by garry.yao

  • Owner set to garry.yao
  • Status changed from new to assigned

Changed 7 years ago by garry.yao

comment:2 Changed 7 years ago by garry.yao

  • Keywords Review? added

comment:3 Changed 7 years ago by fredck

  • Keywords Review+ added; IE Review? removed
  • Summary changed from [IE6] Anchor disappearing to Anchor 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 7 years ago by garry.yao

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

Fixed with [3951].

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