From 4a1c9de869402f71d119ced5110b135ad5dff7ba Mon Sep 17 00:00:00 2001 From: Andreas Haas Date: Thu, 4 May 2017 22:01:26 +0200 Subject: [PATCH] Tree: Clear search string on selection. The Tree node has the ability to jump to a specific item by typing the first few chars of it's name. But on selection ('item_activated' signal), it didn't clear the search string used for that. It was especially annoying in `FileDialog`s and has been bugging me for ages :P With this, you can traverse a directory structure in a FileDiag quickly with the keyboard (like you'd expect from pretty much any modern file browser) :) (cherry picked from commit def41b985685a50385541d116ab62407bbd95c60) --- scene/gui/tree.cpp | 2 ++ 1 file changed, 2 insertions(+) diff --git a/scene/gui/tree.cpp b/scene/gui/tree.cpp index 99ac47d34be..60d0181971e 100644 --- a/scene/gui/tree.cpp +++ b/scene/gui/tree.cpp @@ -1489,6 +1489,7 @@ int Tree::propagate_mouse_event(const Point2i &p_pos, int x_ofs, int y_ofs, bool if (p_doubleclick && (!c.editable || c.mode == TreeItem::CELL_MODE_CUSTOM || c.mode == TreeItem::CELL_MODE_ICON /*|| c.mode==TreeItem::CELL_MODE_CHECK*/)) { //it' s confusing for check emit_signal("item_activated"); + incr_search.clear(); return -1; } @@ -2037,6 +2038,7 @@ void Tree::_input_event(InputEvent p_event) { //bring up editor if possible if (!edit_selected()) { emit_signal("item_activated"); + incr_search.clear(); } } accept_event();