Auroali / armour-bundles

Adds an item that allows quick-swapping between armour sets
MIT License
0 stars 0 forks source link

Alternative to presets. #2

Open u0398 opened 1 month ago

u0398 commented 1 month ago

Rather than using presets, a simpler item interaction like right clicking would be nice. Using an empty bundle would store all equipped armor, while using a filled bundle would attempt to equip items.

This would have disadvantages, but it might make using a lot of bundles and armor sets easier. It might also be a little more intuitive to users.

Auroali commented 1 month ago

i'd still like to keep the presets, but i think the preset setting would conflict with this. maybe preset setting could be moved to a keybind (as in something like you hold the bundle and press one of the already existing preset keys to set one) and then right clicking could be exclusively for this?

u0398 commented 1 month ago

I was thinking about it, and a more flexible system might involve "binding" bundles to armor sets without fixed presets. Something like this might work:

Some kind of visual indicator would be nice to identify an empty bundle that was bound to currently worn armor, and that binding would be broken if the empty bundle was moved out of inventory.

Auroali commented 1 month ago

breaking the binding when the bundle is moved out of the inventory might be difficult with how many ways there are to move items around, but the rest sounds good. with this change should bundles only have enough space for one set of armour?

Auroali commented 1 month ago

also the visual indicator for bound bundles could be something like showing the armour items it’s bound to in the item tooltip, separate from the bundle’s contents

u0398 commented 1 month ago

breaking the binding when the bundle is moved out of the inventory might be difficult with how many ways there are to move items around, but the rest sounds good. with this change should bundles only have enough space for one set of armour?

I think it would be reasonable to say an empty bundle would always be unbound. I'm a little less sure how to handle multiple sets in one bundle without fixed presets determining which set to use.

Auroali commented 3 weeks ago

i think i'll try implementing this once i get the 1.21.3 port fully working