godotengine
/
godot
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fixed "Open" button being enabled when nothing is selected in a FileDialog while in "Open folder" mode. 

(cherry picked from commit db80d56388)
This commit is contained in:
Michael Alexsander Silva Dias 2018-02-15 00:44:11 -02:00 committed by Hein-Pieter van Braam
parent e1fc7776f6
commit 23b9f09c58
1 changed files with 6 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
scene/gui/file_dialog.cpp
View File

@ -287,7 +287,7 @@ bool FileDialog::_is_open_should_be_disabled() {
TreeItem *ti = tree->get_selected(); TreeItem *ti = tree->get_selected();
// We have something that we can't select? // We have something that we can't select?
if (!ti) if (!ti)
return true; return mode != MODE_OPEN_DIR; // In "Open folder" mode, having nothing selected picks the current folder.
Dictionary d = ti->get_metadata(0); Dictionary d = ti->get_metadata(0);
@ -320,16 +320,14 @@ void FileDialog::deselect_items() {
case MODE_OPEN_FILE: case MODE_OPEN_FILE:
case MODE_OPEN_FILES: case MODE_OPEN_FILES:
get_ok()->set_text(RTR("Open")); get_ok()->set_text(RTR("Open"));
get_ok()->set_disabled(false);
break; break;
case MODE_OPEN_DIR: case MODE_OPEN_DIR:
get_ok()->set_text(RTR("Select Current Folder")); get_ok()->set_text(RTR("Select Current Folder"));
get_ok()->set_disabled(false);
break; break;
} }
} }
} }
void FileDialog::_tree_selected() { void FileDialog::_tree_selected() {
TreeItem *ti = tree->get_selected(); TreeItem *ti = tree->get_selected();
@ -756,7 +754,7 @@ void FileDialog::_bind_methods() {
ClassDB::bind_method(D_METHOD("_unhandled_input"), &FileDialog::_unhandled_input); ClassDB::bind_method(D_METHOD("_unhandled_input"), &FileDialog::_unhandled_input);
ClassDB::bind_method(D_METHOD("_tree_selected"), &FileDialog::_tree_selected); ClassDB::bind_method(D_METHOD("_tree_selected"), &FileDialog::_tree_selected);
ClassDB::bind_method(D_METHOD("_tree_db_selected"), &FileDialog::_tree_dc_selected); ClassDB::bind_method(D_METHOD("_tree_dc_selected"), &FileDialog::_tree_dc_selected);
ClassDB::bind_method(D_METHOD("_dir_entered"), &FileDialog::_dir_entered); ClassDB::bind_method(D_METHOD("_dir_entered"), &FileDialog::_dir_entered);
ClassDB::bind_method(D_METHOD("_file_entered"), &FileDialog::_file_entered); ClassDB::bind_method(D_METHOD("_file_entered"), &FileDialog::_file_entered);
ClassDB::bind_method(D_METHOD("_action_pressed"), &FileDialog::_action_pressed); ClassDB::bind_method(D_METHOD("_action_pressed"), &FileDialog::_action_pressed);
@ -881,7 +879,7 @@ FileDialog::FileDialog() {
filter = memnew(OptionButton); filter = memnew(OptionButton);
filter->set_stretch_ratio(3); filter->set_stretch_ratio(3);
filter->set_h_size_flags(SIZE_EXPAND_FILL); filter->set_h_size_flags(SIZE_EXPAND_FILL);
filter->set_clip_text(true); //too many extensions overflow it filter->set_clip_text(true); // too many extensions overflows it
hbc->add_child(filter); hbc->add_child(filter);
vbc->add_child(hbc); vbc->add_child(hbc);
@ -890,9 +888,8 @@ FileDialog::FileDialog() {
_update_drives(); _update_drives();
connect("confirmed", this, "_action_pressed"); connect("confirmed", this, "_action_pressed");
//cancel->connect("pressed", this,"_cancel_pressed");
tree->connect("cell_selected", this, "_tree_selected", varray(), CONNECT_DEFERRED); tree->connect("cell_selected", this, "_tree_selected", varray(), CONNECT_DEFERRED);
tree->connect("item_activated", this, "_tree_db_selected", varray()); tree->connect("item_activated", this, "_tree_dc_selected", varray());
tree->connect("nothing_selected", this, "deselect_items"); tree->connect("nothing_selected", this, "deselect_items");
dir->connect("text_entered", this, "_dir_entered"); dir->connect("text_entered", this, "_dir_entered");
file->connect("text_entered", this, "_file_entered"); file->connect("text_entered", this, "_file_entered");
@ -922,7 +919,6 @@ FileDialog::FileDialog() {
exterr->set_text(RTR("Must use a valid extension.")); exterr->set_text(RTR("Must use a valid extension."));
add_child(exterr); add_child(exterr);
//update_file_list();
update_filters(); update_filters();
update_dir(); update_dir();