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

[Issue Report]: Pickup gold from inventory #7781

Open
locksoft opened this issue Feb 24, 2025 · 8 comments
Open

[Issue Report]: Pickup gold from inventory #7781

locksoft opened this issue Feb 24, 2025 · 8 comments

Comments

@locksoft
Copy link

locksoft commented Feb 24, 2025

Operating System

Nintendo Switch

DevilutionX version

1.5.4

Describe

This is still related to issue 6988 (#6988), but I discovered another bug picking gold from inventory.

I know it's useless to pick gold from the stash but on Nintendo Switch, if you accidentally open the gold input form, then the game is stuck and there's no way to close it. You can only quit the game and restart it from the system. I think it's better to disable the function until it gets fixed or removed.

There also another problem related to inventory. If I try to select an amount of gold, the whole slot is picked instead:

http://gofile.me/6RvCQ/1kDvyGrft

If it's not considered worth of fixing (like the pickup from the stash) maybe it's better to disable the function.

Note: this issue is similar to 7780 (#7780) but not identical, that's why I kept them separate.

To Reproduce

Open the form to choose gold amount to pickup and insert a number

Expected Behavior

Get the exact amount of gold I type

Additional context

No response

@locksoft locksoft changed the title [Issue Report]: [Issue Report]: Pickup gold from inventory Feb 24, 2025
@StephenCWills
Copy link
Member

Please upload your video here instead of gofile.

@locksoft
Copy link
Author

As I said in the other post I tried to upload them here, but I got the message "Failed to upload". I think it may be related to the file size, they go from 31 to 76 MB. :(

@StephenCWills
Copy link
Member

Diablo.-.Money.bug.mp4

@AJenbo
Copy link
Member

AJenbo commented Feb 28, 2025

You can probably easily reduce there size by trans coding it, it's not like the quality is important here.

@AJenbo
Copy link
Member

AJenbo commented Feb 28, 2025

Open the form to choose gold amount to pickup and insert a number

Would you please focus more on making it clear how and when the issue happens, not suggested solutions or workarounds. Even with the video I'm not fully sure what's happening and I don't think disabling features is a good option if the issue can actually be solved.

@locksoft
Copy link
Author

Open the form to choose gold amount to pickup and insert a number

Would you please focus more on making it clear how and when the issue happens, not suggested solutions or workarounds. Even with the video I'm not fully sure what's happening and I don't think disabling features is a good option if the issue can actually be solved.

If I use the action button on a slot with coins I should be able to type an amount of coins to pickup (splitting it). But, like the same operation on the stash, it doesn’t works: regardless the amount you type, you’ll end up picking up all the coin (in the stash, instead, you won’t pick up anything and there’s a case when you can’t close the input popup anymore and need to force quit the game).

@StephenCWills
Copy link
Member

StephenCWills commented Feb 28, 2025

If I use the action button on a slot with coins I should be able to type an amount of coins to pickup (splitting it). But, like the same operation on the stash, it doesn’t works: regardless the amount you type, you’ll end up picking up all the coin

I suppose this happens because the Nintendo Switch on-screen keyboard does not actually include an Enter key that can be used to submit the gold splitting dialog. Your assumption that the Confirm button on the controller would submit the dialog is wrong; that button is working as intended by lifting the whole gold pile. However, given that we can't rely on the platform to provide an event for the Enter key, we should probably find a way to change how it works.

(in the stash, instead, you won’t pick up anything and there’s a case when you can’t close the input popup anymore and need to force quit the game).

This doesn't make sense, though. There are several controller buttons that can be used to dismiss the gold withdrawal popup. This sounds like a separate issue that probably wouldn't be fixed even if we resolve the inventory issue reported here. Please open a separate issue to report this one so we can track both issues independently.

Also, keep in mind that many of us do not actually own a Nintendo Switch console capable of running this game so it would be helpful if you provided a video to show exactly how to reproduce it and what happens when you press various buttons.

@locksoft
Copy link
Author

locksoft commented Feb 28, 2025

I suppose this happens because the Nintendo Switch on-screen keyboard does not actually include an Enter key that can be used to submit the gold splitting dialog. Your assumption that the Confirm button on the controller would submit the dialog is wrong; that button is working as intended by lifting the whole gold pile. However, given that we can't rely on the platform to provide an event for the Enter key, we should probably find a way to change how it works.

Usually the "done" (confirm/submit) button on the keyboard works as an "enter" in games. Like when you have to choose a name for a character or insert other information: you type the value, press "done" (or the "+" button) and it works.
Here in Diablo the behavior is weird, but I'll explain in issue #6988 that I'm going to update in a few minutes with a new video.

Edit: topic updated: #6988 (comment)

(in the stash, instead, you won’t pick up anything and there’s a case when you can’t close the input popup anymore and need to force quit the game).
This doesn't make sense, though. There are several controller buttons that can be used to dismiss the gold withdrawal popup. This sounds like a separate issue that probably wouldn't be fixed even if we resolve the inventory issue reported here. Please open a separate issue to report this one so we can track both issues independently.

I tried my best to replicate it after I updated to 1.5.4 and I wasn't able to make it happen. I will keep an eye on it but maybe it was solved with some update.

Also, keep in mind that many of us do not actually own a Nintendo Switch console capable of running this game so it would be helpful if you provided a video to show exactly how to reproduce it and what happens when you press various buttons.

Sure, I'll do my best.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants