Changeset 4635


Ignore:
Timestamp:
12/09/09 16:12:01 (4 years ago)
Author:
garry.yao
Message:

Fixing #4733: dialog gives up trying to give focus while all focusable are checked.

Location:
CKEditor/trunk
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • CKEditor/trunk/CHANGES.html

    r4634 r4635  
    9191                <li><a href="http://dev.fckeditor.net/ticket/4690">#4690</a> : Fixed the parsing of nested inline elements.</li> 
    9292                <li><a href="http://dev.fckeditor.net/ticket/4450">#4450</a> : Fixed selecting multiple table cells before apply justify commands generates spurious paragraph in Firefox.</li> 
     93                <li><a href="http://dev.fckeditor.net/ticket/4733">#4733</a> : Fixed dialog opening sometimes hang up Firefox and Safari.</li> 
    9394        </ul> 
    9495        <h3> 
  • CKEditor/trunk/_source/plugins/dialog/plugin.js

    r4616 r4635  
    260260                                return; 
    261261 
    262                         var currentIndex = ( me._.currentFocusIndex + offset + focusList.length ) % focusList.length; 
     262                        var startIndex = ( me._.currentFocusIndex + offset + focusList.length ) % focusList.length, 
     263                                currentIndex = startIndex; 
    263264                        while ( !focusList[ currentIndex ].isFocusable() ) 
    264265                        { 
    265266                                currentIndex = ( currentIndex + offset + focusList.length ) % focusList.length; 
    266                                 if ( currentIndex == me._.currentFocusIndex ) 
     267                                if ( currentIndex == startIndex ) 
    267268                                        break; 
    268269                        } 
Note: See TracChangeset for help on using the changeset viewer.
© 2003 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy