Changeset 5494


Ignore:
Timestamp:
05/14/2010 05:30:28 PM (5 years ago)
Author:
garry.yao
Message:

#5673: Properly set focus when editing frame is focused.

Location:
CKEditor/branches/versions/3.3.x
Files:
3 edited

Legend:

Unmodified
Added
Removed
  • CKEditor/branches/versions/3.3.x/CHANGES.html

    r5488 r5494  
    9696                <li><a href="http://dev.fckeditor.net/ticket/5617">#5617</a> : Html filter system does not allow two 'text' filter rules.</li>
    9797                <li><a href="http://dev.fckeditor.net/ticket/5663">#5663</a> : General memory clean up after destroying last instance.</li>
     98                <li><a href="http://dev.fckeditor.net/ticket/5673">#5673</a> : [Firefox] Tab key have to press twice to focus the editor.</li>
    9899                <li>Updated the following language files:<ul>
    99100                        <li><a href="http://dev.fckeditor.net/ticket/5432">#5432</a> : Dutch;</li>
  • CKEditor/branches/versions/3.3.x/_source/plugins/tab/plugin.js

    r5189 r5494  
    5252                        }
    5353
    54                         if ( CKEDITOR.env.webkit )
     54                        if ( CKEDITOR.env.webkit || CKEDITOR.env.gecko )
    5555                        {
    5656                                editor.on( 'key', function( ev )
  • CKEditor/branches/versions/3.3.x/_source/plugins/wysiwygarea/plugin.js

    r5488 r5494  
    485485                                                }
    486486
    487                                                 var focusTarget = ( CKEDITOR.env.ie || CKEDITOR.env.webkit ) ?
    488                                                                 domWindow : domDocument;
    489 
    490                                                 focusTarget.on( 'blur', function()
     487                                                domWindow.on( 'blur', function()
    491488                                                        {
    492489                                                                editor.focusManager.blur();
    493490                                                        });
    494491
    495                                                 focusTarget.on( 'focus', function()
    496                                                         {
    497                                                                 // Force the cursor blinking. (#5622
    498                                                                 CKEDITOR.env.gecko && blinkCursor();
     492                                                domWindow.on( 'focus', function()
     493                                                        {
     494                                                                var doc = editor.document;
     495                                                                if ( CKEDITOR.env.gecko || CKEDITOR.env.opera )
     496                                                                        doc.getBody().focus();
     497                                                                else if ( CKEDITOR.env.webkit )
     498                                                                        doc.getDocumentElement().focus();
     499
    499500                                                                editor.focusManager.focus();
    500501                                                        });
     
    781782                                                                else if ( editor.window )
    782783                                                                {
    783                                                                         // [Webkit] Force the cursor blinking
    784                                                                         // when setting focus manually. (#5622)
    785                                                                         CKEDITOR.env.webkit && blinkCursor();
    786 
    787                                                                         if ( CKEDITOR.env.opera )
    788                                                                                 editor.document.getBody().focus();
    789                                                                         else
    790                                                                                 editor.window.focus();
     784                                                                        editor.window.focus();
    791785
    792786                                                                        editor.selectionChange();
Note: See TracChangeset for help on using the changeset viewer.
© 2003 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy