Ticket #4107: 4107_2.patch

_tests/dt/plugins/styles/styles.html

@@ -427 +427 @@
                         assert.isTrue( style.checkElementRemovable( element, true ) );
                 },
 
+                test_checkElementRemovable_fontFamily : function()
+                {
+                        var element = CKEDITOR.dom.element.createFromHtml( '<span style="font-family: georgia,  serif">Test Font</span>', 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 +592 @@
                         assert.areSame( '<p>&nbsp;paragraph1&nbsp; paragraph1<br />paragraph2</p><p>&nbsp;para&nbsp;&nbsp; graph3</p>',
                                          result );
                 },
+
+
                 name : document.title
         };
 })() );

_source/plugins/styles/plugin.js

@@ -1200 +1200 @@
                 // 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();
         }