WerDei / Biome-Replacer

MIT License
3 stars 3 forks source link

A GUI #15

Open justendmeplease opened 4 days ago

justendmeplease commented 4 days ago

While the way it is is tolerable it is far from desired imo. A great improvement would be a GUI ingame for easier changes. A bonus suggestion is a way to just disable biomes as that would be much easier to make ocean only worlds that dont suck. Love the mod!

WerDei commented 4 days ago

Hi, thanks for opening the issue! Please excuse me for the yapping ahead.

I thought about adding a GUI when I first made the mod, but decided not to since most people won't be changing config often, and I also had no experience making any GUIs for Minecraft. I still think it's a bit overkill.

But a next best thing might be to add a command for changing rules. It's simpler to do, and it can suggest biomes/tags as you type. It could also serve as a stepping stone to adding something like a settings screen later down the line, if need be.

Removing biomes altogether is a popular request but, if I remember correctly, this is not at all a good idea since it could mess up the distribution of other biomes, or even straight up crash the game. But I don't see why I can't try adding it anyway, with a proper warning to let people know it's an unsafe/advanced feature.

I'm gonna create new issues for these ideas, and leave this one open to talk about GUI specifically.

LegendarySpy commented 4 days ago

Dev of the NeoForge port here - I did some research when I saw this message earlier. If we were to implement it we could definitely follow the approach that structurify did, which uses YACL (Yet another config lib) for the UI. Luckily YACL is Fabric and Neoforge compatible so it would mainly be the same config menu and code.

It's just important that this config option writes to the config file (Im not totally sure how it works in the backend) because we still need to have support for servers and YACL is meant for clients.

WerDei commented 4 days ago

we still need to have support for servers and YACL is meant for clients.

That's another reason why a command would be a better option, since it would "just work" on servers.