Ticket #5132 (closed Bug: fixed)

Opened 5 years ago

Last modified 4 years ago

Formatting automatically applied to all characters/words when only single character/word is highlighted

Reported by: pranav Owned by: garry.yao
Priority: Normal Milestone: CKEditor 3.3
Component: Core : Styles Version: 3.1
Keywords: Oracle Review+ Cc: Senthil, Pranav

Description

Write some text e.g. A B C in editor area. Select "A" and click on underline button. It underlines the text. Now select "B" and click on underline button. It underlines "B" correctly. Now select "C" and click on italic button. You will observe that entire text including "A" and "B" is changed to italic. Same is the case with words.

This issue was reproducible in nightly build with Internet Explorer 8 and Firefox 3.6.

Attachments

5132.patch (792 bytes) - added by garry.yao 4 years ago.

Change History

comment:1 Changed 5 years ago by fredck

  • Keywords Confirmed added
  • Milestone set to CKEditor 3.3

comment:2 follow-up: ↓ 3 Changed 4 years ago by garry.yao

  • Keywords Pending added; Confirmed removed

Works for me with the following source on both IE and Firefox:

<p>A B C</p>

Any other clue?

comment:3 in reply to: ↑ 2 Changed 4 years ago by pranav

Replying to garry.yao:

Works for me with the following source on both IE and Firefox:

<p>A B C</p>

Any other clue?

I was able to reproduce the issue in nightly build with IE8. Can you please varify it once again?

comment:4 Changed 4 years ago by pranav

Please refer to this video demonstrating the issue.

http://www.youtube.com/watch?v=S3pqXM8atAc

I was using latest nightly build with IE8.

Changed 4 years ago by garry.yao

comment:5 Changed 4 years ago by garry.yao

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

Confirmed, have to be very care to select only the character (without including space) to reproduce the bug.

The patch is trying to cover a logic whole in 'range::enlarge' method.

Ticket Test added:
run OR view source.

comment:6 Changed 4 years ago by alfonsoml

  • Keywords Review+ added; Pending Review? removed

comment:7 Changed 4 years ago by garry.yao

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

Fixed with [5220].

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