[drawers] support more info for single drawer cabinets#52
Conversation
|
IMO it wouldn't be too much if they were split across multiple lines, something like this: Could even put the quantities, but maybe that would be too much. |
With 1x2 it seems fine, but just imagine 4 lines on a small screen. |
|
Also it may be nicer to have it limited to the item name (no "xyz wooden drawer"). Edit: Just like with technic chests. If nobody is against this idea, I'm going to do this making the whole PR smaller :D |
Co-authored-by: OgelGames <olliverdc28@gmail.com>
This reverts commit 8e928c0.
Count + short item description (or item name) Counts >= 1000 -> 1k
|
After trying several options I feel this is the most useable:
|
|
I've tried supporting 1x2 and 2x2 cabinets too: https://github.com/mt-mods/wrench/tree/drawerLabel2 The tooltip is kinda useable but the short description cuts off multiple lines, so the hotbar text isn't satisfactory. |
I don't like the formatting of that, it looks messy:
I tried my idea (tree/drawers_info), which looks like this:
I think the size is okay, it's bigger than most tooltips but still only slightly taller than an inventory slot, and much easier to read. |
|
Sure, one line per item is neater but short description will still be "Wooden Drawers ...." and no actual info. Which pretty much defeats the purpouse of this idea. |
|
Why does the short description matter? |
|
Because that is the info shown on hotbar when changing item. Imagine having 8 wrenched drawers on hotbar and you want to place them in a specific order but all it says is "Wooden drawer with items" i.e. same as what it says now. Edit: at this stage, the player is not interested in the "wooden drawer" part, they know that by now, they have filled and wrenched it. What they need is info on the content type. |
|
Can we get the single drawer merged and discuss the format for multi-drawer cabinets in another PR? |
|
What if we made it possible to set the short description too? It wouldn't be too hard to do, and then it could be different. For multi item drawers the short description could just specify the first item and combine the others, something like "Drawer with 99 Cobblestone + 2.5k other items".
I'd rather get it all done in one PR. |
|
Turns out the |
|
https://github.com/minetest-mods/unified_inventory/blob/0db7e7ba4cbee9bcbea9e1136da07fbcadb55fb8/item_names.lua#L62 and following (I really think we should merge this as title says: just single drawers and figure this part out in the longer term) |
|
Oh it's not even in the engine... That makes it easier to change though. |


The other drawer types were left out for reasons:
When need arrises, we can then search for a solution for multi-drawer cabinets.