Changeset 7370


Ignore:
Timestamp:
03/06/12 17:26:09 (2 years ago)
Author:
garry.yao
Message:

#8698: Fix 'Esc' key to close color picker dialog.

Location:
CKEditor/trunk
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • CKEditor/trunk/CHANGES.html

    r7369 r7370  
    5151                <li><a href="http://dev.ckeditor.com/ticket/8629">#8629</a> : Optimized float panel layout when there's no enough spaces in viewport.</li> 
    5252                <li><a href="http://dev.ckeditor.com/ticket/7955">#7955</a> : [FF] Fix "Page Up"/"Page Down" in wysiwyg mode cause selection lost.</li> 
     53                <li><a href="http://dev.ckeditor.com/ticket/8698">#8698</a> : Fix "Esc" shortcut key to close color picker dialog.</li> 
    5354        </ul> 
    5455        <h3> 
  • CKEditor/trunk/_source/plugins/colordialog/dialogs/colordialog.js

    r6348 r7370  
    6262                var onMouseout = $tools.addFunction( clearHighlight ), 
    6363                        onClick = updateSelected, 
    64                         onClickHandler = CKEDITOR.tools.addFunction( onClick ), 
    65                         onFocus = updateHighlight, 
    66                         onBlur = clearHighlight; 
     64                        onClickHandler = CKEDITOR.tools.addFunction( onClick ); 
    6765 
    6866                var onKeydownHandler = CKEDITOR.tools.addFunction( function( ev ) 
     
    8381                                                nodeToMove = relative.getChild( [element.getParent().getIndex(), 0] ); 
    8482                                                nodeToMove.focus(); 
    85                                                 onBlur( ev, element ); 
    86                                                 onFocus( ev, nodeToMove ); 
    8783                                        } 
    8884                                        ev.preventDefault(); 
     
    9793                                                { 
    9894                                                        nodeToMove.focus(); 
    99                                                         onBlur( ev, element ); 
    100                                                         onFocus( ev, nodeToMove ); 
    10195                                                } 
    10296                                        } 
     
    119113                                                { 
    120114                                                        nodeToMove.focus(); 
    121                                                         onBlur( ev, element ); 
    122                                                         onFocus( ev, nodeToMove ); 
    123115                                                        ev.preventDefault( true ); 
    124116                                                } 
    125                                                 else 
    126                                                         onBlur( null, element ); 
    127117                                        } 
    128118                                        // relative is TR 
     
    133123                                                { 
    134124                                                        nodeToMove.focus(); 
    135                                                         onBlur( ev, element ); 
    136                                                         onFocus( ev, nodeToMove ); 
    137125                                                        ev.preventDefault( true ); 
    138126                                                } 
    139                                                 else 
    140                                                         onBlur( null, element ); 
    141127                                        } 
    142128                                        break; 
     
    149135                                                nodeToMove = relative.getChild( 0 ); 
    150136                                                nodeToMove.focus(); 
    151                                                 onBlur( ev, element ); 
    152                                                 onFocus( ev, nodeToMove ); 
    153137                                                ev.preventDefault( true ); 
    154138                                        } 
     
    158142                                                nodeToMove = relative.getLast().getChild( 0 ); 
    159143                                                nodeToMove.focus(); 
    160                                                 onBlur( ev, element ); 
    161                                                 onFocus( ev, nodeToMove ); 
    162144                                                ev.preventDefault( true ); 
    163145                                        } 
    164                                         else 
    165                                                 onBlur( null, element ); 
    166146                                        break; 
    167147                                default : 
     
    279259                                                                                var table = CKEDITOR.document.getById( this.domId ); 
    280260                                                                                table.on( 'mouseover', updateHighlight ); 
     261                                                                                CKEDITOR.event.useCapture = true; 
     262                                                                                table.on( 'focus', updateHighlight ); 
     263                                                                                CKEDITOR.event.useCapture = false; 
     264 
    281265                                                                                // In WebKit, the table content must be inserted after this event call (#6150) 
    282266                                                                                CKEDITOR.env.webkit && table.setHtml( html ); 
Note: See TracChangeset for help on using the changeset viewer.
© 2003 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy