Add vertical view distance
NotStirred opened this issue ยท 15 comments
This is not as easy as it may seem. CubeDistanceGraph doesn't really work that way.
What would be the best way to store a vertical view distance client side? Forge config file? options.txt?
The target is forge config file but there is a problem with that:
in 1.12.2 I could modify forge config from code and store it back. So I could have the vertical view distance slider modify the config value and store it.
Modifying config values from code is no longer possible. So right now just make some dummy setter and getter for it in the config object and leave it to figure out later.
Correction: I read the code now and apparently sometime between last time I looked at it and now an option to set config values programatically has been added.
And no you shouldn't use EarlyConfig for that. EarlyConfig has one purpose: configuring stuff for mixin and stuff that needs to be loaded arbitrarily early. Vertical view distance can be just standard forge client config.
Okay, I've got vertical view & render distance working. Sort of. This is a hack if you've ever seen one, so be aware, but I've got a prototype.
Commit: AidanLovelace@ed9d112
updated and merged in d41a712