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 some fallblock issues #2849

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

weluvgoatz
Copy link
Member

There existed an issue where the fallblock would not properly convey that it was going to fall, and I fixed that in this PR. Also, enemies are properly squished by fallblocks.
NOTE: This PR removes the feature where bumpers would cling to fallblocks. This is because the feature is re-added in a more consistent way in PR #2846.
NOTE 2: I think the fallblock needs a new, unique graphic. Maybe when it is drawn it can be added to this PR.

@Rusty-Box
Copy link
Member

If you have any ideas/suggestions of what to use for a fallblock graphic I can work on one. I know that I am using this object for ruin themed levels where some bricks of the ceiling may fall down, so one graphic I will be doing is a cracked/broken version of the ruin brick

@mrkubax10 mrkubax10 added category:code status:needs-review Work needs to be reviewed by other people type:bugfix Pull Requests that fix bugs. status:in-progress Progress has been done, but more is intended be done and removed status:needs-review Work needs to be reviewed by other people labels Mar 27, 2024
@weluvgoatz weluvgoatz added this to the 0.7.0 milestone Apr 20, 2024
@weluvgoatz weluvgoatz marked this pull request as ready for review June 1, 2024 21:23
@weluvgoatz
Copy link
Member Author

I decided that the graphics for fallblock can be decided later. This is solely for bug fixing.

@weluvgoatz weluvgoatz removed this from the 0.7.0 milestone Jun 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:code status:in-progress Progress has been done, but more is intended be done type:bugfix Pull Requests that fix bugs.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants