Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cannot drag multiple items in inventory grid #234

Open
GustavoLR548 opened this issue Sep 1, 2024 · 2 comments
Open

Cannot drag multiple items in inventory grid #234

GustavoLR548 opened this issue Sep 1, 2024 · 2 comments
Labels
bug Something isn't working

Comments

@GustavoLR548
Copy link

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"

image

  • 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 peter-kish added the bug Something isn't working label Sep 7, 2024
@peter-kish
Copy link
Owner

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

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
Copy link
Author

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! 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants