Conversation
| // Parse attachment nodes | ||
| if (!string.IsNullOrWhiteSpace(requiredNodes)) | ||
| { | ||
| foreach (string nodeName in requiredNodes.Split(',')) |
There was a problem hiding this comment.
How about allowing the requiredNodes key to be specified more than once, rather than splitting by commas? That tends to work better with MM patches.
There was a problem hiding this comment.
That does sound a bit more standard.
The commas are a leftover from the previous module version which clustered every node group into one. The commas were used like seperators in parallel lists to have them paired up.
There was a problem hiding this comment.
The transform list is liable to become quite long and other mods take the comma approach(well, semicolon, but I'd changed the split identifier just now)
Working from patches its'a already very flexible
Module is event driven and doesn't run inflight. disabled the frame behaviour
|
Put a hold on this, I wish to add localisation support for the display name |
The display name for the attachment visuals didn't have loc reference support isEnabled had turned off the capping toggle added general loc references using squad cfgs
|
@JonnyOThan Appropriate changes made and the missing loc support has been added |
Added new modules with descriptions for ModuleAttachmentVisuals, ModuleExclusiveResourceConverter, and ModuleToggleTracking.
Update README with new module descriptions
2 participants
Uh oh!
There was an error while loading. Please reload this page.