Skip to content

Monsters#170

Merged
BrentDaMage merged 16 commits intonbcraft-org:masterfrom
lucanero:monsters
Jul 1, 2025
Merged

Monsters#170
BrentDaMage merged 16 commits intonbcraft-org:masterfrom
lucanero:monsters

Conversation

@lucanero
Copy link
Copy Markdown
Contributor

How should I feel?
Creatures lie here
Looking through the window
Monster

lucanero and others added 10 commits June 15, 2025 17:11
* finish Spider and add to EntityRenderDispatcher
* finish Skeleton, Arrow, ArrowRenderer class and add to EntityRenderDispatcher
* instantiate Bow in item init fn
* adjusted HumanoidMobRenderer to support carried items
 * Added platform-agnostic AppPlatform::readAssetFile()
Copy link
Copy Markdown
Member

@BrentDaMage BrentDaMage left a comment

Choose a reason for hiding this comment

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

See code comments.
Additionally, the arrows emit the "drr" sound as soon as they are fired, and upon hitting the ground, they disappear.

Comment thread source/world/entity/Player.cpp Outdated
Comment thread source/world/entity/Skeleton.cpp Outdated
Comment thread source/world/entity/Skeleton.hpp Outdated
Comment thread source/world/entity/Spider.hpp Outdated
Comment thread source/world/entity/Zombie.cpp Outdated
Comment thread source/world/phys/AABB.cpp
Comment thread source/world/entity/Arrow.hpp Outdated
Comment thread source/world/entity/Arrow.cpp Outdated
Comment thread source/world/entity/Arrow.cpp Outdated
Comment thread source/client/renderer/entity/SpiderRenderer.hpp Outdated
Copy link
Copy Markdown
Member

@BrentDaMage BrentDaMage left a comment

Choose a reason for hiding this comment

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

LGTM

@BrentDaMage BrentDaMage merged commit ab513d1 into nbcraft-org:master Jul 1, 2025
7 checks passed
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.

2 participants