Boss Bomb Throwing #272
Boss Bomb Throwing #272
Conversation
| ItemKind::Bomb { .. } => { | ||
| panic!("Can't throw bomb") | ||
| } |
There was a problem hiding this comment.
It might be interesting to come back in future and make it so the player can pickup and throw the bomb before it explodes.
There was a problem hiding this comment.
For sure, I thing the only problem is that the bomb stays "fusing" for I think 0.36 seconds, so not much time to grab and throw it now, but it for sure makes some fun gameplay
There was a problem hiding this comment.
yup, I was thinking we could change the duration potentially or have it extend a bit if picked up fast enough.
| #[derive(Component, Clone)] | ||
| pub struct Explodable { | ||
| pub attack: AttackMeta, | ||
| pub timer: Timer, | ||
| pub fusing: bool, | ||
| pub animated_sprite: AnimatedSpriteSheetBundle, | ||
| pub explosion_frames: AttackFrames, | ||
| } |
There was a problem hiding this comment.
Does explodable need to have its own AttackMeta and AttackFrames ?
AttackMeta already has its own AttackFrames but couldnt Explodable be a component with only timer and fusing and animated_sprite?
Then add the Explodable component to an entity with an AttackMeta and query for both when you need to deal with both.
There was a problem hiding this comment.
Not sure about the other thing you said, but we need two attack frames actually, one for the bomb explosion, and one for the frames the boss throws the bombs
src/attack.rs
Outdated
| if let Ok((animation, explodable)) = explodable_query.get(**parent) { | ||
| if animation.current_frame >= attack_frames.startup | ||
| && animation.current_frame <= attack_frames.active | ||
| { | ||
| commands.entity(entity).insert(Collider::cuboid( | ||
| explodable.attack.hitbox.size.x / 2., | ||
| explodable.attack.hitbox.size.y / 2., | ||
| )); | ||
| } | ||
| } |
There was a problem hiding this comment.
I'd like to avoid special casing this activation code, I've done a bit of a refactor, added the hitbox metadata directly to Attack now which should let you remove this extra query and block if you add a With<Explodable> to the or tuple on the parent_query.
|
bors merge |
Implements boss bomb throwing
Closes #269