Skip to content

Add vine physics#1107

Merged
aramperes merged 3 commits intoGlowstoneMC:devfrom
x4e:vinephysics-pr
Feb 11, 2021
Merged

Add vine physics#1107
aramperes merged 3 commits intoGlowstoneMC:devfrom
x4e:vinephysics-pr

Conversation

@x4e
Copy link
Contributor

@x4e x4e commented Jan 29, 2021

This PR adds physics to vines.

This means that every time a block updates near a vine the vine will break if it is not below another vine and is not currently on a solid block.

Previously placing a vine on a block, then destroying the block would result in a floating vine. Same for hanging a vine off another vine and destroying the parent vine.

@x4e
Copy link
Contributor Author

x4e commented Jan 29, 2021

Should be good to go now, only potential thing to look at in the future is support for vines being on top/bottom of blocks, this seems to have some sort of weird support in the vanilla edition that maybe should be replicated.

Copy link
Member

@aramperes aramperes left a comment

Choose a reason for hiding this comment

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

Code looks good, however I'm not sure if this will be merged because there is a soft code-freeze on dev until we can merge the 1.16 branch.

The patch is fairly small so I'm personally not opposed to merging now. cc. @mastercoms

@aramperes aramperes merged commit 4ebe170 into GlowstoneMC:dev Feb 11, 2021
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