Skip to content

Add Enemy health bars. #30

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

Merged
merged 6 commits into from
Mar 30, 2024
Merged

Add Enemy health bars. #30

merged 6 commits into from
Mar 30, 2024

Conversation

karnkaul
Copy link
Member

@karnkaul karnkaul commented Mar 29, 2024

Refactor progress bars to use a single rounded quad as the background, and a shrinking progress quad.
Also balance weapons and enemy HP a bit: buff beam, buff creep HP, nerf kinetic.

Screencap:

recording-2024-03-30_10.48.52.mp4

Close #28

@karnkaul karnkaul marked this pull request as ready for review March 29, 2024 14:05
@karnkaul karnkaul requested a review from swagween March 29, 2024 14:05
Copy link
Contributor

@swagween swagween left a comment

Choose a reason for hiding this comment

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

looks good!

@@ -15,6 +17,9 @@ Home::Home(bave::App& app, Services const& services) : Scene(app, services, "Hom
auto const& resources = services.get<Resources>();
auto const tasks = std::array{
util::create_font_atlas_task(resources.main_font, {TextHeight{100}, TextHeight{60}}),
std::function<void()>{[] { std::this_thread::sleep_for(1s); }},
Copy link
Contributor

Choose a reason for hiding this comment

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

what's this doing?

Copy link
Member Author

Choose a reason for hiding this comment

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

Oops this needs to be reverted, nice catch! It's basically test code to "see" the progress bar update when loading the home scene. Since there's only one task we never get to see it move.

@karnkaul karnkaul force-pushed the karnage/health-bars branch from 20f7a7b to 7f4688f Compare March 30, 2024 12:22
@karnkaul karnkaul enabled auto-merge (squash) March 30, 2024 12:22
@karnkaul karnkaul merged commit 9eb333a into main Mar 30, 2024
5 checks passed
@karnkaul karnkaul deleted the karnage/health-bars branch March 30, 2024 12:27
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.

Health bars
2 participants