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

Left hand setup in multiplayer swaps left and right arms (and held items) for others looking at the left handed player #296

Open
treyo-dev opened this issue Aug 29, 2024 · 4 comments
Labels
bug Something isn't working

Comments

@treyo-dev
Copy link

If you have the Skin Customization set to Main Hand: left and the VR Controller Reverse Hands set to On, then in a LAN game a player looking at the left handed VR player will see his arms (and held items) as swapped, with the left arm on the right side and the right arm on the left side.

This occurs in Minecraft 1.21 with the vivecraft-1.21-1.1.11-b2-fabric.jar.

Here's a video clip showing the issue.

VivecraftLeftHandBug.mp4
@fayer3
Copy link
Member

fayer3 commented Aug 29, 2024

yeah, the vanilla main hand option currently isn't well supported

@fayer3 fayer3 added the bug Something isn't working label Aug 29, 2024
@treyo-dev
Copy link
Author

I went ahead and checked out the code and made a fix for this

Do you have rules or guidelines around the handling of pull requests?

@fayer3
Copy link
Member

fayer3 commented Sep 5, 2024

don't think that is really the right way of solving that
the issues is in more places than just that, and what you did doesn't fix the root cause I think
the issue wit hthe skin main hand is 2 fold

  • we don't check it client side at all, the item is always inthe right hand (left hand with reversed hands)
  • reversed hands doesn't flip the item transforms for 3rd person

@treyo-dev
Copy link
Author

You're absolutely right. My fix is definitely more of a hack and not fixing the root problem.

You're welcome to the change if you want it, but I completely understand holding off for a more proper fix.

Maybe at some point I'll try taking a deeper look, but I don't have the time to do so at this current moment in time. So I'll just plan on leaving it at that for now.

Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants
@treyo-dev @fayer3 and others