epic/OpenRA
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Add duplicate hotkey tracking with a boolean property on the definition

Browse Source
This commit is contained in:
Ivaylo Draganov
2019-09-10 00:30:53 +03:00
committed by abcdefg30
parent df2300bee0
commit 32309bb8ea
2 changed files with 8 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
OpenRA.Game/HotkeyDefinition.cs
View File

@@ -20,6 +20,7 @@ namespace OpenRA
public readonly Hotkey Default = Hotkey.Invalid;
public readonly string Description = "";
public readonly HashSet<string> Types = new HashSet<string>();
public bool HasDuplicates = false;
public HotkeyDefinition(string name, MiniYaml node)
{

10
OpenRA.Mods.Common/Widgets/Logic/SettingsLogic.cs
View File

@@ -149,9 +149,7 @@ namespace OpenRA.Mods.Common.Widgets.Logic
remapButton.GetColor = () =>
{
return modData.Hotkeys.GetFirstDuplicate(hd.Name, modData.Hotkeys[hd.Name].GetValue(), hd) != null ?
hotkeyInvalidColor :
hotkeyValidColor;
return hd.HasDuplicates ? hotkeyInvalidColor : hotkeyValidColor;
};
if (selectedHotkeyDefinition == hd)
@@ -492,6 +490,9 @@ namespace OpenRA.Mods.Common.Widgets.Logic
if (selectedHotkeyDefinition == null)
selectedHotkeyDefinition = hd;
if (modData.Hotkeys.GetFirstDuplicate(hd.Name, modData.Hotkeys[hd.Name].GetValue(), hd) != null)
hd.HasDuplicates = true;
BindHotkeyPref(hd, template, hotkeyList);
}
}
@@ -800,6 +801,9 @@ namespace OpenRA.Mods.Common.Widgets.Logic
WidgetUtils.TruncateButtonToTooltip(selectedHotkeyButton, hotkeyEntryWidget.Key.DisplayString());
modData.Hotkeys.Set(selectedHotkeyDefinition.Name, hotkeyEntryWidget.Key);
Game.Settings.Save();
foreach (var hd in modData.Hotkeys.Definitions)
hd.HasDuplicates = modData.Hotkeys.GetFirstDuplicate(hd.Name, modData.Hotkeys[hd.Name].GetValue(), hd) != null;
}
void ResetHotkey()