span colors switching from HEX to RGB

[rud]

While working on my editor settings today, i accidentaly noticed my flash movie wasn't loading colors anymore .. after checking the xml i was loading (which is fckeditor generated), i noticed all the <span> blocks are now using RGB instead of HEX colors ..

Now i'm almost sure this is not related to my settings. I've even disabled my custom config file (and triple emptied cache), but still, i get RGB instead of HEX when i select a peace of text, and apply a color from the color panel.

[amerigo5]

This bug dramatically affects the HTML source by simply updating a HTML text using IE and FireFox. If the HTML source is created in IE, the color uses hexadecimal and when the same HTML source is updated using FireFox, suddenly all the color style gets converted to RGB. Can the priority of this bug be increased?

[Wojciech Olchawa]

Confirmed

[Martin Kou]

Firefox seems to have a tendency to turn color related CSS attributes to RGB format automagically, which was observed in the ticket #617 as well. It seems color related CSS attributes will need to be protected to properly fix this issue.

Since #617 is targeted to 2.7, I'm targeting this related ticket to 2.7 as well.

[Alfonso Martínez de Lizarrondo]

Proposed patch

[Alfonso Martínez de Lizarrondo]

While trying to fix this issue I've found that the css normalization isn't applied to the output data, so for example #1709 and #2929 aren't really fixed.

So in the patch I moved the normalizeCssText to CKEDITOR.tools, and the replacements at the end of the function have been adjusted so it properly cleans whitespace and removes the trailing semi colon.

This will help with the issues related to styles and marked as dups of #1810, but of course it will mean a slight penalty in performance, although I don't think that it will be something that can be noticed.

[Garry Yao]

This feature has to be configurable and delivered by htmlDataProcessor plugin instead of the core codes.

[Alfonso Martínez de Lizarrondo]

Let's handle this in 3.4 combined with #5528 to avoid performance problems.

[Wiktor Walc]

#6638 was marked as duplicate.

[Piotrek Koszuliński]

This issue was fixed long time ago.