Skip to content

add support for partial draining of things like gt cells#87

Closed
chrombread wants to merge 1 commit intomasterfrom
partial-fluid-container-operations
Closed

add support for partial draining of things like gt cells#87
chrombread wants to merge 1 commit intomasterfrom
partial-fluid-container-operations

Conversation

@chrombread
Copy link
Member

@chrombread chrombread commented Jan 4, 2026

fixes https://discord.com/channels/181078474394566657/522098956491030558/1457456489638658048

the rest of the code was copied from mui1 when it was written, but not this part for some reason. maybe it was implemented at a later point in time? see https://github.com/GTNewHorizons/ModularUI/blob/master/src/main/java/com/gtnewhorizons/modularui/common/widget/FluidSlotWidget.java#L446
either way, tested in game and it seems to work with gt cells.

@chrombread chrombread added the enhancement New feature or request label Jan 4, 2026
@Dream-Master Dream-Master added the 🚧 Testing on Zeta Do not merge yet, testing this PR on Zeta label Jan 9, 2026
@danyadev
Copy link
Member

danyadev commented Feb 4, 2026

I'm working on a fluid handling refactoring PR that's going to supersede this PR, so it doesn't need to be reviewed right now

@danyadev
Copy link
Member

danyadev commented Feb 4, 2026

Done: #93

@chrombread
Copy link
Member Author

this can be closed since it's superseded.

@chrombread chrombread closed this Feb 5, 2026
@Dream-Master Dream-Master deleted the partial-fluid-container-operations branch February 5, 2026 06:31
@Dream-Master Dream-Master removed the 🚧 Testing on Zeta Do not merge yet, testing this PR on Zeta label Feb 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants