Imgui: Remember position of movable overlays #13331
Conversation
Allow users to move the FPS, VPS, Speed, and Performance Graph overlays.
Remember the position of various moveable overlays so users don't have to move them every time they start a new game. Window data is stored in Config/ImGui.ini. Affected overlays: - NetPlay Chat - NetPlay Golf - Movie statistics - All of the overlays in the Graphics->Advanced tab - Scissor Rectangles
|
Been considering this for a long time because it's immensely useful for my editor but didn't know if we hadn't enabled it in our main widgets for some reason. Happy to see this. Thanks! |
|
I have a few quibblers with this. Can the player reset the window positions back to default? What happens if the windows are taken off-screen (due to resizing of the window or something)? |
A quick search seems to indicate they would have to delete the config file. We could add a dedicated method but seems like this problem is no different than our other configs. Maybe it'd be better to wait to have a generic solution for all?
The windows are currently bound to the render area. So if you resized the app window to be across multiple monitors, you'd have that whole space to move these windows around. |
|
As iwubcode mentioned, deleting the .ini file will revert the positions to their defaults. If you wanted to reset just one overlay you could delete only that section, and if you want to place the overlays really precisely you can manually edit their positions. Having a UI option for resetting the overlays would be nice, but the Advanced tab is rather crowded as it is so I don't know how you'd want to handle that. As for overlays going off-screen, I've gone into that here. |
(The first commit is just #13330. Making this a draft until that gets merged, but this PR is ready for review)
Remember the position of various moveable overlays so users don't have to move them every time they start a new game.
Window data is stored in Config/ImGui.ini.
Affected overlays:
I've tested everything except NetPlay Chat, but given that it uses the same ImGuiCond_FirstUseEver flag as the others I'd be surprised if it didn't work.