-
Notifications
You must be signed in to change notification settings - Fork 814
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
Remove stash gold sharing #7462
Remove stash gold sharing #7462
Conversation
Maybe a better solution would be introducing a personal stash page per character that gets saved to a new file in the archive, where gold for towners will only read from the individual stash |
Some platforms cannot remove gold from stash. |
I tend to use the term "computer" to not exclude Mac/FreeBSD/etc :D |
Yeah that's the solution I have been planning for my self. I think the current implementation for this PR assumes that a few people not liking the current solution means no one does, but I think I have seen as many people appreciating it as has complained about it. Which I take as a majority generally liking it. I have myself run in to situations where I would have preferred that the gold was in a private stash, but I see this more as something that can be improved rather then something that is broken. Two complications that we have to figure out:
Maybe a solution could be to have both shared and private gold, and a button to push an amount between them? |
The latter is what I had in mind. We can add 2 buttons to the stash window where one is labeled "Private" and the other "Shared". Pressing one keeps it pressed down and pops the other one up. The shared one would go on as usual with no changes; just the current stash how it is. The private one would read and write a single page within the sv/hsv archive and not display page changing buttons. |
Yeah an option to switch between private/shared might make sens, then gold can be transferred though the hero's inventory just like any other item. |
Prevents utilizing stash gold while at towner stores.
Justification: