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

Merge pull request #65957 from EricEzaM/GH-65956-fix-ed-shortcut-section-order 

Update Editor Shortcuts tree creation to always have sections in same order
This commit is contained in:
Rémi Verschelde 2022-09-21 00:00:45 +02:00 committed by GitHub
parent c44e1c4e31 722378479d
commit 1a0943e0f8
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 31 additions and 26 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

27
editor/editor_settings_dialog.cpp
View File

@ -416,25 +416,21 @@ void EditorSettingsDialog::_update_shortcuts() {
List<String> slist;
EditorSettings::get_singleton()->get_shortcut_list(&slist);
slist.sort(); // Sort alphabetically.
const EditorPropertyNameProcessor::Style name_style = EditorPropertyNameProcessor::get_settings_style();
const EditorPropertyNameProcessor::Style tooltip_style = EditorPropertyNameProcessor::get_tooltip_style(name_style);
// Create all sections first.
for (const String &E : slist) {
Ref<Shortcut> sc = EditorSettings::get_singleton()->get_shortcut(E);
if (!sc->has_meta("original")) {
continue;
}
// Shortcut Section
TreeItem *section;
String section_name = E.get_slice("/", 0);
if (sections.has(section_name)) {
section = sections[section_name];
} else {
section = shortcuts->create_item(root);
continue;
}
TreeItem *section = shortcuts->create_item(root);
const String item_name = EditorPropertyNameProcessor::get_singleton()->process_name(section_name, name_style);
const String tooltip = EditorPropertyNameProcessor::get_singleton()->process_name(section_name, tooltip_style);
@ -453,8 +449,17 @@ void EditorSettingsDialog::_update_shortcuts() {
sections[section_name] = section;
}
// Shortcut Item
// Add shortcuts to sections.
for (const String &E : slist) {
Ref<Shortcut> sc = EditorSettings::get_singleton()->get_shortcut(E);
if (!sc->has_meta("original")) {
continue;
}
String section_name = E.get_slice("/", 0);
TreeItem *section = sections[section_name];
// Shortcut Item
if (!shortcut_filter.is_subsequence_ofn(sc->get_name())) {
continue;
}