zdoom/raze-gles
0
0
Fork 0
mirror of https://github.com/ZDoom/raze-gles.git synced 2025-01-12 11:10:39 +00:00
Code Issues Projects Releases Packages Wiki Activity

- "fixed" keybinding not responding for all keys.

Browse source 
This looks like an ImGui bug - it doesn't like two identically labelled buttons on the same page, it seems. I now work around this by using strings with different numbers of spaces. Stupid but effective.
This commit is contained in:
Christoph Oelckers 2019-11-14 22:48:23 +01:00
parent def63209be
commit 619a7541f1
1 changed files with 4 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
source/common/optionmenu/optionmenu.cpp
View file

@ -45,6 +45,7 @@ bool ShowOptionMenu()
ImGui::End(); ImGui::End();
return false; return false;
} }
FString emptiness = " ";
for (int i = 0; i < NUMGAMEFUNCTIONS; i++) for (int i = 0; i < NUMGAMEFUNCTIONS; i++)
{ {
@ -58,6 +59,9 @@ bool ShowOptionMenu()
al = buttonMap.GetButtonName(i); al = buttonMap.GetButtonName(i);
auto binds = Bindings.GetKeysForCommand(cmd); auto binds = Bindings.GetKeysForCommand(cmd);
al = C_NameKeys(binds.Data(), binds.Size()); al = C_NameKeys(binds.Data(), binds.Size());
// This looks like an ImGui bug - it doesn't like two identically labelled buttons on the same page, it seems.
// Work around this by using strings with different numbers of spaces. Stupid but effective.
if (al.IsEmpty()) { al = emptiness; emptiness << " "; }
if (ImGui::Button(al, ImVec2(450 * scale, 0))) if (ImGui::Button(al, ImVec2(450 * scale, 0)))
{ {
activeCommand = cmd; activeCommand = cmd;