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

SilentOpenChest improvements #80

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

crazycat256
Copy link
Contributor

This PR modifies the SilentOpenChest feature.

  • When a vanish player opens a chest, he becomes a spectator only for 2 ticks (instead of the whole time the chest is open).
  • The player no longer receives packets relating to his gamemode change.

These changes prevent the player from passing through the blocks when opening the chest, and prevent the chest from closing if the player moves.

@LeonMangler
Copy link
Owner

Thank you for the PR! I have had packet adapters for GAME_STATE_CHANGE and NAMED_ENTITY_SPAWN in the past, and they kept breaking with new minecraft releases. Currently I feel like it may perhaps even make more sense to not use them to increase stability. Either way, the current way the silent open chest feature works is very wonky, and I wish someday there would be an API-only option without temporary gamemodes.

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

Successfully merging this pull request may close these issues.

2 participants