Skip to content

feat(ContainerPreview): Add shulker content preview to item render in inventories#286

Merged
beanbag44 merged 4 commits intolambda-client:1.21.11from
IceTank:improvement/shulker-content-preview
Apr 26, 2026
Merged

feat(ContainerPreview): Add shulker content preview to item render in inventories#286
beanbag44 merged 4 commits intolambda-client:1.21.11from
IceTank:improvement/shulker-content-preview

Conversation

@IceTank
Copy link
Copy Markdown
Contributor

@IceTank IceTank commented Apr 23, 2026

image

This PR adds shulker content preview renders to items with containers like shulker boxes and bundles.

@beanbag44
Copy link
Copy Markdown
Member

this is really cool but theres a small issue. For example, this shulker is displaying a dirt block when most of the items are pickaxes. This also happened with my signed books shulker. I assume this is because of stack size, but i think it would be better to compare, either by item count, ignoring stack size, or accounting for the maximum stack size.

image

@IceTank
Copy link
Copy Markdown
Contributor Author

IceTank commented Apr 24, 2026

Yeah I can make it weigh item count relative to an items max stack size. But I think the current method is the same that rusherhack uses and I'm used to that.

@beanbag44
Copy link
Copy Markdown
Member

maybe a mode setting for item count weight

Copy link
Copy Markdown
Member

@beanbag44 beanbag44 left a comment

Choose a reason for hiding this comment

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

small changes then merge

Comment thread src/main/java/com/lambda/mixin/render/DrawContextMixin.java Outdated
Comment thread src/main/kotlin/com/lambda/module/modules/render/ContainerPreview.kt Outdated
Comment thread src/main/kotlin/com/lambda/module/modules/render/ContainerPreview.kt Outdated
Comment thread src/main/kotlin/com/lambda/module/modules/render/ContainerPreview.kt Outdated
Comment thread src/main/kotlin/com/lambda/module/modules/render/ContainerPreview.kt Outdated
@IceTank IceTank requested a review from beanbag44 April 25, 2026 13:05
Copy link
Copy Markdown
Member

@beanbag44 beanbag44 left a comment

Choose a reason for hiding this comment

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

Awesome sauce, good to merge

@beanbag44 beanbag44 merged commit 57b5027 into lambda-client:1.21.11 Apr 26, 2026
@github-project-automation github-project-automation Bot moved this from Backlog to Done in Kanban Apr 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants