Skip to content

Fluids: Creating Your First Fluid#539

Merged
cassiancc merged 84 commits intoFabricMC:mainfrom
MaxURhino:main
Mar 26, 2026
Merged

Fluids: Creating Your First Fluid#539
cassiancc merged 84 commits intoFabricMC:mainfrom
MaxURhino:main

Conversation

@MaxURhino
Copy link
Copy Markdown
Contributor

@MaxURhino MaxURhino commented Mar 24, 2026

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

MaxURhino and others added 30 commits January 12, 2026 14:27
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.
Copy link
Copy Markdown
Contributor

@Kilip1000 Kilip1000 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This points out a few of the smaller, so far unadressed complaints I have.

Copy link
Copy Markdown
Member

@its-miroma its-miroma left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

babe, wake up, @its-miroma has requested changes again!

// :::bucket
// :::abstractFluid
@Override
public void animateTick(Level world, BlockPos pos, FluidState state, RandomSource random) {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

cassiancc and others added 2 commits March 25, 2026 18:16
Co-authored-by: Miroma <its.miroma@proton.me>
Co-Authored-By: Miroma <its.miroma@proton.me>
@modmuss50 modmuss50 requested review from modmuss50 and removed request for CelDaemon and Tenneb22 March 25, 2026 22:51
Copy link
Copy Markdown
Member

@modmuss50 modmuss50 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Copy link
Copy Markdown
Member

@its-miroma its-miroma left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

@its-miroma its-miroma mentioned this pull request Mar 26, 2026
@cassiancc cassiancc dismissed modmuss50’s stale review March 26, 2026 18:37

Fog changes have been removed as requested.

@cassiancc cassiancc requested a review from its-miroma March 26, 2026 18:46
@cassiancc cassiancc merged commit e4f4ab6 into FabricMC:main Mar 26, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new-content A new page or set of pages

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants