-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Stuffs to do #6
Labels
help wanted
Extra attention is needed
Comments
Have you gathered enough interest? |
@JustSimplyKyle Similar mod by me https://modrinth.com/mod/mod-sets |
Thanks! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Kept here to replace
TODO.md
in the repo. Items marked "(?)" are still under consideration.Emergency changesFabric Loader v0.12.0 introduces a massive refactor on mod loading mechanics, which breaks ThatOrThis completely. A rewrite needs to be done before it's too late. As the rewrite will break compatibility with Fabric Loader v0.11.x, support for these versions will be dropped on ThatOrThis v0.3.0.
FabricInternals
to keep only interfacing code and move inner classes to a new package (commit de94067)Stuffs to a better & cleaner config schema: (Include in v0.3.0?)
GENERATED
rules' buttons (text frommodmenu.descriptionTranslation.XXX
? Format?)Check forLanguageAdapter
s and GrossFabricHacks entrypoints on injecting a mod, and if found, issue a non-fatal error (LanguageAdapter found in "xxx-language-support"! Expect it to break
)modmenu.parent
; need more reflection hacks)ArgumentModCandidateFinder
(Add --fabric.addMods / -Dfabric.addMods for adding extra path separator separated mods, list file if prefixed with @ FabricMC/fabric-loader#470)@xxx.xxx
hack with JSON-based text (Text net.minecraft.text.Text.Serializer.fromJson(String)
)RuleHolder
to be like a container, similar toChoiceHolder
? (Good for consistency but otherwise unnecessary for now)disabled
intoChoiceHolder
(Not possible for now due toMap<>
inheritance, but should be done)CyclingButtonWidget
)The text was updated successfully, but these errors were encountered: