Merge pull request #5861 from lkcampbell/3063-fix

redmunds · redmunds · commit 9333acfe6e97 · 2013-11-06T13:47:45.000-08:00
Automatically scroll find selection into view
diff --git a/src/search/FindReplace.js b/src/search/FindReplace.js
@@ -143,6 +143,7 @@ define(function (require, exports, module) {
                 // no need to scroll if the line with the match is in view
                 centerOptions = Editor.BOUNDARY_IGNORE_TOP;
             }
+            cm.scrollIntoView({from: cursor.from(), to: cursor.to()});
             editor.setSelection(cursor.from(), cursor.to(), true, centerOptions);
             state.posFrom = cursor.from();
             state.posTo = cursor.to();

Original file line number	Diff line number	Diff line change
`@@ -143,6 +143,7 @@ define(function (require, exports, module) {`
`143`	`143`	`// no need to scroll if the line with the match is in view`
`144`	`144`	`centerOptions = Editor.BOUNDARY_IGNORE_TOP;`
`145`	`145`	`}`
	`146`	`+ cm.scrollIntoView({from: cursor.from(), to: cursor.to()});`
`146`	`147`	`editor.setSelection(cursor.from(), cursor.to(), true, centerOptions);`
`147`	`148`	`state.posFrom = cursor.from();`
`148`	`149`	`state.posTo = cursor.to();`