Skip to content

Properly remove items from shelves - #13567

Merged
Lulu13022002 merged 2 commits into
PaperMC:mainfrom
Lulu13022002:fix/shelves-inventory
Aug 5, 2026
Merged

Properly remove items from shelves#13567
Lulu13022002 merged 2 commits into
PaperMC:mainfrom
Lulu13022002:fix/shelves-inventory

Conversation

@Lulu13022002

Copy link
Copy Markdown
Contributor

@Lulu13022002
Lulu13022002 requested a review from a team as a code owner January 19, 2026 17:53
@github-project-automation github-project-automation Bot moved this to Awaiting review in Paper PR Queue Jan 19, 2026

@lynxplay lynxplay left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works ish however the chiseled bookshelf has this incredibly annoying implementation detail of using block state to track its visual instead of the BE itself, which means that the code suggested by every guide and post on this planet (call your local BlockState#update() method after modification) will still lead to this visual break from the actual BE state.

I don't think there is anything sane we can do against this however.

@Lulu13022002
Lulu13022002 force-pushed the fix/shelves-inventory branch from 4886980 to d4494f9 Compare April 30, 2026 18:31
@Lulu13022002
Lulu13022002 force-pushed the fix/shelves-inventory branch from d4494f9 to e250396 Compare August 5, 2026 18:37
@Lulu13022002
Lulu13022002 enabled auto-merge (squash) August 5, 2026 18:41
@Lulu13022002
Lulu13022002 merged commit 06a7587 into PaperMC:main Aug 5, 2026
2 checks passed
@github-project-automation github-project-automation Bot moved this from Awaiting review to Merged in Paper PR Queue Aug 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Merged

Development

Successfully merging this pull request may close these issues.

3 participants