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
Block Damage systems #4
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Everything looks fine!
@Override | ||
public void renderOverlay() { | ||
if (blockSelectionRenderer == null) { | ||
Texture texture = Assets.getTextureRegion("health:blockdamageeffects#1").get().getTexture(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Even though the BlockDamageEffects has been moved into the module, this line is creating NoSuchElementException
.
{ | ||
"location": {}, | ||
"particleDataSprite": { | ||
"texture": "core:fx_smoke" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This will need to be changed to "health:fx_smoke" and the file should be duplicated to health because I'm fairly sure that there are other modules that use it.
deltas/engine/prefabs/test.prefab
Outdated
@@ -0,0 +1,5 @@ | |||
{ | |||
"RigidBody": { | |||
"kinematic": false |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this file still be here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think this is required, so will remove it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The regen is working! It would be nice to open an issue to fix the regen speed later though
Adds required assets and system for functioning of damage to blocks.
How to test:
BuilderSampleGameplay
template withHealth
Module activated in advanced settings