Repository navigation
Held item component - #3697
Held item component#3697Crepestrom wants to merge 9 commits into
Conversation
| // ############################# Client only stuff ################################ | ||
| pub const client = struct { | ||
| const Component = struct { | ||
| heldItem: items.Item, |
There was a problem hiding this comment.
would it not make more sense to have the index into the inventory here? This has duplicated the saving of the item. this could lead to out of sync issues. (the inventory was changed through a sync command, but the held item sync command has not yet been run / was not even triggered because of some misconfiugration
In my opinion (just mine, not a fact):
The general inventory is in a component and this is an index into that components items.
For example a system that renders the player would use both components to get the item and then display it.
There was a problem hiding this comment.
i dont think the client knows whats in other players inventories is the issue i think think
There was a problem hiding this comment.
ok it also references a InventoryAndSlot now
for #2734
adds a held item component that syncs and tracks what item the player has selected