peter-kish / gloot

A universal inventory system for the Godot game engine.

MIT License

651 stars 28 forks source link

Cannot drag multiple items in inventory grid #234

Open GustavoLR548 opened 2 months ago

GustavoLR548 commented 2 months ago

When selecting multiple items with the "Multi" option from "Select Mode", it is not possible to consistently select them, nor is it possible to drag them together in the "CtrlInventoryGridEx".

How to test this:

First go to the "res://examples/inventory_grid_ex_transfer.tscn" in this repository, and change the property "Select Mode" from "CtrlInventoryGridLeft" to "Multi"

Now if you try to play the scene and drag the items, this is what happens:

test

When clicking the the item, it flickers. Then after a few more clicks, it works, but is still not possible to drag them.

peter-kish commented 2 months ago

I think I managed fix the multi-select issue in c122bbdef35bf6c60a549f7e95b012d2c9c31dba.

On the other hand, "multi-dragging" is not really supported (yet). Dragging multiple items complicates things quite a bit when it comes to displaying the dragged items, as well as checking for available space in the target inventory.

I think a lot of thought needs to be put into defining how the inventory should behave in such cases in the first place, before implementing it. Of course, if you have any ideas, feel free to suggest them.

GustavoLR548 commented 1 month ago

Oh, I see! I assumed 'multi-dragging' was already supported—my mistake 😅. I took a closer look at the code, and it does seem like, for now, dragging is limited to one item at a time. I can understand that implementing multi-dragging wouldn't be a straightforward task.

At the moment, I don't have any specific ideas for it, but if anything comes to mind, I'll definitely share it here. By the way, your plugin is fantastic, and if I decide to implement any game that has an inventory system, I'll be sure to stick with yours! 😄