[Neoforge] Spear with Better Combat acts as a two-handed weapon despite having a tiny model
v3xed-v0id opened this issue ยท 2 comments
Versions
- Silent Gear: 4.0.4
- Silent's Gems: N/A
- Silent Lib: 10.3.0
- Forge: Neoforge 21.1.68
- Modpack: N/A
- Optifine Installed: No
Expected Behavior
i would expect the spear to act as a one-handed spear and not a two-handed spear, also with it's current stats it doesnt make sense for the spear to be giving up an entire extra weapon slot (bc its two-handed so ur not able to use a shield with it or dual wield) as its stats are more on par with a broadsword than really anything two-handed, i would recommend changing the spear animation to a one-handed spear animation personally, maybe using the trident animation? this seems like the easiest solution personally
alternatively, if you wanted to lean the spear into being two-handed, maybe elongating the model a bit to make it fit two-handed while maybe buffing the weapon reach and lowering swing speed sounds like a good idea (since u gotta keep in mind that spear means ur giving up a weapon/shield slot cus its two-handed, so making it act more like a two-handed weapon in terms of pure stats seems like a good idea, my opinion)
honestly i would be ok with either of these solutions, u pick which one u like more i think, tho the first one is probably easiest to implement cus i doubt you would have to change textures around n stuff and it would still fit lol
(Description here)
Actual Behavior
Spear instead has a very tiny model, it looks off with better combat when being used since it was assigned to be two-handed while not being given the stats for a proper two-handed weapon as two-handed means you will lack a shield/dual-wield, and the spear was given stats more similar to a one-handed broadsword.
(Description here)
Links/Images
- (Link)
Steps to Reproduce the Problem
- boot up game
- swing spear once acquired
- observe
I think the problem is the animation template used for the spear in this case.
https://github.com/ZsoltMolnarrr/BetterCombat/blob/1.21.1/common/src/main/resources/data/bettercombat/weapon_attributes/spear.json
You can see that it's marked for two handed.
{ "attributes": { "attack_range": 3.5, "pose": "bettercombat:pose_two_handed_polearm", "two_handed": true, "category": "spear", "attacks": [ { "hitbox": "FORWARD_BOX", "damage_multiplier": 1, "angle": 0, "upswing": 0.5, "animation": "bettercombat:two_handed_stab_right", "swing_sound": { "id": "bettercombat:spear_stab" } } ] } }
A way to fix that is to change the "two" for "one" so that it uses the one handed animations.
As you can see in the trident file, it uses one handed stab:
{ "parent": "bettercombat:spear", "attributes" : { "attack_range": 2.75, "pose": "", "two_handed": false, "category": "trident", "attacks": [ { "animation": "bettercombat:one_handed_stab" } ] } }