Fix dupe, doubble click and shift click

[4lve]

Description

This PR fixes:

• Dropping with q or shift q not working
• Dupe glitch when dropping from inventory
• Shift clicking
• Doubble clicking

Testing

Please follow our Coding Guidelines

[Snowiiii]

Does this fix any issues ?, Maybe #547

[4lve]

Yep, it fixes that issue

[Snowiiii]

I notice that our currently implementation has a bug which disallows the player to interact with an item the first time they open there Inventory. Can you may fix this as well?

[4lve]

Ye that's odd, but that was a bug before I think cause even logging all clickevents it won't show the first interaction

[4lve]

That issue is caused because the state id is off sync

[4lve]

I fixed so it works first try now, and I also added partial support for equipping armor with shift clicking

[Snowiiii]

Nice, It fixed now. Great job @4lve. So i have one last bug i would love to have fixed 😅. Maybe you noticed that when you begin to drag an item in inventory and then close the inventory while having the stack dragged, it is not reseted but still being dragged when open the inventory the next time

[4lve]

@Snowiiii Thx, I could maybe fix that in another PR

[kralverde]

Hey @4lve I made a review, should be good to merge after. The item dragging can be another PR. Could you help me understand what the state_id does? At the end there was something weird with state_id -= 2 and stuff.

[4lve]

Idk, I just found that the state id decreases by 2 after closing a container. Found it by trail and error

[kralverde]

I broke something...

[kralverde]

Sorry @4lve I tried to push to your PR but something got messed up. Could you merge my PR to your fork and then re-open the PR? Your authorship should be retained. Thanks for working on this!