Skip to content
This repository has been archived by the owner on Jan 7, 2025. It is now read-only.

Make ItemBuilder able to incorporate existing ItemStacks and modify them #11

Open
lokka30 opened this issue Sep 7, 2022 · 2 comments
Open
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@lokka30
Copy link
Member

lokka30 commented Sep 7, 2022

I'll pile my current (lack of) thoughts here:

  • Do not try to recreate the itemstack. It should modify the itemstack as the builder is used, and return the itemstack object on build.
  • Clone the itemstack when modifying it.
@lokka30 lokka30 added the enhancement New feature or request label Sep 7, 2022
@lokka30 lokka30 self-assigned this Sep 7, 2022
@lokka30 lokka30 added this to the v0.2 milestone Oct 27, 2022
@HSGamer
Copy link

HSGamer commented Jan 5, 2023

I guess it could be something like this

@lokka30
Copy link
Member Author

lokka30 commented Jan 5, 2023

I guess it could be something like this

That's a very cool item builder! Thanks.

I can't remember why I added an item builder class - I think it was just for MicroLib parity - so I might remove my ItemBuilder if I don't have a use for it (which means that I would have found a nice GUI library).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants