Opened 6 years ago

Closed 6 years ago

#5623 closed Bug (fixed)

Apply two styles problematic

Reported by: garry.yao Owned by: garry.yao
Priority: Normal Milestone: CKEditor 3.3
Component: Core : Styles Version: 3.2.1
Keywords: Confirmed Firefox Review+ Cc:




Reproducing Procedures

  1. Open the 'replacebyclass' sample page and make the following selection:
    	[This is ]some...
  2. Apply 'Bold' style on the selected content;
  3. Make the new selection as following and apply 'Italic' Style:
    	<strong>This [is ]</strong> some...
    • Actual Result: The italic style has expanded onto the previous word.
    • Expected Result: Only the selected content receive the new style.

Attachments (1)

5623.patch (2.0 KB) - added by garry.yao 6 years ago.

Download all attachments as: .zip

Change History (4)

Changed 6 years ago by garry.yao

comment:1 Changed 6 years ago by garry.yao

  • Keywords Review? added
  • Owner set to garry.yao
  • Status changed from new to assigned

comment:2 Changed 6 years ago by fredck

  • Keywords Review+ added; Review? removed

I remember we've used bookmarks in the past, but for some reason we've changed it to the current code. Maybe we were simply trying to use the original bookmark, while you're proposing creating a new bookmark after enlarging the range.

The change is good, but we must test and watch the style system pretty well before the release.

comment:3 Changed 6 years ago by garry.yao

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

Fixed with [5464].

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