AzerothWarsLR / WarcraftLegacies

This is a complete rewrite of the Warcraft 3 map Azeroth Wars: Legacy Reborn written in C#.
MIT License
12 stars 7 forks source link

Siege Walker name is misleading based on its attack type(s) #3104

Open Technopig1992 opened 2 months ago

YakaryBovine commented 2 months ago

Not quite prepared to call this one a bug but definitely worth throwing in an improvement for.

Technopig1992 commented 2 months ago

I tend to forget about these small ones when doing some "fine tuning" so just here as a reminder

vampirika commented 2 months ago

Just double checked.

Siege tank (Kul'tiras) does have siege damage. I incorrectly assumed "Assault tank" (goblins) was called siege tank as everyone calls it goblin siege. Alternatively, the bipedal star-wars looking ones (siege walker) do not have siege either.

@Technopig1992 can maybe take to a discord chat in how we might wanna proceed with these.