Ticket #5018 (closed Bug: fixed)

Opened 5 years ago

Last modified 4 years ago

Find highlighter removes bgcolor and font color from words

Reported by: damo Owned by: garry.yao
Priority: Normal Milestone: CKEditor 3.3
Component: Core : Styles Version: SVN (CKEditor) - OLD
Keywords: IBM Confirmed Review+ Cc: JoeK

Description

  1. Open the CK Editor.
  2. Apply text colour or background colour to a word.
  3. Click on find icon
  4. Search for the word from step 2.
  5. See that the word gets highlighted
  6. Click on cancel button to close the find and replace light box

Actual result: the text colour or background colour that you have applied to the word is removed.

Expected result: the text colour or back ground colour applied to the word should remain

Attachments

5018.patch (753 bytes) - added by garry.yao 5 years ago.
5018_2.patch (802 bytes) - added by garry.yao 5 years ago.
5018_3.patch (1.8 KB) - added by garry.yao 4 years ago.
5018_4.patch (2.4 KB) - added by garry.yao 4 years ago.

Change History

comment:1 Changed 5 years ago by garry.yao

  • Keywords Confirmed added
  • Owner set to garry.yao
  • Status changed from new to assigned
  • Component changed from General to Core : Styles
  • Version set to SVN (CKEditor)

Nice catch!

Changed 5 years ago by garry.yao

comment:2 Changed 5 years ago by garry.yao

  • Keywords Review? added

comment:3 Changed 5 years ago by fredck

  • Keywords Review- added; Review? removed

Instead of setting the childRule in the configuration option, let's have it hardcoded (when loading find_highlight), making the settings simpler, avoiding errors by third developers and also making the fix backwards compatible with custom configuration files.

Changed 5 years ago by garry.yao

comment:4 Changed 5 years ago by garry.yao

  • Keywords Review? added; Review- removed

comment:5 Changed 5 years ago by fredck

  • Keywords Review- added; Review? removed

I've tested the patch with the ticket TC and it's still not working.

Changed 4 years ago by garry.yao

comment:6 Changed 4 years ago by garry.yao

  • Keywords Review? added; Review- removed

Proposing of a new approach.

comment:7 Changed 4 years ago by fredck

  • Keywords Review- added; Review? removed
  1. In FF, load the following HTML:
<p>You <span style="background-color: rgb(0, 255, 0);">are</span> using CKEditor.</p>
  1. Open the Find dialog.
  2. Find the word "are". Note that the word is not properly highlighted.
  3. Click "Cancel" to close the dialog.

The word "are" doesn't loose the background, but now it's foreground color is white (almost) and the source shows the following:

<p>You <span style="background-color: rgb(0, 0, 68); color: rgb(255, 255, 255);"><span style="background-color: rgb(0, 255, 0);">are</span></span> using CKEditor.</p>

The source output should instead remain untouched.

Also, if you'll be still proposing that additional property in the style definition, please rename it to "fullMatch". "unique" doesn't tell us much.

Changed 4 years ago by garry.yao

comment:8 Changed 4 years ago by garry.yao

  • Keywords Review? added; Review- removed

comment:9 Changed 4 years ago by fredck

  • Keywords Review+ added; Review? removed

comment:10 Changed 4 years ago by garry.yao

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

Fixed with [5290].

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