godotengine/godot
1
0
Fork 0
mirror of https://github.com/godotengine/godot.git synced 2025-11-15 13:51:40 +00:00
Code Releases Wiki Activity

Allow selecting multiple files in the editor translation/remap dialogs

Browse Source 
This makes it faster to add several translations.

The undo/redo messages were also tweaked to give better context.
This commit is contained in:
Hugo Locurcio
2020-12-29 09:26:45 +01:00
parent edccc0bbdf
commit d4bd107fd5
2 changed files with 44 additions and 35 deletions
Download Patch File Download Diff File Expand all files Collapse all files

71
editor/localization_editor.cpp
View File

@@ -78,18 +78,21 @@ void LocalizationEditor::_notification(int p_what) {
} }
void LocalizationEditor::add_translation(const String &p_translation) { void LocalizationEditor::add_translation(const String &p_translation) {
_translation_add(p_translation); PackedStringArray translations;
translations.push_back(p_translation);
_translation_add(translations);
} }
void LocalizationEditor::_translation_add(const String &p_path) { void LocalizationEditor::_translation_add(const PackedStringArray &p_paths) {
PackedStringArray translations = ProjectSettings::get_singleton()->get("locale/translations"); PackedStringArray translations = ProjectSettings::get_singleton()->get("locale/translations");
if (translations.has(p_path)) { for (int i = 0; i < p_paths.size(); i++) {
return; if (!translations.has(p_paths[i])) {
// Don't add duplicate translation paths.
translations.push_back(p_paths[i]);
}
} }
translations.push_back(p_path); undo_redo->create_action(vformat(TTR("Add %d Translations"), p_paths.size()));
undo_redo->create_action(TTR("Add Translation"));
undo_redo->add_do_property(ProjectSettings::get_singleton(), "locale/translations", translations); undo_redo->add_do_property(ProjectSettings::get_singleton(), "locale/translations", translations);
undo_redo->add_undo_property(ProjectSettings::get_singleton(), "locale/translations", ProjectSettings::get_singleton()->get("locale/translations")); undo_redo->add_undo_property(ProjectSettings::get_singleton(), "locale/translations", ProjectSettings::get_singleton()->get("locale/translations"));
undo_redo->add_do_method(this, "update_translations"); undo_redo->add_do_method(this, "update_translations");
@@ -129,7 +132,7 @@ void LocalizationEditor::_translation_res_file_open() {
translation_res_file_open_dialog->popup_file_dialog(); translation_res_file_open_dialog->popup_file_dialog();
} }
void LocalizationEditor::_translation_res_add(const String &p_path) { void LocalizationEditor::_translation_res_add(const PackedStringArray &p_paths) {
Variant prev; Variant prev;
Dictionary remaps; Dictionary remaps;
@@ -138,13 +141,14 @@ void LocalizationEditor::_translation_res_add(const String &p_path) {
prev = remaps; prev = remaps;
} }
if (remaps.has(p_path)) { for (int i = 0; i < p_paths.size(); i++) {
return; //pointless already has it if (!remaps.has(p_paths[i])) {
// Don't overwrite with an empty remap array if an array already exists for the given path.
remaps[p_paths[i]] = PackedStringArray();
}
} }
remaps[p_path] = PackedStringArray(); undo_redo->create_action(vformat(TTR("Translation Resource Remap: Add %d Path(s)"), p_paths.size()));
undo_redo->create_action(TTR("Add Remapped Path"));
undo_redo->add_do_property(ProjectSettings::get_singleton(), "locale/translation_remaps", remaps); undo_redo->add_do_property(ProjectSettings::get_singleton(), "locale/translation_remaps", remaps);
undo_redo->add_undo_property(ProjectSettings::get_singleton(), "locale/translation_remaps", prev); undo_redo->add_undo_property(ProjectSettings::get_singleton(), "locale/translation_remaps", prev);
undo_redo->add_do_method(this, "update_translations"); undo_redo->add_do_method(this, "update_translations");
@@ -158,7 +162,7 @@ void LocalizationEditor::_translation_res_option_file_open() {
translation_res_option_file_open_dialog->popup_file_dialog(); translation_res_option_file_open_dialog->popup_file_dialog();
} }
void LocalizationEditor::_translation_res_option_add(const String &p_path) { void LocalizationEditor::_translation_res_option_add(const PackedStringArray &p_paths) {
ERR_FAIL_COND(!ProjectSettings::get_singleton()->has_setting("locale/translation_remaps")); ERR_FAIL_COND(!ProjectSettings::get_singleton()->has_setting("locale/translation_remaps"));
Dictionary remaps = ProjectSettings::get_singleton()->get("locale/translation_remaps"); Dictionary remaps = ProjectSettings::get_singleton()->get("locale/translation_remaps");
@@ -170,10 +174,12 @@ void LocalizationEditor::_translation_res_option_add(const String &p_path) {
ERR_FAIL_COND(!remaps.has(key)); ERR_FAIL_COND(!remaps.has(key));
PackedStringArray r = remaps[key]; PackedStringArray r = remaps[key];
r.push_back(p_path + ":" + "en"); for (int i = 0; i < p_paths.size(); i++) {
r.push_back(p_paths[i] + ":" + "en");
}
remaps[key] = r; remaps[key] = r;
undo_redo->create_action(TTR("Resource Remap Add Remap")); undo_redo->create_action(vformat(TTR("Translation Resource Remap: Add %d Remap(s)"), p_paths.size()));
undo_redo->add_do_property(ProjectSettings::get_singleton(), "locale/translation_remaps", remaps); undo_redo->add_do_property(ProjectSettings::get_singleton(), "locale/translation_remaps", remaps);
undo_redo->add_undo_property(ProjectSettings::get_singleton(), "locale/translation_remaps", ProjectSettings::get_singleton()->get("locale/translation_remaps")); undo_redo->add_undo_property(ProjectSettings::get_singleton(), "locale/translation_remaps", ProjectSettings::get_singleton()->get("locale/translation_remaps"));
undo_redo->add_do_method(this, "update_translations"); undo_redo->add_do_method(this, "update_translations");
@@ -381,17 +387,20 @@ void LocalizationEditor::_translation_filter_mode_changed(int p_mode) {
undo_redo->commit_action(); undo_redo->commit_action();
} }
void LocalizationEditor::_pot_add(const String &p_path) { void LocalizationEditor::_pot_add(const PackedStringArray &p_paths) {
PackedStringArray pot_translations = ProjectSettings::get_singleton()->get("locale/translations_pot_files"); PackedStringArray pot_translations = ProjectSettings::get_singleton()->get("locale/translations_pot_files");
for (int i = 0; i < pot_translations.size(); i++) { for (int i = 0; i < p_paths.size(); i++) {
if (pot_translations[i] == p_path) { for (int j = 0; j < pot_translations.size(); j++) {
return; //exists if (pot_translations[j] == p_paths[i]) {
continue; //exists
} }
} }
pot_translations.push_back(p_path); pot_translations.push_back(p_paths[i]);
undo_redo->create_action(TTR("Add files for POT generation")); }
undo_redo->create_action(vformat(TTR("Add %d file(s) for POT generation"), p_paths.size()));
undo_redo->add_do_property(ProjectSettings::get_singleton(), "locale/translations_pot_files", pot_translations); undo_redo->add_do_property(ProjectSettings::get_singleton(), "locale/translations_pot_files", pot_translations);
undo_redo->add_undo_property(ProjectSettings::get_singleton(), "locale/translations_pot_files", ProjectSettings::get_singleton()->get("locale/translations_pot_files")); undo_redo->add_undo_property(ProjectSettings::get_singleton(), "locale/translations_pot_files", ProjectSettings::get_singleton()->get("locale/translations_pot_files"));
undo_redo->add_do_method(this, "update_translations"); undo_redo->add_do_method(this, "update_translations");
@@ -685,8 +694,8 @@ LocalizationEditor::LocalizationEditor() {
translations->add_child(tvb); translations->add_child(tvb);
HBoxContainer *thb = memnew(HBoxContainer); HBoxContainer *thb = memnew(HBoxContainer);
thb->add_spacer();
thb->add_child(memnew(Label(TTR("Translations:")))); thb->add_child(memnew(Label(TTR("Translations:"))));
thb->add_spacer();
tvb->add_child(thb); tvb->add_child(thb);
Button *addtr = memnew(Button(TTR("Add..."))); Button *addtr = memnew(Button(TTR("Add...")));
@@ -702,8 +711,8 @@ LocalizationEditor::LocalizationEditor() {
tmc->add_child(translation_list); tmc->add_child(translation_list);
translation_file_open = memnew(EditorFileDialog); translation_file_open = memnew(EditorFileDialog);
translation_file_open->set_file_mode(EditorFileDialog::FILE_MODE_OPEN_FILE); translation_file_open->set_file_mode(EditorFileDialog::FILE_MODE_OPEN_FILES);
translation_file_open->connect("file_selected", callable_mp(this, &LocalizationEditor::_translation_add)); translation_file_open->connect("files_selected", callable_mp(this, &LocalizationEditor::_translation_add));
add_child(translation_file_open); add_child(translation_file_open);
} }
@@ -732,8 +741,8 @@ LocalizationEditor::LocalizationEditor() {
tmc->add_child(translation_remap); tmc->add_child(translation_remap);
translation_res_file_open_dialog = memnew(EditorFileDialog); translation_res_file_open_dialog = memnew(EditorFileDialog);
translation_res_file_open_dialog->set_file_mode(EditorFileDialog::FILE_MODE_OPEN_FILE); translation_res_file_open_dialog->set_file_mode(EditorFileDialog::FILE_MODE_OPEN_FILES);
translation_res_file_open_dialog->connect("file_selected", callable_mp(this, &LocalizationEditor::_translation_res_add)); translation_res_file_open_dialog->connect("files_selected", callable_mp(this, &LocalizationEditor::_translation_res_add));
add_child(translation_res_file_open_dialog); add_child(translation_res_file_open_dialog);
thb = memnew(HBoxContainer); thb = memnew(HBoxContainer);
@@ -764,8 +773,8 @@ LocalizationEditor::LocalizationEditor() {
tmc->add_child(translation_remap_options); tmc->add_child(translation_remap_options);
translation_res_option_file_open_dialog = memnew(EditorFileDialog); translation_res_option_file_open_dialog = memnew(EditorFileDialog);
translation_res_option_file_open_dialog->set_file_mode(EditorFileDialog::FILE_MODE_OPEN_FILE); translation_res_option_file_open_dialog->set_file_mode(EditorFileDialog::FILE_MODE_OPEN_FILES);
translation_res_option_file_open_dialog->connect("file_selected", callable_mp(this, &LocalizationEditor::_translation_res_option_add)); translation_res_option_file_open_dialog->connect("files_selected", callable_mp(this, &LocalizationEditor::_translation_res_option_add));
add_child(translation_res_option_file_open_dialog); add_child(translation_res_option_file_open_dialog);
} }
@@ -825,8 +834,8 @@ LocalizationEditor::LocalizationEditor() {
add_child(pot_generate_dialog); add_child(pot_generate_dialog);
pot_file_open_dialog = memnew(EditorFileDialog); pot_file_open_dialog = memnew(EditorFileDialog);
pot_file_open_dialog->set_file_mode(EditorFileDialog::FILE_MODE_OPEN_FILE); pot_file_open_dialog->set_file_mode(EditorFileDialog::FILE_MODE_OPEN_FILES);
pot_file_open_dialog->connect("file_selected", callable_mp(this, &LocalizationEditor::_pot_add)); pot_file_open_dialog->connect("files_selected", callable_mp(this, &LocalizationEditor::_pot_add));
add_child(pot_file_open_dialog); add_child(pot_file_open_dialog);
} }

8
editor/localization_editor.h
View File

@@ -72,22 +72,22 @@ class LocalizationEditor : public VBoxContainer {
String localization_changed; String localization_changed;
void _translation_file_open(); void _translation_file_open();
void _translation_add(const String &p_path); void _translation_add(const PackedStringArray &p_paths);
void _translation_delete(Object *p_item, int p_column, int p_button); void _translation_delete(Object *p_item, int p_column, int p_button);
void _translation_res_file_open(); void _translation_res_file_open();
void _translation_res_add(const String &p_path); void _translation_res_add(const PackedStringArray &p_paths);
void _translation_res_delete(Object *p_item, int p_column, int p_button); void _translation_res_delete(Object *p_item, int p_column, int p_button);
void _translation_res_select(); void _translation_res_select();
void _translation_res_option_file_open(); void _translation_res_option_file_open();
void _translation_res_option_add(const String &p_path); void _translation_res_option_add(const PackedStringArray &p_paths);
void _translation_res_option_changed(); void _translation_res_option_changed();
void _translation_res_option_delete(Object *p_item, int p_column, int p_button); void _translation_res_option_delete(Object *p_item, int p_column, int p_button);
void _translation_filter_option_changed(); void _translation_filter_option_changed();
void _translation_filter_mode_changed(int p_mode); void _translation_filter_mode_changed(int p_mode);
void _pot_add(const String &p_path); void _pot_add(const PackedStringArray &p_paths);
void _pot_delete(Object *p_item, int p_column, int p_button); void _pot_delete(Object *p_item, int p_column, int p_button);
void _pot_file_open(); void _pot_file_open();
void _pot_generate_open(); void _pot_generate_open();