-
Notifications
You must be signed in to change notification settings - Fork 108
I'm Landmaster, and I'm volunteering to tackle the ≥1.14.4 update. #612
Comments
A billion times yes! |
The biggest comment I'll add here is that Inventory Tweaks does a ton more than just a simple static sort -- the order depends on client configuration (the tree files), for the main inventory there's the whole second layer of being able to force specific items into specific slots, etc. This means it's not really well-suited to being implemented entirely on the server, unlike, say cpw's simpler sorter, at least without spending a ton of effort to make sure everyone's configurations get synced to the server somehow. (This goes kinda doubly if you end up implementing something like the API used to support the old AE2 mode where it essentially has to have an ordering function on the client inside AE2's list sorting) |
@Kobata Fair enough. The translation key sorting that I have so far is only temporary and I'll definitely replace it with a customizable config-based one once I get the basic functionality working. |
Synchronizing custom configs with the server seems possible, so maybe a server-based implementation can be a good approach. While very different, it would probably be more stable, performant and resistant to Minecraft API changes so I'm not against the idea. I've never written a server mod but if I get this well, the main downside is just that the mod would only work for local games + servers where the mod is installed on both the client and server, instead of working everywhere by default. With also the concern of packet retrocompatibility between client & server I think. Thanks for tackling this and best wishes for the rewrite 😄 (PS. When you get to custom configs, I hope using the same file names and format will be considered! Making IT Reborn a drop-in replacement without having players rewrite the config would be a killer feature) |
I've already pushed a few beta builds for the rewrite, though I'm waiting for CurseForge to approve the project. |
First builds are released at https://www.curseforge.com/minecraft/mc-mods/it-reborn! |
Like the idea, don't like that its stuck with needing to be installed client side, makes it kinda useless for me, I don't play singleplayer or on servers where I can freely add mods to them |
Anything I can do to help? give me instructions api links for tasks that need to be written/converted, documentation or test suites to write and i got you :p I'm just not really up for handling and investigating on my own at this moment without some encouragement/motivation/interest |
Hi Landmaster, I was looking for small open source mod to get help contribute to in order to expand my expertise with Java and I loved using inventory tweaks, I'd like to help bring it up to date. |
@TheBeardedQuack Check out the project's GitHub at https://github.com/Landmaster/InvTweaksReborn Also, Inventory Tweaks is now on 1.15.2! |
Ah, man. The entire charm of InvTweaks was that it could be used client-side; I was looking forward to the idea of being able to use this on a realm. That sucks. At least it's being worked on again. Good effort. |
@Vulano The client-side functionality is on my to-do list (https://github.com/Landmaster/InvTweaksReborn/issues/8). |
I'd be more then happy to rewrite InventoryTweaks from the ground up for compatibility with Fabric. |
Hello. I'm Landmaster, and you may know me as the creator of the Tinkers addon PlusTiC. I saw that you were looking for someone to bring this mod to 1.14.4 and later versions, and I'm here to ask if I can help create a rewrite of this mod in order to port it to said versions. So far, I've been able to get buttons and a basic (i.e. sorting by translation key) sorting implementation working, and I've created a repository containing my early work on the port that I started yesterday. If you guys have any questions/concerns, please feel free to reply to this issue.
BTW, thanks for the work that you've done for modded Minecraft over the years!
Kind regards,
Landmaster
Eminentia ex minimis
The text was updated successfully, but these errors were encountered: