Per Slot Text Toggle

[viciscat]

Moved Slot Text config to UI & Visuals and allowed configuring each adder individually

Adders just need to override a method that returns a ConfigInformation record.

Also I wanted to add a data fixer to move things but couldn't manage to do that :(

[kevinthegreat1]

The code looks fine. I am wondering if all that ConfigInformation is needed instead of just an id but should be fine.

Also cc @Emirlol if he wants to look at the api. The system is that SlotTextAdder gets config information from each slot text adder instance, than calls SlotTextManager with the config information to get whether it's enabled.

[Emirlol]

Looks pretty good. 2 things though:

• Perhaps the ConfigInformation could be stored in SimpleSlotTextAdder, which is passed in with the constructor with a default overriding method to retrieve it rather than each class handling it by itself.
• The way that the default value of true for enabled is hardcoded in multiple places (the createSlotTextToggles method and the getOrDefault calls) rather than just handing the value from the default config kinda feels unnecessary.

[viciscat]

Added constructor to SimpleSlotTextAdder
But I don't see what you mean for the defaults, the getOrDefault call is there in case it's null. And if there's a config default it won't be null, so it will take that.

[AzureAaron]

The type should be Object2BooleanOpenHashMap otherwise map accesses are being boxed, you will also need to use the getBoolean method when reading the options

addressed