Stormforge-gg / Bugtracker-Fordring

8 stars 3 forks source link

[Talent][Warrior] Sweeping strikes #928

Closed bonfax4 closed 2 years ago

bonfax4 commented 2 years ago

Description:

Current behaviour:

  1. Being consumed on-hit with whirlwind / bladestorm.
  2. Sweeping doesn't have a positional requirement. It targets the nearest regardless of positioning.

Expected behaviour:

Whirlwind and bladestorm specifically should consume 1 charge and only cleave 1 instance of damage per spin. Cleave should "cleave" per target hit.

There is and should be a postional requirement for every atack if you're not aoe'ing with whirlwind or bladestorm.

source :

https://wowpedia.fandom.com/wiki/Sweeping_Strikes?oldid=2399581

Wrath of the Lich King Hotfix (2009-12-17): Sweeping Strikes will now only trigger once per whirlwind rotation of Bladestorm.

Regarding positioning https://wowwiki-archive.fandom.com/wiki/Sweeping_Strikes?oldid=2410017

Sweeping Strikes doesn't always work; if you don't see the damage above the second mob's head, chances are the enemies aren't positioned correctly. Try to move around so that they're close together and standing side-by-side to get the effect of the ability.

https://web.archive.org/web/20130810120034/http://elitistjerks.com/f81/t22705-dps_compendium//

Sweeping Strikes has a very short range, no more than a few yards, and seems to function within the 180 degree frontal arc that cleave does. With this in mind it's important to try and be as close as possible to both mobs when using Sweeping Strikes, or targets may end up out of range.

Retail video of warrior using Sweeping strikes very often and you can see how charges arent instantly being consumed without some sort of positioning being in play, he retains stacks while being near pets and othr players very often unless they're somewhat positioned in front: https://www.warcraftmovies.com/movieview.php?id=152529 ( Site is a bit weird to get videos to load so give it time and refresh or try a different browser)

To note that there are 2 ID's for sweeping strikes and i believe one of them is specifically for bladestorm/whirlwhind: https://wotlkdb.com/?spell=12723 https://wotlkdb.com/?spell=26654. Most likely one of these specifically doesnt have a positional requirement for storm/whirlwind.

Even my details addon separates my sweeping strikes dmg when i cast it with bladestorm VS when used with regular atacks.

https://www.mmo-champion.com/threads/2022048-Sweeping-strikes-and-ww Another expansion but they do mention how sweeping strikes id (26654) works specifically for bladestorm and whirlwind.

Revision:

Cold-Martijn commented 2 years ago

rev 1364