Skip to content

feat: add enemy health bar system - #36

Merged
AhmedSobhy01 merged 9 commits into
mainfrom
feat/enemy-health-bar
Apr 19, 2026
Merged

feat: add enemy health bar system#36
AhmedSobhy01 merged 9 commits into
mainfrom
feat/enemy-health-bar

Conversation

@AhmedSobhy01

Copy link
Copy Markdown
Collaborator
  • Add a new EnemyHealthBarSystem that renders screen-space health bars above enemies
image image image

@AhmedSobhy01
AhmedSobhy01 force-pushed the feat/enemy-health-bar branch 4 times, most recently from 6f68253 to 6f3a2f3 Compare April 18, 2026 22:37
@AhmedSobhy01
AhmedSobhy01 force-pushed the feat/enemy-health-bar branch from 6f3a2f3 to 271a1d8 Compare April 18, 2026 22:39

@AhmedAmrNabil AhmedAmrNabil left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

checked code, will run to test now

Comment thread src/common/systems/ui-renderer.hpp
Comment thread src/game/systems/enemy-health-bar.hpp Outdated
Comment thread src/game/systems/enemy-health-bar.hpp Outdated
Comment thread src/states/play-state.hpp
AhmedAmrNabil
AhmedAmrNabil previously approved these changes Apr 19, 2026

@AhmedAmrNabil AhmedAmrNabil left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

LGBT

Comment thread src/game/systems/enemy-health-bar.hpp Outdated
LoayAhmed304
LoayAhmed304 previously approved these changes Apr 19, 2026

@LoayAhmed304 LoayAhmed304 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGBTQ

@LoayAhmed304
LoayAhmed304 dismissed their stale review April 19, 2026 10:07

Pending Changes

@AhmedAmrNabil AhmedAmrNabil left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

LGBTQ+

@LoayAhmed304 LoayAhmed304 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGBTQIA+

@AhmedSobhy01
AhmedSobhy01 merged commit 23c8b55 into main Apr 19, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants