Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix rotation for falling mesh degrotate nodes #11159

Merged
merged 3 commits into from Apr 28, 2021

Conversation

Wuzzy2
Copy link
Contributor

@Wuzzy2 Wuzzy2 commented Apr 3, 2021

If a node with drawtype="mesh" and paramtype2="[color]degrotate" falls, the rotation of that node was wrong.

Also, the drawtype test nodes in DevTest were quite buggy as well and not very useful for testing (pyramid looks the same after 90° rotation).

I fixed both this.

How to test

Place the degrotate nodes from DevTest on the ground. Use the Param2 Tool to set a rotation and/or color. Use the Falling Node Tool with the Punch key to force a fall. Test all possible variations. The color and rotation of the falling node must be the same as the color and rotation of the node.

Notifications

@numberZero

@Wuzzy2
Copy link
Contributor Author

Wuzzy2 commented Apr 3, 2021

Update: I've also updated the /test_place_nodes command to properly apply the correct degrotate nodes tests. Use /test_place_nodes in DevTest to test this.

@sfan5 sfan5 added the Bugfix 🐛 PRs that fix a bug label Apr 3, 2021
@sfan5 sfan5 changed the title Fix rotation if mesh degrotate node falls Fix rotation for falling mesh degrotate nodes Apr 3, 2021
@sfan5 sfan5 added the @ Builtin label Apr 3, 2021
@sfan5 sfan5 merged commit 228f1c6 into minetest:master Apr 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants