Conversation
microlith57
left a comment
microlith57
left a comment
There was a problem hiding this comment.
looks good! will test shortly
| and | ||
| -- only show leaf mods | ||
| (not scene.onlyShowLeafMods | ||
| or not next(scene.modDependents[mod.info.Name] or {})) |
There was a problem hiding this comment.
it would be nice if this didn't hide favourited mods, since that way favouriting a mod acts as a way for the user to express that they use it for something on top of it just being a dependency
for instance, extvars is commonly used as a dependency, but players may care about its standalone functionality and therefore want to see it in the list
There was a problem hiding this comment.
I agree, I hadn't thought of whether users would favorite dependencies or not. I included the feature but I'm not too sure how to add it to this PR since I am relatively new to making PR requests.
However, replacing the highlighted text with this block adds the functionality to keep favourited mods shown.
(not scene.onlyShowLeafMods
or (not next(scene.modDependents[mod.info.Name] or {})
or (mod.info.IsFavorite)))
2 participants
Adds a checkbox to hide dependencies.
Closes #162