Dynamic FPS

Dynamic FPS

16M Downloads

Alternative version without cloth config included?

K0-RR opened this issue · 5 comments

commented

Most people already have cloth config like 30 times because of other mods. Can you release an additional version without it?
Maybe you could switch cloth for some other config system ex. Omega config?

commented

I've been weighing the pros and cons of requiring cloth as an external dependency rather than embedding it. I suppose the mod already requires Fabric API, and cloth tends to be the main thing that breaks with updates while the mod itself just keeps working. #29 recently addressed this but was closed by the author in favor of Quilt's upcoming built-in config system.

commented

Another thing that could be done is have the dependency optional, since you still can change the settings in the config file (while not instant). For ModMenu compat you'd just make the method return null (or a provider that returns null) if cloth isn't loaded.

commented

Yeah, that's a great idea. I'm honestly not sure why I didn't do it sooner—I think I was scared it might break regular config functionality, but it doesn't seem to. I think I will indeed not be jar-in-jaring cloth config anymore—here's a pre-release of sorts without it, lol (still can't believe GitHub doesn't let you upload raw JAR files).
dynamic-fps-2.0.6.jar.zip

commented

Why not release it for everyone then?

commented

I'd assume it's here so the issue authors (and anyone interested) can test it first, and then the release will come, just like in previous issues.