SilentChaos512 / Silent-Gear

Modular tool/armor mod (formerly part of Silent's Gems)
https://www.curseforge.com/minecraft/mc-mods/silent-gear
MIT License
84 stars 38 forks source link

[Neoforge] Spear with Better Combat acts as a two-handed weapon despite having a tiny model #709

Open v3xed-v0id opened 1 month ago

v3xed-v0id commented 1 month ago

Versions

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

the problem: 2024-10-11_00 11 15 2024-10-11_00 11 32

solution proposal: 2024-10-11_00 11 24 2024-10-11_00 11 18

Steps to Reproduce the Problem

  1. boot up game
  2. swing spear once acquired
  3. observe
v3xed-v0id commented 1 month ago

i appreciate you adding better combat support in the latest update btw! :D

ffuentesm commented 1 month ago

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" } ] } }