Context Navigation

← Previous Changeset
Next Changeset →

Changeset 5760

Timestamp:

07/28/10 00:06:30 (3 years ago)

Author:

fredck

Message:

#5280 : IE: Scrollbars where reacting improperly when clicking in the bar space. [5738] has been reverted and a different solution has been found to satisfy #5634.

Location:

CKEditor/trunk

Files:

: 3 edited

CHANGES.html (modified) (1 diff)
_source/plugins/selection/plugin.js (modified) (1 diff)
_source/plugins/wysiwygarea/plugin.js (modified) (1 diff)

Legend:

: Unmodified
: Added
: Removed

CKEditor/trunk/CHANGES.html

r5750	r5760
95	95	<li><a href="http://dev.fckeditor.net/ticket/5781">#5781</a> : Firefox: Editing was not possible in an empty document.</li>
96	96	<li><a href="http://dev.fckeditor.net/ticket/5293">#5293</a> : Firefox: Unwanted BR tags were being left in the editor output when it should be empty.</li>
	97	<li><a href="http://dev.fckeditor.net/ticket/5280">#5280</a> : IE: Scrollbars where reacting improperly when clicking in the bar space.</li>
97	98	<li>Updated the following language files:<ul>
98	99	<li><a href="http://dev.fckeditor.net/ticket/5962">#5962</a> : German;</li>

CKEditor/trunk/_source/plugins/selection/plugin.js

-                      r5742
+                      r5760
                                                         restoreEnabled = 1;
                                                 });
+                                                // In IE6/7 the blinking cursor appears, but contents are
+                                                // not editable. (#5634)
+                                                if ( CKEDITOR.env.ie && ( CKEDITOR.env.ie7Compat || CKEDITOR.env.version < 8 || CKEDITOR.env.quirks ) )
+                                                {
+                                                        // The 'click' event is not fired when clicking the
+                                                        // scrollbars, so we can use it to check whether
+                                                        // the empty space following <body> has been clicked.
+                                                        html.on( 'click', function( evt )
+                                                        {
+                                                                if ( evt.data.getTarget().getName() == 'html' )
+                                                                        editor.getSelection().getRanges()[ 0 ].select();
+                                                        });
+                                                }
                                                 // IE fires the "selectionchange" event when clicking

CKEditor/trunk/_source/plugins/wysiwygarea/plugin.js

-                      r5749
+                      r5760
+                                                {
                                                         editor.focus();
-                                                        // In IE7 the blinking cursor appears, but contents are not editable. (#5634)
-                                                        if ( CKEDITOR.env.ie7Compat )
-                                                                editor.getSelection().getRanges()[ 0 ].select();
                                                 } );
                                 } );

Note: See TracChangeset for help on using the changeset viewer.