Skip to content
This repository has been archived by the owner on Sep 6, 2021. It is now read-only.

Fix issue #6612: Find in Files panel doesn't reset scroll... #6629

Merged
merged 2 commits into from
Jan 30, 2014
Merged

Fix issue #6612: Find in Files panel doesn't reset scroll... #6629

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
ee35535
Scroll to top after the panel is visible
TomMalbran Jan 24, 2014
1c98d35
Merge from master
Jan 30, 2014
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion src/search/FindInFiles.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -418,7 +418,6 @@ define(function (require, exports, module) {
$searchContent
.empty()
.append(Mustache.render(searchResultsTemplate, {searchList: searchList, Strings: Strings}))
.scrollTop(0) // Otherwise scroll pos from previous contents is remembered
.off(".searchList") // Remove the old events

// Add the click event listener directly on the table parent
Expand Down Expand Up @@ -497,6 +496,7 @@ define(function (require, exports, module) {
$selectedRow = null;
}
searchResultsPanel.show();
$searchContent.scrollTop(0); // Otherwise scroll pos from previous contents is remembered

if (dialog) {
dialog._close();
Expand Down
1 change: 1 addition & 0 deletions src/search/FindReplace.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -587,6 +587,7 @@ define(function (require, exports, module) {
});

replaceAllPanel.show();
$replaceAllTable.scrollTop(0); // Otherwise scroll pos from previous contents is remembered
}

/** Shows the Find-Replace search bar at top */
Expand Down