javierriveracastro / betteroll-swade

A Better Rolls port for SWADE
GNU General Public License v3.0
16 stars 32 forks source link

Martial Artist/Warrior are adding an extra damage die #601

Closed ddbrown30 closed 1 year ago

ddbrown30 commented 1 year ago

The Martial Artist edge should give the character a d4 damage die or upgrade it if they already have it from something like Claws or Brawler. The Martial Warrior edge then upgrades that die another type.

What BR2 is doing is assuming that MA is always giving a d4 and MW is always giving a d6, which is incorrect and just makes everything more complicated to deal with if you're using anything other than the simplest case.

The additional dice should be removed and should be instead manually implemented in item actions like other similar edges.

javierriveracastro commented 1 year ago

I'm not sure about that. It makes sense but it can also broke some people setup. Let me think a little more about it.

ddbrown30 commented 1 year ago

Yeah, fair enough. Always tough to make a change like this since you can't please everyone.

javierriveracastro commented 1 year ago

I have asked a bit around. Everybody believes this is a good change. They also believe that it will break some people games and that sucks.

I have really no idea of what to do about this. Probably accept it as I believe not a lot of people are relaying on it.

ddbrown30 commented 1 year ago

Yeah, my vote should be obvious but I would just make the change. It's an easy fix for anyone who was relying on it vs an unsolvable problem for people who aren't.