Ticket #188: 188_3.patch

_source/plugins/button/plugin.js

@@ -210 +210 @@
 
                 output.push(
                                         ' onkeydown="return CKEDITOR.tools.callFunction(', keydownFn, ', event);"' +
-                                        ' onfocus="return CKEDITOR.tools.callFunction(', focusFn,', event);"' +
-                                ' onclick="CKEDITOR.tools.callFunction(', clickFn, ', this); return false;">' +
+                                        ' onfocus="return CKEDITOR.tools.callFunction(', focusFn,', event);" ' +
+                                        ( CKEDITOR.env.ie ? 'onclick="return false;" onmouseup' : 'onclick' ) +         // #188
+                                                '="CKEDITOR.tools.callFunction(', clickFn, ', this); return false;">' +
                                         '<span class="cke_icon"' );
 
                 if ( this.icon )

_source/plugins/listblock/plugin.js

@@ -96 +96 @@
                                                         '<li id=', id, ' class=cke_panel_listItem role=presentation>' +
                                                                 '<a id="', id, '_option" _cke_focus=1 hidefocus=true' +
                                                                         ' title="', title || value, '"' +
-                                                                        ' href="javascript:void(\'', value, '\')"' +
-                                                                        ' onclick="CKEDITOR.tools.callFunction(', this._.getClick(), ',\'', value, '\'); return false;"',
+                                                                        ' href="javascript:void(\'', value, '\')" ' +
+                                                                        ( CKEDITOR.env.ie ? 'onmouseup' : 'onclick' ) +         // #188
+                                                                                '="CKEDITOR.tools.callFunction(', this._.getClick(), ',\'', value, '\'); return false;"',
                                                                         ' role="option"' +
                                                                         ' aria-posinset="' + ++this._.size + '">',
                                                                         html || value,

_source/plugins/menu/plugin.js

@@ -478 +478 @@
                                 output.push(
 //                                      ' onkeydown="return CKEDITOR.ui.button._.keydown(', index, ', event);"' +
                                         ' onmouseover="CKEDITOR.tools.callFunction(', menu._.itemOverFn, ',', index, ');"' +
-                                        ' onmouseout="CKEDITOR.tools.callFunction(', menu._.itemOutFn, ',', index, ');"' +
-                                        ' onclick="CKEDITOR.tools.callFunction(', menu._.itemClickFn, ',', index, '); return false;"' +
+                                        ' onmouseout="CKEDITOR.tools.callFunction(', menu._.itemOutFn, ',', index, ');" ' +
+                                        ( CKEDITOR.env.ie ? 'onmouseup' : 'onclick' ) +         // #188
+                                                '="CKEDITOR.tools.callFunction(', menu._.itemClickFn, ',', index, '); return false;"' +
                                         '>' +
                                                 '<span class="cke_icon_wrapper"><span class="cke_icon"' +
                                                         ( this.icon ? ' style="background-image:url(' + CKEDITOR.getUrl( this.icon ) + ');background-position:0 ' + offset + 'px;"'

_source/plugins/richcombo/plugin.js

@@ -199 +199 @@
 
                         output.push(
                                         ' onkeydown="CKEDITOR.tools.callFunction( ', keyDownFn, ', event, this );"' +
-                                        ' onfocus="return CKEDITOR.tools.callFunction(', focusFn, ', event);"' +
-                                        ' onclick="CKEDITOR.tools.callFunction(', clickFn, ', this); return false;">' +
+                                        ' onfocus="return CKEDITOR.tools.callFunction(', focusFn, ', event);" ' +
+                                        ( CKEDITOR.env.ie ? 'onmouseup' : 'onclick' ) +         // #188
+                                                '="CKEDITOR.tools.callFunction(', clickFn, ', this); return false;">' +
                                                 '<span>' +
                                                         '<span id="' + id + '_text" class="cke_text cke_inline_label">' + this.label + '</span>' +
                                                 '</span>' +