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

Fix avatar attachment positions. #181

Merged
merged 15 commits into from
Sep 17, 2023
Merged

Conversation

Gigabyte5671
Copy link
Member

@Gigabyte5671 Gigabyte5671 commented Sep 13, 2023

This PR fixes the attachment of the camera and nametag to the player's avatar.
Previous:
nametags float far above the avatar's head when sitting

Updated:
nametags are correctly positioned above the avatar's head when sitting

Video example of the new camera attachment.


A "Camera Bobbing" toggle has been added to the graphics settings menu. This toggles the bouncing/bobbing effect visible in first-person view that results from the new camera attachment method. (This setting is enabled by default).
image


This PR also adds a rolling average hysteresis utility class. This utility acts as a low-pass filter, smoothing values that change frequently.


Closes #180.
Closes #142.

Provides a simple rolling average hysteresis.
Allows the nametag to move up/down along with animations.

Related to #142.
To consolidate metaverse API methods.
To consolidate entities and components.
This reverts commit 1beb5e8.

This commit introduced a circular dependency that prevented the app from starting.
I'm reverting it because I could not fix it via conventional changes
@Gigabyte5671 Gigabyte5671 added bug Something isn't working user experience labels Sep 13, 2023
@Gigabyte5671 Gigabyte5671 marked this pull request as ready for review September 15, 2023 04:47
@digisomni
Copy link
Member

We should put the setting in the control settings as it's more a UX thing than a graphical thing.

@Gigabyte5671
Copy link
Member Author

Sounds good. Where abouts in the control settings menu do you think it should go?

@Gigabyte5671 Gigabyte5671 added the question Further information is requested label Sep 16, 2023
@digisomni
Copy link
Member

Let's add a tab for "Camera" with an eye icon.

@Gigabyte5671
Copy link
Member Author

Let's add a tab for "Camera" with an eye icon.

Added with commit 20dd0c7.

@Gigabyte5671 Gigabyte5671 removed the question Further information is requested label Sep 17, 2023
@digisomni digisomni merged commit 5baa8cb into master Sep 17, 2023
3 checks passed
@Gigabyte5671 Gigabyte5671 deleted the fix/avatar-attachment-positions branch September 17, 2023 11:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working user experience
Projects
None yet
2 participants