search

Friendly0Fire / GW2Clarity

MIT License
34 stars 15 forks source link

Update BuffsList.inc / Asset for Spear Buff #59

Closed Gramgoyle closed 2 months ago

Gramgoyle commented 2 months ago

Added new Buff from Spear Skill to some professions (Engineer, Mesmer, Necromancer, Guardian, Revenant, Ranger)

First time to do dis kind of thing please double check >.<. Tried to upload the buff icon to asset but it said no push . So I guess I just upload it here ?

thank you for making Clarity ♥

Abyssal_Raze Clarity Focused Illuminated Panther's_Prowl Soul_Shards

Gramgoyle commented 2 months ago

OwO I just notice some new buff that I can add for spear

Gramgoyle commented 2 months ago

Additional icon for buffs

So for Energize and Harden in game it still place holder so I use the skill icon for it instead (just like Seethe and Ripple in-game)

Distracting_Throw Energize Harden Lightning_Rod_(engineer_spear_skill) Ripple Seethe Symbol_of_Luminance_(effect)

Friendly0Fire commented 2 months ago

Can you please verify with the PR build whether the buffs work correctly? I currently don't have spear builds setup so I can't easily check.

https://github.com/Friendly0Fire/GW2Clarity/actions/runs/10550034318/artifacts/1852274262

Gramgoyle commented 2 months ago

All Spear buff works correctly :D Thank you! Sorry for the typo >.<