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

Wind particles are at negative Z value #3144

Open
6 tasks done
Alasdairbugs opened this issue Dec 16, 2024 · 5 comments · May be fixed by #3202
Open
6 tasks done

Wind particles are at negative Z value #3144

Alasdairbugs opened this issue Dec 16, 2024 · 5 comments · May be fixed by #3202
Assignees
Labels

Comments

@Alasdairbugs
Copy link
Contributor

SuperTux Version

0.6.3-1505-gc2e6b766d

System Information

win 11 64 bit

Expected Behavior

wind particles should not be at this low of a Z value i think

because its an object, it should default to 50

Actual Behavior

its at below -50

Steps To Reproduce Actual Behavior

place some tiles at z50

place wind, set to fancy particles. interacts with badguys, objects. observe.

Additional Information

No response

Guidelines For Reporting Issues

  • I have read https://github.com/SuperTux/supertux/blob/master/CONTRIBUTING.md#bug-reports.
  • I have verified this isn't an issue that's already been reported.
  • I have verified this isn't a discussion, or an issue about a crash or a feature request, but rather an actual bug ─ that is, the game did something not intended.
  • I have verified this issue is not about wrong translations (use Transifex for those), or anything unsupported (e.g. third-party add-ons).
  • In this report, I have only included details about one (1) bug.
  • If I make a mistake while submitting this report, I agree to use the "Edit" feature to correct it, instead of closing this issue and opening a new one.
@MatusGuy
Copy link
Member

@tobbi
Copy link
Member

tobbi commented Feb 10, 2025

No, that's for the editor image. Actual wind particles are at LAYER_BACKGROUNDTILES - 1, aka -99.

Where do we want that?

@Alasdairbugs
Copy link
Contributor Author

i think because its an object it should be z50?

@tobbi
Copy link
Member

tobbi commented Feb 10, 2025

How about making it configurable?

@tobbi tobbi self-assigned this Feb 10, 2025
tobbi added a commit that referenced this issue Feb 10, 2025
@tobbi tobbi linked a pull request Feb 10, 2025 that will close this issue
@tobbi
Copy link
Member

tobbi commented Feb 10, 2025

Please test the PR above.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants