[FCKeditor-Commits] r6294 - in CKEditor/trunk: . _source/plugins/htmldataprocessor

[<com...@fc...>]

Author: Saare
Date: 2011-01-05 14:37:56 +0100 (Wed, 05 Jan 2011)
New Revision: 6294

Modified:
   CKEditor/trunk/CHANGES.html
   CKEditor/trunk/_source/plugins/htmldataprocessor/plugin.js
Log:
#5930: [IE] Style definitions were lowercased.

Modified: CKEditor/trunk/CHANGES.html
===================================================================
--- CKEditor/trunk/CHANGES.html	2011-01-05 11:01:03 UTC (rev 6293)
+++ CKEditor/trunk/CHANGES.html	2011-01-05 13:37:56 UTC (rev 6294)
@@ -72,6 +72,7 @@
 		<li><a href="http://dev.ckeditor.com/ticket/6641">#6641</a> : Copying and pasting links inside the editor wasn't working.</li>
 		<li><a href="http://dev.ckeditor.com/ticket/4208">#4208</a> : The <a href="http://docs.cksource.com/ckeditor_api/symbols/CKEDITOR.config.html#.disableObjectResizing">disableObjectResizing</a> setting now works on IE.</li>
 		<li><a href="http://dev.ckeditor.com/ticket/6242">#6242</a> : [IE] Editing existing links with href of a relative path mangles containing text.</li>
+		<li><a href="http://dev.ckeditor.com/ticket/5930">#5930</a> : [IE] Style definitions are no longer lowercased.</li>
 	</ul>
 	<h3>
 			CKEditor 3.5</h3>

Modified: CKEditor/trunk/_source/plugins/htmldataprocessor/plugin.js
===================================================================
--- CKEditor/trunk/_source/plugins/htmldataprocessor/plugin.js	2011-01-05 11:01:03 UTC (rev 6293)
+++ CKEditor/trunk/_source/plugins/htmldataprocessor/plugin.js	2011-01-05 13:37:56 UTC (rev 6294)
@@ -251,10 +251,13 @@
 	if ( CKEDITOR.env.ie )
 	{
 		// IE outputs style attribute in capital letters. We should convert
-		// them back to lower case.
+		// them back to lower case, while not hurting the values (#5930)
 		defaultHtmlFilterRules.attributes.style = function( value, element )
 		{
-			return value.toLowerCase();
+			return value.replace( /(^|;)([^\:]+)/g, function( match )
+				{
+					return match.toLowerCase();
+				});
 		};
 	}