Skip to content

Item Frames - #3584

Draft
Crepestrom wants to merge 4 commits into
PixelGuys:masterfrom
Crepestrom:item-frame
Draft

Item Frames#3584
Crepestrom wants to merge 4 commits into
PixelGuys:masterfrom
Crepestrom:item-frame

Conversation

@Crepestrom

Copy link
Copy Markdown
Contributor
image image

@Crepestrom
Crepestrom marked this pull request as ready for review September 8, 2026 03:36
@ikabod-kee ikabod-kee moved this to Waiting for artistic review in PRs to review Sep 8, 2026

@Wunka Wunka 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.

There is still a lot of leftover naming from the sign code you copied.
This makes bugs possible thorugh for example your callback where you lock the signs mutex and not the item frames one

@Crepestrom

Copy link
Copy Markdown
Contributor Author

oops didnt catch that
updated a bunch of the names

Comment thread src/block_entity.zig Outdated
main.items.Item.toBytes(data.item, writer);
}

pub fn updateTextFromClient(pos: Vec3i, newText: main.items.Item) void {

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.

still naming from the sign

@careeoki careeoki 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.

Attempting to put items in frames freezes the game for me, with and without debug mode.
It doesn't crash so I can't show a log.

@IntegratedQuantum IntegratedQuantum moved this from Waiting for artistic review to WIP/not ready for review in PRs to review Sep 8, 2026
@IntegratedQuantum

Copy link
Copy Markdown
Member

Please wait until the block entity rewrite is finished, to avoid having to migrate the save data.
In the future block entities will probably use the ECS too.

@Crepestrom
Crepestrom marked this pull request as draft September 8, 2026 17:36
@Crepestrom

Copy link
Copy Markdown
Contributor Author

ok will wait

@du82

du82 commented Sep 9, 2026

Copy link
Copy Markdown

I like it! Although I think more colors than just green should be possible

@TyrrelMeter

Copy link
Copy Markdown
Contributor

Item frames in many colours would be really cool, I agree. Might be a fair bit of work for Crepestrom when Carrie will probably redo the texture anyway.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: WIP/not ready for review

Development

Successfully merging this pull request may close these issues.

7 participants