Skip to content

feat(block): implementing shelf mushroom block (26.3) - #756

Merged
coco875 merged 17 commits into
Steel-Foundation:masterfrom
aure31:shelf-mushroom
Oct 19, 2026
Merged

coco875 merged 17 commits into
Steel-Foundation:masterfrom
aure31:shelf-mushroom

Conversation

@aure31

@aure31 aure31 commented Oct 10, 2026 •

Copy link
Copy Markdown
Contributor

Type of change

  • Block implementation
  • Item implementation
  • Command implementation
  • Entity implementation
  • Bug fix
  • New feature
  • Breaking change
  • Refactor / code cleanup
  • Performance improvement
  • Chore / tooling

Description

implement the new shelf mushroom block and the new block config that goes with it. Also matching bouncing logic with vanilla
it add the new fallDistanceReduction property to blocks:
Steel-Foundation/SteelExtractor#31

How this was tested

ingame testing

Screenshots / logs

image

Checklist

  • Code builds w/o errors or warnings
  • Self-reviewed the diff
  • Docs updated (if applicable)
  • No leftover debug code / comments

Classes / commands modified:

Additional notes

@github-actions github-actions Bot added the Enhancement New feature or request label Oct 10, 2026
@aure31 aure31 changed the title feat(block): implementing shelf mushroom block feat(block): implementing shelf mushroom block (26.3) Oct 10, 2026
@aure31
aure31 marked this pull request as ready for review October 11, 2026 19:30
@rainnvfx

Copy link
Copy Markdown
Contributor

LGTM

Tested in-game:

  • Interactions with water/lava
  • Behavior against pistons
  • Bounce animation
  • Fall damage reduction
  • Crafting recipes
  • Drop rates through both growth stages
  • Generation in dappled forests

Everything matches Vanilla

@Joniii11 Joniii11 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

this is what i found which is important

Comment thread steel-core/src/behavior/block/mod.rs Resolved
Comment thread steel-core/src/entity/entity/mod.rs Resolved

@Joniii11 Joniii11 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

this is just some nitpicking most of it some are some good things mabe idk

Comment thread steel-core/src/behavior/block/mod.rs Resolved
Comment thread steel-core/src/entity/entity/mod.rs Resolved
Comment thread steel-core/src/entity/entity/mod.rs Resolved

@Joniii11 Joniii11 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM well done thank you very much! :)

@JunkyDeveloper
JunkyDeveloper changed the base branch from 26.3 to master October 13, 2026 23:10
@github-actions

Copy link
Copy Markdown

This pull request has conflicts with the base branch "master". Please resolve those so we can test out your changes.

@github-actions

Copy link
Copy Markdown

Conflicts have been resolved! 🎉

@github-actions github-actions Bot added the Block Request/implementation/Bug of a block label Oct 14, 2026
Comment thread steel-core/src/behavior/block/mod.rs Resolved
@aure31

aure31 commented Oct 16, 2026

Copy link
Copy Markdown
Contributor Author

@JunkyDeveloper it's fix

@Joniii11 Joniii11 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

Comment thread steel-core/src/entity/entity/mod.rs Resolved
JunkyDeveloper
JunkyDeveloper previously approved these changes Oct 18, 2026
@coco875
coco875 merged commit 0473ad4 into Steel-Foundation:master Oct 19, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved-by-coco875 approved-by-junkydeveloper Block Request/implementation/Bug of a block Enhancement New feature or request Refactor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants