suriyun-production / mmorpg-kit-docs

This is document for MMORPG KIT project (https://www.assetstore.unity3d.com/#!/content/110188?aid=1100lGeN)
https://suriyun-production.github.io/mmorpg-kit-docs
48 stars 11 forks source link

[Small paid request]Skill Damage #2453

Closed moepi2k closed 5 months ago

moepi2k commented 5 months ago

we discused this already. also in discord we had that many times. its a small change with a big improvement which really laks in kit atm when work with skills. and its something everyone can use

example. You have 2 kind of weapons

Weapon A -> 10 Slashing Damage Weapon B -> 10 Pircing Damage

then you have 1 skill, lets say Power Attack. This skill can be used with both weapons. and the skill should do additional damage to the weapon damage. like +150% Weapon damage. So here comes to problem. you have that option already but you need to assign a damage element. which will override the damage element from weapon. lets say we assign Slashing Damage to the Skill then the damage output would be

Weapon A -> 15 Slashing Damage Weapon B -> 15 Slashing Damage (instead of Pircing)

https://gyazo.com/371c9b79f89ec68942cdb311bbf538d5

even in kits demo, you use it without damage element. which uses the default damage element. but the problem with this is you can not make a skill which be used for diffrent kind of weapon damage.

i think a simple damage modificator would be used or if you keep the damage element empty in the weapon damage infliction it just use the damage element from weapon.

insthync commented 5 months ago

Can you just try to implement it and make a pull request ?

You said it is simple so I hope you will.

moepi2k commented 5 months ago

jup i will do

moepi2k commented 5 months ago

okey, it seems its not a simple task :P all the damage multiplier needs a damage element. sadly. i need to rethink then how i should setup my skills.