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

enhancement: bank withdraw behavior #1903

Conversation

Arufonsu
Copy link
Contributor

  • Implements the 'FindAvailableInventorySpaceForItem' method to ensure sliders display the correct values for available inventory space.
  • Instead of blindly opening the item withdraw slider with a max quantity of 0 when there's no space left in the inventory (pointless), we now inform the player: "There is no space left in your inventory for that item!"

Before:

2023-09-14.21-14-50.mp4

After:

2023-09-14.22-00-38.mp4

@Arufonsu Arufonsu added the enhancement Minor feature addition or quality of life change label Sep 15, 2023
@Arufonsu Arufonsu requested review from a team September 15, 2023 01:15
@Arufonsu Arufonsu force-pushed the enhancement/bank-withdraw-slider branch 2 times, most recently from 295dd3d to b13e32a Compare September 15, 2023 01:45
Intersect.Client/Entities/Player.cs Outdated Show resolved Hide resolved
@Arufonsu Arufonsu requested a review from lodicolo September 26, 2023 01:43
@Arufonsu Arufonsu force-pushed the enhancement/bank-withdraw-slider branch from 9bd65fc to b4ce650 Compare October 2, 2023 23:38
@Arufonsu Arufonsu self-assigned this Oct 13, 2023
@Arufonsu Arufonsu force-pushed the enhancement/bank-withdraw-slider branch from b4ce650 to fffc008 Compare October 13, 2023 03:50
- Implements the 'FindAvailableInventorySpaceForItem' method to ensure sliders display the correct values for available inventory space.
- Instead of blindly opening the item withdraw slider with a max quantity of 0 when there's no space left in the inventory (pointless), we now inform the player: "There is no space left in your inventory for that item!"
itemID default (Guid) is Guid.Empty
inventorySlot default (int) is null
@Arufonsu Arufonsu force-pushed the enhancement/bank-withdraw-slider branch from fffc008 to 9a13333 Compare October 21, 2023 17:43
@Arufonsu Arufonsu requested review from a team October 21, 2023 17:43
@Arufonsu Arufonsu marked this pull request as draft October 21, 2023 19:31
@Arufonsu Arufonsu closed this Oct 21, 2023
@Arufonsu Arufonsu mentioned this pull request Oct 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Minor feature addition or quality of life change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants