WordPress.org

Make WordPress Core

Ticket #28328: 28328.17.patch

File 28328.17.patch, 2.6 KB (added by iseulde, 4 years ago)
  • src/wp-admin/js/editor-expand.js

     
    122122                        }, 200 );
    123123                } );
    124124
    125                 editor.on( 'keyup', function() {
    126                         var offset = getCursorOffset(),
     125                // Make sure the cursor is always visible.
     126                // This is not only necessary to keep the cursor between the toolbars,
     127                // but also to scroll the window when the cursor moves out of the viewport to a wpview.
     128                // Setting a buffer > 0 will prevent the browser default.
     129                // Some browsers will scroll to the middle,
     130                // others to the top/bottom of the *window* when moving the cursor out of the viewport.
     131                editor.on( 'keyup', function( event ) {
     132                        var VK = tinymce.util.VK,
     133                                key = event.keyCode,
     134                                offset = getCursorOffset(),
    127135                                windowHeight = $window.height(),
     136                                buffer = 10,
    128137                                cursorTop, cursorBottom, editorTop, editorBottom;
    129138
    130139                        if ( ! offset ) {
     
    133142
    134143                        cursorTop = offset.top + editor.getContentAreaContainer().getElementsByTagName( 'iframe' )[0].getBoundingClientRect().top;
    135144                        cursorBottom = cursorTop + offset.height;
     145                        cursorTop = cursorTop - buffer;
     146                        cursorBottom = cursorBottom + buffer;
    136147                        editorTop = $adminBar.outerHeight() + $tools.outerHeight() + $visualTop.outerHeight();
    137                         editorBottom = $window.height() - $bottom.outerHeight();
     148                        editorBottom = windowHeight - $bottom.outerHeight();
    138149
    139                         if ( cursorTop < editorTop || cursorBottom > editorBottom ) {
    140                                 window.scrollTo( window.pageXOffset, cursorTop + window.pageYOffset - windowHeight / 2 );
     150                        if ( cursorTop < editorTop && ( key === VK.UP || key === VK.LEFT ) ) {
     151                                window.scrollTo( window.pageXOffset, cursorTop + window.pageYOffset - editorTop );
     152                        } else if ( cursorBottom > editorBottom ) {
     153                                window.scrollTo( window.pageXOffset, cursorBottom + window.pageYOffset - editorBottom );
    141154                        }
    142155                } );
    143156
     
    145158                        var selection = editor.selection,
    146159                                node = selection.getNode(),
    147160                                range = selection.getRng(),
    148                                 view, clone, right, offset;
     161                                view, clone, offset;
    149162
    150163                        if ( tinymce.Env.ie && tinymce.Env.ie < 9 ) {
    151164                                return;
     
    159172                                if ( clone.startContainer.length > 1 ) {
    160173                                        if ( clone.startContainer.length > clone.endOffset ) {
    161174                                                clone.setEnd( clone.startContainer, clone.endOffset + 1 );
    162                                                 right = true;
    163175                                        } else {
    164176                                                clone.setStart( clone.startContainer, clone.endOffset - 1 );
    165177                                        }
     
    175187                        }
    176188
    177189                        if ( ! offset.height ) {
    178                                 return false;
     190                                return;
    179191                        }
    180192
    181193                        return offset;