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