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

Fix double clicking to stack items #921

Merged
merged 1 commit into from
Oct 1, 2021
Merged

Conversation

calcastor
Copy link
Contributor

Previously PGM used SportPaper's InventoryClickedEvent to handle double clicking to stack items. However, it is no longer used since the introduction of Spigot 1.8 compatibility. Given this, onPlayerClickInventory should no longer ignore COLLECT_TO_CURSOR. This should resolve issues reported with double clicking to stack items needing more than one double click to work that were reported by players in OCC's most recent tournament.

Previously PGM used SportPaper's InventoryClickedEvent to handle COLLECT_TO_CURSOR, but since it is no longer used in PGM for compatibility reasons, COLLECT_TO_CURSOR should not be ignored as it can cause issues with double clicking to stack items.

Signed-off-by: BT (calcastor/mame) <[email protected]>
@Electroid Electroid merged commit 9783baf into PGMDev:dev Oct 1, 2021
@calcastor calcastor deleted the inventory-dc branch October 2, 2021 03:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants