Mouse Tweaks

Mouse Tweaks

284M Downloads

Time stamp in config

016Nojr opened this issue ยท 12 comments

commented

Can you remove this? I dont see why it should be in there?

commented

timestamp

commented

Yeah, that's the default behavior of java.util.Properties, I don't see an easy way of disabling it.

commented

Not at home so can't check myself; could you show me what you're talking about? I don't think I explicitly store any timestamps into the config.

commented

then if you dont use comments, maybe you can use json as config?

commented

Why though? The current system using Properties is about as simple as it can possibly get from both the code and the user friendliness perspective. I don't see how this timestamp comment is such a big issue to change anything.

commented

Why arent you using forge's Configuration? It's much easier, has comment system and users are more used to it

commented

Well, at least because MT doesn't require Forge: it can be loaded with just LiteLoader. Since it's such a simple mod, I'd like to keep it this way.

commented

I think the issue was that we use git for modpack dev and this changing every time counts as git change, but I guess we could just add it to .ignore file

commented

Hmm I see. I'll mark this as enhancement then and see what I can do next time I update the mod. Maybe not save the file unless there's a change or not using Properties altogether.

commented

This file is supposed to be user/client only, not modpack? You could move it to local/mousetweaks.properties

commented

It is useful for modpacks: certain mods add their own wheel tweak functionality in which case the one in Mouse Tweaks needs to be manually disabled (by the modpack authors) to avoid conflicts. Also I've never seen that kind of folder structure.

commented

In the latest version I'm writing the config file manually, without any timestamps and with settings in a determined order.