Projectile Damage Attribute [Fabric & Forge]

Projectile Damage Attribute [Fabric & Forge]

12M Downloads

Power enchantment compat

ConqAra opened this issue ยท 8 comments

commented

Can you make it so power enchantment displays the additional projectile damage like sharpness does for swords/axes? Would be really nice qol!

commented

Does sharpness do that?
Can you provide an example screenshot or something please?

commented

image

Yeah it shows the extra damage in the dmg tool tip, instead of 7 it shows 10 when you have sharpness

commented

Although it was modded, so ill try it really quick with vanilla client to make sure

commented

image
Yup, does it in vanilla too!

commented

After additional consideration, I think this would be incorrect to implement.
Considering that armor can also grant projectile damage, there is no way to accurately display the projectile damage combined from every sources with power enchantment.

commented

I mean, damage is also not correctly displayed regardless because smite (would only provide the additional dmg for certain mobs) and other modded enchantments, that do extra damage that it wont account for, will not display the change. minecraft doesn't properly display damage either in that case, its more of a "this item with this enchantment specifically does this much dmg" and then its up to the player to do the math afterwards with all those other enchantments/effects that alter attributes.

detecting and displaying the result of math for every source of damage is unreasonable. it was more of a simple quality of life to see your damage for power just like sharpness. it doesn't have to be 100% correct its just nice to have.

commented

I'm not here to argue though, it's completely up to you to implement or not. I was just giving my insight on it.

commented

I was thinking long about this, and there is no way this could work anything other than Vanilla Bow. :(
Power Enchantment is considered with arbitrary multiplication values, so modded bow can make any calculation based on the applied level. As long as Vanilla Minecraft doesn't change to allow any kind of support for this, I cannot implement this.

Thanks for the suggestion tho, the idea itself is great!