Fix hud position reset#5385
Conversation
…ause it's already set up to not reset HUD position stuff.
|
Have you already confirmed that |
|
I did some manual testing here, confirmed that most of the stuff in the "Silly" tab does not get reset by the "Reset All" button. Is that intentional? |
|
No, that's just because the silly tab doesn't have an array of CVars used there, so it isn't hooked up to the old reset function. |
|
Any chance you could add them? Feels to me like they should be reset when you hit Reset All |
|
In my mind I think it should only reset colors. The silly tab should have its only reset. The accompanying RandomizeAll button only randomizes colors. We can always rename the funcs if that's an issue. |
|
Ah that is fair, wasn't thinking about the Randomize All button. |
…ause it's already set up to not reset HUD position stuff. (HarbourMasters#5385)
Changes the Reset All callback back to
CosmeticsEditor_ResetAll, because it's already set up to not reset HUD position stuff. I tried replacing stuff in that function with CVarClearBlock calls, but it reloads the CVars after each one because of how that flow works, and it's extremely unperformant.Build Artifacts