[VE] Fix Frames/Output folders not saving from config menu #152

Merged
vea-sa merged 1 commit from ve-config-fix into main 2025-06-20 17:16:59 +00:00
vea-sa commented 2025-06-20 16:35:03 +00:00 (Migrated from github.com)

Fixes https://github.com/IllusionMods/HSPlugins/issues/150.

I think this issue has been there for a very long time. In fact, almost the entire config is not bound properly using ConfigEntry<>.

It exposes many internal values (e.g. indexes) that should stay hidden and not pollute the F1 menu. I'll review the entire config later when I find the time.

Fixes https://github.com/IllusionMods/HSPlugins/issues/150. I think this issue has been there for a very long time. In fact, almost the entire config is not bound properly using `ConfigEntry<>`. It exposes many internal values (e.g. indexes) that should stay hidden and not pollute the F1 menu. I'll review the entire config later when I find the time.
ManlyMarco commented 2025-06-20 17:16:34 +00:00 (Migrated from github.com)

Plugins in this repo were all using some sort of abstraction layer for settings because of originally having IPA compatibility, so there's likely a lot more of this sort of issues. Removing the abstraction would be the easiest way to find code that is likely buggy.

Plugins in this repo were all using some sort of abstraction layer for settings because of originally having IPA compatibility, so there's likely a lot more of this sort of issues. Removing the abstraction would be the easiest way to find code that is likely buggy.
ManlyMarco (Migrated from github.com) approved these changes 2025-06-20 17:16:52 +00:00
Sign in to join this conversation.
No description provided.