Ticket #4546: 4546.patch

_source/core/tools.js

@@ -267 +267 @@
                 },
 
                 /**
+                 * Replace characters can't be represented through CSS Selectors string
+                 * by CSS Escape Notation where the character escape sequence consists
+                 * of a backslash character (\) followed by the orginal characters.
+                 * Ref: http://www.w3.org/TR/css3-selectors/#grammar
+                 * @param cssSelectText
+                 * @return the escaped selector text. 
+                 */
+                escapeCssSelector : function( cssSelectText )
+                {
+                        return cssSelectText.replace( /[\s#:.,$*^\[\]()~=+>]/g, '\\$&' );
+                },
+
+                /**
                  * Gets a unique number for this CKEDITOR execution session. It returns
                  * progressive numbers starting at 1.
                  * @function

_tests/dt/core/tools.html

@@ -155 +155 @@
                 {
                         assert.areSame( '   ', CKEDITOR.tools.repeat( ' ', 3 ) );
                 },
-
+                
+                test_escapeCssSelector : function ()
+                {
+                        assert.areSame( '\\.\\,\\*\\=\\~\\$\\^\\(\\)\\ \\:\\#\\+\\>', CKEDITOR.tools.escapeCssSelector( '.,*=~$^() :#+>' ) );
+                },
+
                 name : document.title
         };
 })() );

_source/skins/kama/skin.js

@@ -147 +147 @@
                                 {
                                         var cssContent,
                                                 uiStyle = addStylesheet( CKEDITOR.document ),
-                                                cssId = '#cke_' + editor.name.replace('.', '\\.');
+                                                cssId = '#cke_' + CKEDITOR.tools.escapeCssSelector( editor.name );
 
                                         var cssSelectors =
                                                 [