Index: _tests/dt/plugins/styles/styles.html =================================================================== --- _tests/dt/plugins/styles/styles.html (revision 4118) +++ _tests/dt/plugins/styles/styles.html Wed Sep 02 19:51:04 CST 2009 @@ -427,6 +427,15 @@ assert.isTrue( style.checkElementRemovable( element, true ) ); }, + test_checkElementRemovable_fontFamily : function() + { + var element = CKEDITOR.dom.element.createFromHtml( 'Test Font', doc ); + + var style = new CKEDITOR.style( { element : 'span', styles : { 'font-family' : '#(family)' } }, { family : 'Georgia, serif;' } ); + + assert.isTrue( style.checkElementRemovable( element, true ) ); + }, + test_ticket_3091 : function() { var element = doc.getById( '_P1' ); @@ -583,6 +592,8 @@ assert.areSame( '
paragraph1 paragraph1
paragraph2
para graph3
', result ); }, + + name : document.title }; })() ); Index: _source/plugins/styles/plugin.js =================================================================== --- _source/plugins/styles/plugin.js (revision 4050) +++ _source/plugins/styles/plugin.js Wed Sep 02 19:43:42 CST 2009 @@ -1200,6 +1200,7 @@ // Compensate tail semi-colon. return styleText.replace( /\s*([;:])\s*/, '$1' ) .replace( /([^\s;])$/, '$1;') + .replace( /,\s+/g, ',' ) // Trimming spaces after comma (e.g. font-family name)(#4107). .toLowerCase(); }