Skip to content

Feat/boosts#9

Open
senicko wants to merge 3 commits into
masterfrom
feat/boosts
Open

Feat/boosts#9
senicko wants to merge 3 commits into
masterfrom
feat/boosts

Conversation

@senicko
Copy link
Copy Markdown
Contributor

@senicko senicko commented Feb 5, 2023

Add boosts plugin and write simple movement and collision logic

Comment thread src/boost.rs
if let Some((entity, _)) = rapier_context.cast_shape(
paddle_transform.translation.truncate(),
0.,
Vec2::new(0., -1.),
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

castuj do miejsca w którym potem znajdzie się boost. proponuje dodać zmienną movement_vector o którą będziesz jednocześnie castował i poruszał boosta, tak jak to robimy w ball.rs

Comment thread src/boost.rs
1.,
QueryFilter::default().predicate(&|entity| boost_query.get(entity).is_ok()),
) {
// TODO: apply boost effect or sth
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

zrób to od razu w tym PR. możesz tu wywoływać event, który będzie potem łapać paletka.

Comment thread src/boost.rs
Comment thread src/boost.rs
.get(&textures.boost)
.expect("Boost texture is not loaded");

commands.spawn(BoostBundle::new(
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

boosty mają się spawnić losowo po zniszczeniu bloczka

@imustend
Copy link
Copy Markdown
Contributor

imustend commented Feb 6, 2023

Lgtm

@imustend
Copy link
Copy Markdown
Contributor

lgtm

Copy link
Copy Markdown
Contributor

@imustend imustend left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Copy link
Copy Markdown
Contributor

@imustend imustend left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

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.

3 participants