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

Improve waving definitions #3515

Closed
wants to merge 1 commit into from
Closed

Conversation

BlockMen
Copy link
Contributor

@BlockMen BlockMen commented Jan 1, 2016

  • Allow more drawtypes to use waving (firelike, mesh, nodebox)
  • Don't hardcode waving type, defined by nodedef

Backport from https://github.com/blockplanet/blockplanet. Since I'm the copyright holder and doin this PR here it can be used ofc.

- Allow more drawtypes to use waving (firelike, mesh, nodebox)
- Don't hardcode waving type, defined by nodedef
@HybridDog
Copy link
Contributor

https://github.com/minetest/minetest/pull/3515/files?w=1

l need waving glass for birch leaves.

@nerzhul
Copy link
Member

nerzhul commented Jan 4, 2016

static const u8 offset = 3; please

@paramat
Copy link
Contributor

paramat commented Dec 28, 2016

Possibly replaced by #3497

@paramat paramat added the Adoption needed The pull request needs someone to adopt it. Adoption welcomed! label Mar 21, 2017
@paramat
Copy link
Contributor

paramat commented Mar 21, 2017

Author has left MT.
@sofar could you grab anything you like from this for #3497 so we can close it?

@paramat
Copy link
Contributor

paramat commented Apr 25, 2017

Partially replaced by #3497

@paramat paramat closed this Apr 25, 2017
@paramat paramat reopened this Apr 25, 2017
@SmallJoker
Copy link
Member

Sofar and nerzhul agreed on closing. The author is gone and the pull gets replaced by ^.

@SmallJoker SmallJoker closed this Apr 26, 2017
@paramat
Copy link
Contributor

paramat commented Apr 26, 2017

I agree to close too.

@Zughy Zughy removed the Adoption needed The pull request needs someone to adopt it. Adoption welcomed! label May 1, 2022
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

6 participants