Browse Source

Merge pull request #76523 from Tefatika/feature/tefatika/76454-command-palette-use-key-name

Command Palette search now also uses original English command names
Rémi Verschelde 2 years ago
parent
commit
e6dba24a26
1 changed files with 8 additions and 2 deletions
  1. 8 2
      editor/editor_command_palette.cpp

+ 8 - 2
editor/editor_command_palette.cpp

@@ -75,9 +75,15 @@ void EditorCommandPalette::_update_command_search(const String &search_text) {
 		r.shortcut_text = E.value.shortcut;
 		r.shortcut_text = E.value.shortcut;
 		r.last_used = E.value.last_used;
 		r.last_used = E.value.last_used;
 
 
-		if (search_text.is_subsequence_ofn(r.display_name)) {
+		bool is_subsequence_of_key_name = search_text.is_subsequence_ofn(r.key_name);
+		bool is_subsequence_of_display_name = search_text.is_subsequence_ofn(r.display_name);
+
+		if (is_subsequence_of_key_name || is_subsequence_of_display_name) {
 			if (!search_text.is_empty()) {
 			if (!search_text.is_empty()) {
-				r.score = _score_path(search_text, r.display_name.to_lower());
+				float key_name_score = is_subsequence_of_key_name ? _score_path(search_text, r.key_name.to_lower()) : .0f;
+				float display_name_score = is_subsequence_of_display_name ? _score_path(search_text, r.display_name.to_lower()) : .0f;
+
+				r.score = MAX(key_name_score, display_name_score);
 			}
 			}
 
 
 			entries.push_back(r);
 			entries.push_back(r);