search

fluffy-mods / ColonyManager

Colony manager for RimWorld
Other
71 stars 39 forks source link

[BUG] Readme does not list mod dependencies #125

Closed serenewaffles closed 5 years ago

serenewaffles commented 5 years ago

Per Issue 124, properly displaying research options requires the user also have ResearchTree installed. The Readme and installation instructions should mention the requirement.

shiribailem commented 5 years ago

Technically not a dependency, you can still use it and the research options are usable... just not pretty.

Still, might be worthwhile to acknowledge, can't hurt to add a line of "As this game adds research options, a research tab mod like ResearchTree or ResearchPal is recommended."

Just fyi, all mods that add research options have issues, it's one area of the mod api that's sorely lacking and a research tab mod is recommended anytime you add items to the research tree because otherwise there's no way to prevent them from overlapping or really guarantee that they'll look decent at all.

serenewaffles commented 5 years ago

Please see RT Fuse for an example of a mod that adds research options, doesn't require another mod to display research options, and has a readable layout.

shiribailem commented 5 years ago

I've also used Colony Manager without a research tab mod before. You're making a mountain out of a molehill, it was just a little unsightly but definitely doesn't fall under "require another mod to display research options".

Keep in mind, I was still agreeing with your suggestion, just not as a dependency.

And acknowledging it in the README might help to avoid these threads.

(Also, if you haven't installed it yet, ResearchTree/ResearchPal is a huge QoL improvement)