Fix held slot being set incorrectly (what the heck, past me)
I think for full plugin compatibility we might actually have to reimplement click handling for full control of the click events, which is sad, I was so hoping to avoid that. We might be able to skate by without it by using a separate backing PlayerInventory implementation for the armor-only version. Honestly, the level of compatibility as-is is probably good enough, but I worry. The levels of spaghetti are already getting out of control with all the garbage for placeholder items.
Having typed all that, I think this is probably good enough. Hello and welcome to my rubber ducking process, I do it in the writeup for changes.
I think for full plugin compatibility we might actually have to reimplement click handling for full control of the click events, which is sad, I was so hoping to avoid that. We might be able to skate by without it by using a separate backing PlayerInventory implementation for the armor-only version. Honestly, the level of compatibility as-is is probably good enough, but I worry. The levels of spaghetti are already getting out of control with all the garbage for placeholder items.
Having typed all that, I think this is probably good enough. Hello and welcome to my rubber ducking process, I do it in the writeup for changes.