Skip to content

Let mushroom branches decay - #3692

Open
careeoki wants to merge 3 commits into
PixelGuys:masterfrom
careeoki:mush-decay
Open

careeoki wants to merge 3 commits into
PixelGuys:masterfrom
careeoki:mush-decay

Conversation

@careeoki

@careeoki careeoki commented Oct 9, 2026

Copy link
Copy Markdown
Contributor

In multiplayer I will often see mushroom caves with floating branches left like this:
Generally it's just kinda odd since you would expect mushroom branches to work the same way as the other branches. Even cacti, which are not trees, still decay their branches.

image I also removed `.decayProhibitor` since there is nothing for them to prohibit the decay of.

Note that to let them decay, you have to chop both the logs and the mushroom caps. This is maybe slightly weird but making mushroom caps decay would be weirder.

@@ -1,5 +1,8 @@
.{
.tags = .{.choppable, .cuttable, .sliceable, .mushroom},
.onUpdate = .{

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.

Should we maybe just put it into _defaults then?
The only other branch without decay is candy cane, and why not give it to candy cane as well for consistency? We could still make the structures non-decaying by using non-decayable branches for them.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Sure yeah

},
.texture = "branch/bolete.png",
},
.decayProhibitor = true,

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 property isn't even present in the code, could you please also remove them in the other files as well?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

ah, that's what it does. Nothing

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In review

Development

Successfully merging this pull request may close these issues.

2 participants