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

[fixed] Blobs will void out properly #1976

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

mugg91
Copy link
Contributor

@mugg91 mugg91 commented Mar 25, 2024

Status

  • READY: this PR is (to the best of your knowledge) ready to be incorporated into the game.

Description

[fixed] Keg and Bomb would explode when touching the void
[fixed] Crate and chest would break when touching the void
[fixed] Water arrow, waterbomb and filled bucket would splash when touching the void
[added] sparkle fadeout sound for keg, bomb and water bomb

Fixes #1925

This PR does changes to some blobs to make them void out properly.

I think the void should be like a very deep "nothing", so blobs should not randomly break apart or explode when touching the void.

It doesn't seem like there is any conventional way of telling if a blob has voided out. The condition this.getPosition().y > map.tilemapheight * map.tilesize seems to work well so I added a new script Void.as with the function isVoidedOut() with that condition. The script was then added to some blobs to prevent unwanted stuff from happening when voided.

Since bomb's, keg's and waterbomb's Sparkle.ogg and WaterSparkle.gg "emit sound effect" would abruptly end when voiding, I added sound effects "SparkleFade.ogg and WaterSparkleFade.ogg which are played by those blobs via sprite.PlaySound() when voiding out to give the impression the blobs are actually falling down into the "nothing".

Tested in offline and online, no problems.

Although I know there is an issue with arrows touching and getting stuck on the void, I didn't encounter it in testing in offline and online. So I couldn't test if this PR fixes that.

Steps to Test or Reproduce

Throw exploding waterbomb, bomb, keg or shoot waterarrow or throw waterbucket, crate or chest into a void.
Notice they will explode, splash or break apart.
After this PR, they won't

@mugg91
Copy link
Contributor Author

mugg91 commented Mar 26, 2024

Looks like some people in Discord didn't like this change. But nobody has made any arguments as to why.
I'm just trying to improve the game here. Rather than posting thumb-downs or saying something sucks, I think it would help more if people give constructive criticism and stay polite. Thank you.

@TerminalHash
Copy link
Contributor

Looks like some people in Discord didn't like this change. But nobody has made any arguments as to why. I'm just trying to improve the game here. Rather than posting thumb-downs or saying something sucks, I think it would help more if people give constructive criticism and stay polite. Thank you.

You're not fixing a bug, you're destroying a fairly useful feature. Maybe it's not realistic that kegs and bombs explode for a vanishing point in the void, maybe it's not have logic, but it solves some problems, like in TDM with rat-archers. There's a slight difference between fixing annoying bugs (which may constitute a skeleton to foul play) and changing quite a few useful features.

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

Successfully merging this pull request may close these issues.

Some blobs don't void out properly
2 participants