Restore previous scroll position when reopening the search help dialog with a old search
This commit is contained in:
parent
942855a9ee
commit
78b9b7c42b
2 changed files with 12 additions and 2 deletions
|
@ -140,7 +140,15 @@ void EditorHelpSearch::_notification(int p_what) {
|
||||||
if (search.is_valid()) {
|
if (search.is_valid()) {
|
||||||
if (search->work()) {
|
if (search->work()) {
|
||||||
// Search done.
|
// Search done.
|
||||||
|
|
||||||
|
// Only point to the perfect match if it's a new search, and not just reopening a old one.
|
||||||
|
if (!old_search)
|
||||||
|
results_tree->ensure_cursor_is_visible();
|
||||||
|
else
|
||||||
|
old_search = false;
|
||||||
|
|
||||||
get_ok()->set_disabled(!results_tree->get_selected());
|
get_ok()->set_disabled(!results_tree->get_selected());
|
||||||
|
|
||||||
search = Ref<Runner>();
|
search = Ref<Runner>();
|
||||||
set_process(false);
|
set_process(false);
|
||||||
}
|
}
|
||||||
|
@ -177,6 +185,7 @@ void EditorHelpSearch::popup_dialog(const String &p_term) {
|
||||||
if (p_term == "") {
|
if (p_term == "") {
|
||||||
search_box->clear();
|
search_box->clear();
|
||||||
} else {
|
} else {
|
||||||
|
old_search = true;
|
||||||
search_box->set_text(p_term);
|
search_box->set_text(p_term);
|
||||||
search_box->select_all();
|
search_box->select_all();
|
||||||
}
|
}
|
||||||
|
@ -186,6 +195,8 @@ void EditorHelpSearch::popup_dialog(const String &p_term) {
|
||||||
|
|
||||||
EditorHelpSearch::EditorHelpSearch() {
|
EditorHelpSearch::EditorHelpSearch() {
|
||||||
|
|
||||||
|
old_search = false;
|
||||||
|
|
||||||
set_hide_on_ok(false);
|
set_hide_on_ok(false);
|
||||||
set_resizable(true);
|
set_resizable(true);
|
||||||
set_title(TTR("Search Help"));
|
set_title(TTR("Search Help"));
|
||||||
|
@ -406,8 +417,6 @@ bool EditorHelpSearch::Runner::_phase_select_match() {
|
||||||
|
|
||||||
if (matched_item)
|
if (matched_item)
|
||||||
matched_item->select(0);
|
matched_item->select(0);
|
||||||
results_tree->ensure_cursor_is_visible();
|
|
||||||
|
|
||||||
return true;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
@ -58,6 +58,7 @@ class EditorHelpSearch : public ConfirmationDialog {
|
||||||
ToolButton *hierarchy_button;
|
ToolButton *hierarchy_button;
|
||||||
OptionButton *filter_combo;
|
OptionButton *filter_combo;
|
||||||
Tree *results_tree;
|
Tree *results_tree;
|
||||||
|
bool old_search;
|
||||||
|
|
||||||
class Runner;
|
class Runner;
|
||||||
Ref<Runner> search;
|
Ref<Runner> search;
|
||||||
|
|
Loading…
Reference in a new issue