HaloOfBlocks / AdditionalGuns

A community addon for MrCrayfish's Gun Mod for 1.16 and above
https://www.curseforge.com/minecraft/mc-mods/additional-guns
GNU General Public License v3.0
16 stars 23 forks source link

Guns Rendered Incorrectly When Main Hand Option is Set to Left #40

Closed ThickPropheT closed 2 years ago

ThickPropheT commented 2 years ago

2022-06-14_23 25 29

Guns seem to be culled incorrectly when held while Skin Customization... > Main Hand is set to Left.

Additionally, when playing SMP, if ANY player has Main Hand set to Left, players will not be able to see each others' guns.

CasualNoLifer commented 2 years ago

Hello. I am not one of the maintainers for this project but I am a contributor working on fixing this type of issue. The models used in this mod do not currently have a left hand offset identical to the right hand. One of my upcoming projects is to further fix the handheld properties for the guns and this is one of the issues I will be attempting to fix along with some of the guns' ADS angles

It is strange to me that the model is appearing at all because its .json file has it display completely offscreen when set in the left hand. The textures/model also appears to be negative rather than positive, giving it a see-through look. I'll do some testing and play around with it as it's possible that there is no left hand support to begin with for the base mod, Mr. Crayfish's Gun Mod. (I am new to the project and am not certain if there is or is not left-hand support.)

ThickPropheT commented 2 years ago

Thanks for the reply. If it turns out that the base mod also does not support this, I can submit a report there.

Autovw commented 2 years ago

To answer @CasualNoLifer :

The models used in this mod do not currently have a left hand offset identical to the right hand.

This is always something I check for when new models are submitted, but it could be that some of the older models still have issues with this. (including the gun shown in the image)

It is strange to me that the model is appearing at all because its .json file has it display completely offscreen when set in the left hand.

We prefer not setting the left hand value because 1. Minecraft will simply "mirror" the right hand position if the left hand is unset. 2. if we have to change the hand positions, we only have to do it once. 3. it simply saves a few lines of code :)

To answer @ThickPropheT :

Additionally, when playing SMP, if ANY player has Main Hand set to Left, players will not be able to see each others' guns.

This sounds more like an issue with MrCrayfish's Gun Mod, but I'm currently unable to check this. Can you verify this by holding guns from MrCrayfish's Gun Mod instead?

CasualNoLifer commented 2 years ago

Thanks for the reply. If it turns out that the base mod also does not support this, I can submit a report there.

All guns have been edited to work in the left hand now. The updated versions will appear in the next update. :)