Show module configuration summary on chassis pipe GUI
Indemnity83 opened this issue ยท 5 comments
The ability on modules is very useful when they're in your inventory, but as soon as you put a module in a chassis pipe you can no longer use this feature.
Obviously, you can click the [!] to see the configuration, but sometimes you just need to quickly check a couple chassis to make sure the settings are like you expected. It would be nice (and there appears to be space) to show the config summary in the chassis GUI.
"Needs Comment"?
Well, I actually independently thought this a couple days ago.
If you open a LP chassis, and press crouch with your mouse over a module in your inventory, it will indeed show the tooltip with configuration information, but as soon as you put it in the module slot, this information does not show up while pressing crouch.
Would definitely be a nice quality-of-life improvement.
take a look at this https://bitbucket.org/ProfMobius/waila/src/41a40b4fee90488a76f25204d3bcea6e9984e4cd/src/main/java/mcp/mobius/waila/api/IWailaDataProvider.java?at=forge_1187&fileviewer=file-view-default
w/ that you can make nbt data available on the client
NBTTagCompound getNBTData(EntityPlayerMP player, TileEntity te, NBTTagCompound tag, World world, int x, int y, int z);
maybe don't remove it from the item then when it's in the pipe but set a flag in the nbt instead?
fyi the "Needs Comments" is basically like a "Considering" tag, but primarily we want to have input from everyone to the topic.
I like the idea as well, but I think it would mean that we have to draw our own tooltip there, right @davboecki ?
The Idea is good, but implementing this is a problem.
The reason that the tooltip doesn't show up inside the chassis pipe is not that it's not implemented, it's casued by the fact that the informations aren't available. When you take a module out of a chassis pipe it's configuration is written to the items NBT and when you place it into a pipe it's loaded from NBT and then removed. The NBT information are synced by minecraft itself. So we would need to come up with a way of having those information on the client and then we can modify the tooltip to show them.