Fluids: Creating Your First Fluid#539
Conversation
Added functionality for AcidFluid interactions with entities and blocks.
Updated the title and description for clarity and corrected formatting.
Added information about creating a legacy block for acid fluid.
Added a new block type 'ACID' as a LiquidBlock using ModFluids.ACID_STILL.
Inject a method to cancel vanilla water screen effect when in custom fluid.
Kilip1000
left a comment
There was a problem hiding this comment.
This points out a few of the smaller, so far unadressed complaints I have.
its-miroma
left a comment
There was a problem hiding this comment.
babe, wake up, @its-miroma has requested changes again!
reference/latest/src/main/java/com/example/docs/block/ModBlocks.java
Outdated
Show resolved
Hide resolved
reference/latest/src/main/java/com/example/docs/fluid/custom/AcidFluid.java
Outdated
Show resolved
Hide resolved
| // :::bucket | ||
| // :::abstractFluid | ||
| @Override | ||
| public void animateTick(Level world, BlockPos pos, FluidState state, RandomSource random) { |
There was a problem hiding this comment.
This animation, if I'm reading it correctly:
- if the fluid is not a source block, nor is it FALLING (?), then play a sound 1/64th of the time
- if it is a source or it's FALLING, add a particle 1/10 of the time.
should be explained in the docs, or the animation should be simplified.
also, I don't know what I'm looking at with the magic values, and at this point I'm too afraid to ask
Co-authored-by: Miroma <its.miroma@proton.me>
Co-Authored-By: Miroma <its.miroma@proton.me>
modmuss50
left a comment
There was a problem hiding this comment.
Please remove the fog stuff from this PR for now, we should come up with a proper solution instead of recommending weird and wonderful workarounds.
reference/latest/src/main/java/com/example/docs/mixin/FogTypeInvoker.java
Outdated
Show resolved
Hide resolved
reference/latest/src/client/java/com/example/docs/mixin/client/CameraMixin.java
Outdated
Show resolved
Hide resolved
its-miroma
left a comment
There was a problem hiding this comment.
i removed some more fog references that were missed. the two stub classes we can keep, as a reminder, and I'll create an issue
reference/latest/src/main/java/com/example/docs/block/ModBlocks.java
Outdated
Show resolved
Hide resolved
reference/latest/src/client/resources/example-mod.client.mixins.json
Outdated
Show resolved
Hide resolved
Fog changes have been removed as requested.
This is my first pull request in my life, but I did want to update the fluid creation tutorial from the old FabricMC wiki.
Preview URL: https://deploy-preview-539--nimble-elf-d9d491.netlify.app/develop/fluids/first-fluid