Skip to content
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

Feat/camera #64

Merged
merged 12 commits into from
May 1, 2024
Merged

Feat/camera #64

merged 12 commits into from
May 1, 2024

Conversation

dmin12
Copy link
Contributor

@dmin12 dmin12 commented Apr 30, 2024

Implemented basic camera functionality and movement. The user is able to control lateral movement with WASD and change the viewing angle using the mouse.

The camera uses the player object (same as current session) to send and receive updates on position, and also sends a vector representing which direction the player is facing.

A new event type ChangeFacingEvent was added to events.hpp to support the viewing angle updates.

This branch also contains changes from feat/creature-player-enemy-tree.

@dmin12 dmin12 changed the base branch from main to dev April 30, 2024 18:49
Copy link
Contributor

@Tyler-Lentz Tyler-Lentz left a comment

Choose a reason for hiding this comment

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

Very very epic PR. It's good to see the events started to get messed with by everyone

My wayland getting bricked is a little less epic, but Xorg works 🤷

Copy link
Contributor

@jhparkt jhparkt left a comment

Choose a reason for hiding this comment

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

Great work. One of the most important part of our game accomplished. Camera works well on my end (windows), and I think I can start to integrate movement that I made with the camera so that we can have first person camera movement!

@dmin12 dmin12 merged commit 642e87b into dev May 1, 2024
10 of 11 checks passed
@Tyler-Lentz Tyler-Lentz deleted the feat/camera branch May 2, 2024 16:51
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.

4 participants