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

Dragging item from ItemSlot off centre. v2.4.3 #167

Closed
kyle-wannacott opened this issue Feb 25, 2024 · 7 comments
Closed

Dragging item from ItemSlot off centre. v2.4.3 #167

kyle-wannacott opened this issue Feb 25, 2024 · 7 comments
Labels
bug Something isn't working
Milestone

Comments

@kyle-wannacott
Copy link
Contributor

kyle-wannacott commented Feb 25, 2024

Hey just updated to 2.4.3; nice work on the icon resizing options!

I found a couple thruple of bugs.

  1. When dragging an item from the ItemSlot the image is off centre (probably same issue as this but for item slot rather than inventory grid: (DrapItem position #160)

  2. The actual item slots default item icon seems to be draggable.

  3. The size of the item dragged from the item slot is not the same as when dragged from the inventory grid (its way bigger.)

Video:

2024-02-25.22-00-08.mp4

P.S

4. Also dropping your item on-top off the ItemSlot makes the default icon dissapear, whereas I think it should probably draw on-top of it. Nevermind using a StyleBoxTexture in Slot style solves this.

@peter-kish peter-kish added the bug Something isn't working label Feb 25, 2024
@kyle-wannacott
Copy link
Contributor Author

kyle-wannacott commented Feb 25, 2024

@peter-kish I also found this bug where your item just vanishes. Not sure what causes it. Seems like it happens dragging and dropping into the same slot that is equipped; when dragging and dropping onto the corner of the item slot (in the top right). Its pretty easy to make it happen.

Untitled.mp4

@peter-kish
Copy link
Owner

Seems like the CtrlItemSlot icon scaling options introduced quite a few bugs 😬 I'll make them a priority for the next release.

@peter-kish peter-kish added this to the 2.4.4 milestone Feb 25, 2024
@peter-kish
Copy link
Owner

I'm afraid I'll need some more information on most of these issues... Apart from the 2nd one (dragable default icon), I wasn't really able to reproduce them. Here's the minimal project I was playing around with:

item_slot_bug_test.zip

@kyle-wannacott
Copy link
Contributor Author

@peter-kish

item_slot_bug_test.zip

I reproduced number 1; it is to do with the scale of the control being less than 1.

Number 3 is present in your example/minimal project; when dragging your item from slot to slot, or back to the inventory grid I think the size of the item should be what is set (width and height) in the protoset (.i.e the same as when dragging item from the inventory grid).

Not sure about the vanishing inventory. I need to look into it more when have time.

@kyle-wannacott
Copy link
Contributor Author

@peter-kish

vanishing_inventory_reproduced.zip

Okay figured out the case of the vanishing inventory you just need to swap between inventory slots and then drop the item onto the current CtrlItemSlot you switched to:

2024-02-28.03-17-43.mp4

I don't think this is related to the other bugs.

@peter-kish
Copy link
Owner

I think there are several unrelated problems addressed here, so I created separate issues for each of them to make discussions easier:

#174
#175
#176
#177

@LeeWannacott Can you take a quick look if anything is missing? After that I'll close this one.

@kyle-wannacott
Copy link
Contributor Author

@peter-kish Yeah fair enough, Looks good to me!

@peter-kish peter-kish closed this as not planned Won't fix, can't repro, duplicate, stale Mar 1, 2024
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