Create hook to register events that won't scroll editor after aceEditEvt

pull/2944/head
Luiza Pagliari 2016-03-30 11:51:18 -03:00
parent 8f2f01dff8
commit 3fb695a7a0
2 changed files with 23 additions and 3 deletions

View File

@ -134,6 +134,13 @@ Things in context:
This hook is made available to edit the edit events that might occur when changes are made. Currently you can change the editor information, some of the meanings of the edit, and so on. You can also make internal changes (internal to your plugin) that use the information provided by the edit event. This hook is made available to edit the edit events that might occur when changes are made. Currently you can change the editor information, some of the meanings of the edit, and so on. You can also make internal changes (internal to your plugin) that use the information provided by the edit event.
## aceRegisterNonScrollableEditEvents
Called from: src/static/js/ace2_inner.js
Things in context: None
When aceEditEvent (documented above) finishes processing the event, it scrolls the viewport to make caret visible to the user, but if you don't want that behavior to happen you can use this hook to register which edit events should not scroll viewport. The return value of this hook should be a list of event names.
## aceRegisterBlockElements ## aceRegisterBlockElements
Called from: src/static/js/ace2_inner.js Called from: src/static/js/ace2_inner.js
@ -166,11 +173,11 @@ Called from: src/static/js/pad_editbar.js
Things in context: Things in context:
1. ace - the ace object that is applied to this editor. 1. ace - the ace object that is applied to this editor.
2. toolbar - Editbar instance. See below for the Editbar documentation. 2. toolbar - Editbar instance. See below for the Editbar documentation.
Can be used to register custom actions to the toolbar. Can be used to register custom actions to the toolbar.
Usage examples: Usage examples:
* [https://github.com/tiblu/ep_authorship_toggle]() * [https://github.com/tiblu/ep_authorship_toggle]()

View File

@ -369,6 +369,19 @@ function Ace2Inner(){
return thisAuthor; return thisAuthor;
} }
var _nonScrollableEditEvents = {
"applyChangesToBase": 1
};
_.each(hooks.callAll('aceRegisterNonScrollableEditEvents'), function(eventType) {
_nonScrollableEditEvents[eventType] = 1;
});
function isScrollableEditEvent(eventType)
{
return !_nonScrollableEditEvents[eventType];
}
var currentCallStack = null; var currentCallStack = null;
function inCallStack(type, action) function inCallStack(type, action)
@ -506,7 +519,7 @@ function Ace2Inner(){
{ {
updateBrowserSelectionFromRep(); updateBrowserSelectionFromRep();
} }
if ((cs.docTextChanged || cs.userChangedSelection) && cs.type != "applyChangesToBase") if ((cs.docTextChanged || cs.userChangedSelection) && isScrollableEditEvent(cs.type))
{ {
scrollSelectionIntoView(); scrollSelectionIntoView();
} }