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

Adding some sound effects and animations for rockbox damage #18208

Merged
merged 8 commits into from Apr 13, 2024

Conversation

mloccy
Copy link
Contributor

@mloccy mloccy commented Mar 14, 2024

[OBJECTS][ENHANCEMENT]

About the PR

The rockbox can be destroyed, but when a player attacked the rockbox it behaved like an object that could not be destroyed, at least not without an explosion. This also makes the rockbox inoperable if it has been destroyed, unlike before where it was still usable except the remote purchasing of ore didn't work.

Why's this needed?

I think if a machine can be destroyed it should show some indication that damage is being done.

(u)mloccy
(+)Adding some animations to the rockbox on damage, and made it so when its broken it cannot be used.

@keywordlabeler keywordlabeler bot added A-Game-Objects The point of this PR is to deal with a specific game object C-Feature A new feature or enhancements to existing features labels Mar 14, 2024
@github-actions github-actions bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Mar 14, 2024
@mloccy
Copy link
Contributor Author

mloccy commented Mar 15, 2024

Video showing new behaviour

RockboxDamage.mov

@github-actions github-actions bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Apr 10, 2024
@github-actions github-actions bot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Apr 10, 2024
@github-actions github-actions bot added the S-Ready-For-Final-Review This PR has been looked over by members of the community. (automatic after 2 approvals) label Apr 11, 2024
@Tarmunora Tarmunora merged commit 2b05c6f into goonstation:master Apr 13, 2024
23 checks passed
github-actions bot pushed a commit that referenced this pull request Apr 13, 2024
@mloccy mloccy deleted the RockboxDamage branch April 13, 2024 18:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Game-Objects The point of this PR is to deal with a specific game object C-Feature A new feature or enhancements to existing features S-Ready-For-Final-Review This PR has been looked over by members of the community. (automatic after 2 approvals) size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants