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

Make thrown bombs explode on impact #2586

Merged
merged 4 commits into from
Aug 25, 2023
Merged

Conversation

Narre
Copy link
Contributor

@Narre Narre commented Aug 17, 2023

Closes #2574.

This PR implements the "explode on impact with other enemies" part + it makes thrown bombs explode on contact with the (a) player.

I'm not sure about the exploding on impact with walls part. It could potentially be based either on whether the hit wall is a vertical wall or a horizontal floor/ceiling, or on the thrown bomb's velocity, but I think it would be better if the bomb doesn't explode when hitting a wall at all. As requested by Rusty, I made it so that the bombs will explode upon being thrown at walls and ceilings, but not floors.

@Narre
Copy link
Contributor Author

Narre commented Aug 20, 2023

OK, I did the wall impact thingy for walls and ceilings.

@Narre Narre changed the title Make thrown bombs explode on impact with badguys and players Make thrown bombs explode on impact Aug 21, 2023
src/badguy/bomb.cpp Outdated Show resolved Hide resolved
src/badguy/goldbomb.cpp Outdated Show resolved Hide resolved
@Narre Narre requested a review from Vankata453 August 22, 2023 19:00
@Vankata453 Vankata453 merged commit bc496de into SuperTux:master Aug 25, 2023
33 checks passed
@mrkubax10 mrkubax10 removed the status:needs-review Work needs to be reviewed by other people label Aug 28, 2023
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.

Bombs explode on impact with other badguys
4 participants