Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#3891 closed Bug (fixed)

[IE7] Font color selector not working after using automatic option

Reported by: Damian Owned by: Garry Yao
Priority: Normal Milestone: CKEditor 3.0
Component: General Version:
Keywords: IBM Confirmed IE Review+ Cc:

Description (last modified by Garry Yao)

To reproduce:

  1. Open nightly Ajax sample
  2. Add some text
  3. Select text and apply a font colour (not automatic)
  4. Notice colour application worked
  5. Re-select the text and apply automatic font colour

Result: the automatic option is not applied. From this point onwards, using the font colour will produce unexpected results.

Attachments (1)

3891.patch (1.4 KB) - added by Garry Yao 8 years ago.

Download all attachments as: .zip

Change History (14)

comment:1 Changed 8 years ago by Artur Formella

Keywords: WorksForMe added

comment:2 Changed 8 years ago by Garry Yao

Description: modified (diff)
Resolution: fixed

WFM, Could you provide more details?

comment:3 Changed 8 years ago by Garry Yao

Description: modified (diff)

comment:4 Changed 8 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.0

comment:5 Changed 8 years ago by Damian

This is still reproducible in the nightly build.

Following the original steps exactly reproduces the issue.

After using automatic, notice that trying to apply a different colour now doesn't work.

comment:6 Changed 8 years ago by Frederico Caldeira Knabben

Keywords: Confirmed IE added; WorksForMe removed
Milestone: CKEditor 3.0

Definitely confimed with IE7 and the trunk Ajax sample.

comment:7 Changed 8 years ago by Garry Yao

Owner: set to Garry Yao
Status: newassigned

Changed 8 years ago by Garry Yao

Attachment: 3891.patch added

comment:8 Changed 8 years ago by Garry Yao

Keywords: Review? added

comment:9 Changed 8 years ago by Frederico Caldeira Knabben

Keywords: Review+ added; Review? removed

comment:10 Changed 8 years ago by Garry Yao

Status: assignedclosed

Fixed with [3922].

comment:11 Changed 8 years ago by Tobiasz Cudnik

Keywords: Review- added; Review+ removed
Resolution: fixed
Status: closedreopened

I'm still able to reproduce it on FF3.5, following steps in description. After selecting automatic, there's no chance to set normal color. It results in following code:

<p>
	f<span style="">o</span>o</p>

After failed try to apply color to first "o".

comment:12 Changed 8 years ago by Frederico Caldeira Knabben

Resolution: fixed
Status: reopenedclosed

Please do not reopen a ticket for regressions opening a dedicated ticket for it instead.

comment:13 Changed 8 years ago by Tobiasz Cudnik

Keywords: Review+ added; Review- removed

Opened new #4193, which is a regression of [4024] and still can be fixed with patch from this ticket.

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