//todo

• padding
• keyboard key multi mapping is borked?
• analog stick options expand by default (also make it look better collapsed - any advice on this would be great!)
• rename window to be different than it was before so imgui.ini isn't an issue (handled this by using a hidden id)
• only show link and ivan tabs when debug is disabled, show all 4 when debug enabled (and show all buttons - not just ivan - when debug)
• slider +/- buttons
• clear all etc. buttons not super visible

edit: I decided to just default them to open for now, this will still be a useful comment when we start adding more analog stick options

@aMannus re:

analog stick options expand by default (also make it look better collapsed - any advice on this would be great!)

when messing around with this I found a few issues:

the issues

adding

ImGui::SetNextItemOpen(true, ImGuiCond_FirstUseEver);

or

ImGui::SetNextItemOpen(true, ImGuiCond_Once);

before

if (ImGui::TreeNode(StringHelper::Sprintf("Analog Stick Options##%d", id).c_str())) {

will make it default to open, but it will always default to open.

the expanded/collapsed state isn't saved in imgui.ini. Closing and reopening the window within the same session is fine, but after exiting and reopening the application it will go back to defaults.

after looking into this a bit more, it seems to also be a problem with all the CollapsingHeader sections too, such as

if (ImGui::CollapsingHeader("Buttons", NULL, ImGuiTreeNodeFlags_DefaultOpen)) {

ideas/paths forward

my gut feeling is that "i told you to go away" is a more frustrating user experience than "i thought you were open" when it comes to those options, but i could be convinced otherwise.

taking that into account i see a few options:

• manually save the expanded/collapsed state of every treenode/collapsingheader
  • if we did this defaulting to open would feel fine to me, closing it will keep it closed
  • this feels like it would involve adding a ton of UI cvars, which doesn't feel ideal implementation-wise
• default to open, have it reopen on people when they restart the application, tell them that's just how it is
• continue to default to close, make it so it is less hidden when closed

I'm leaning towards the "continue to default to close" option for a couple reasons

• once we add stuff like sensitivity and ESS range and other stuff the options section will get bigger, having it default to open could be overwhelming for people that just want to map a stick
• i feel like we should figure out why imgui doesn't let us persist expanded/collapsed instead of trying to work around it with cvars, that may take a bit of time/effort and i'd rather do that than build out a ton of cvars with the hope they'll be temporary

any thoughts/suggestions would be awesome!

Verified all my feedback has been accounted for or fixed 👍