gynt
commented
1 month ago Config merging is a GUI issue, not a module issue, aside from the fact that the module (this repo) needs to iterate over multiple map folders. This line needs to change to support it: https://github.com/UnofficialCrusaderPatch/extension-maploader/blob/main/init.lua#L5
There is currently no defined behaviour for merging multiple map loader instances from multiple plugins. I understand, that we want the neat array like availability in the GUI, but the GUI isn't our priority functionality is. -> Hence i would suggest, that we define a default behaviour for merging into lists in the config, when multiple folders come in, so that the map loader can load all of them -> Until we figure out a GUI for lists like that, we just show the top item of the list in the GUI -> Alternatively we could show a list in the config, but only if there is already a list present in the merged plugin config. To add a new item to the list, you either do it manually or just make another plugin as dependency.