Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Custom particles #32

Open
wants to merge 21 commits into
base: main
Choose a base branch
from
Open

Custom particles #32

wants to merge 21 commits into from

Conversation

tal5
Copy link
Member

@tal5 tal5 commented Apr 18, 2024

Requested on Discord.

Adds custom particles - not going to go over every feature, but overall:

  • Particle script containers
  • ParticleTag & it's relevant features
  • particle command expansions for this

Particles don't actually have any sort of ID, so mixined in a UUID + map to store particles by their UUID so that ParticleTag can base itself on something.

Since this is received from the server & the client can leave/join as many servers as it wants, had to build in some ✨dynamic registry modification✨.
Overall I believe it's relatively safe, as it's purely client-side, properly removes it from all of the registry's maps, and re-freezes it later, but let me know what do you think.

Tried building around existing particle texture features & slightly hack in the textures list you'd usually specify in a json file, mostly to try and stay as close to vanilla as possible to avoid something changing and breaking everything in the future - let me know if you think that's not worth it and we should just have our own texture features though.

@tal5
Copy link
Member Author

tal5 commented May 13, 2024

Updated to 1.20.6 @acikek

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants