search

AshDevFr / encounters_app

https://encounters.4sh.dev
2 stars 0 forks source link

Dex modifier is being added to heavy armour on the dm screen #21

Open RNPGhost opened 3 years ago

RNPGhost commented 3 years ago

Describe the bug Heavy armor in D&D does not consider the dex modifier when calculating AC. This is correctly represented on the D&D Beyond page, however the DM screen incorrectly adds the dex modifier to the calculation.

To Reproduce Steps to reproduce the behavior:

  1. Create a character with a non-zero dex modifier.
  2. Equip heavy armor to this character.
  3. Note the AC provided on the D&D Beyond page.
  4. Add the character to a campaign.
  5. Navigate to the campaign screen with the extension enabled.
  6. Note the AC displayed on the DM campaign screen is different from the AC given by D&D Beyond in that it is equal to the correct AC + dex modifier.

Expected behavior The AC displayed on the DM campaign screen for characters wearing heavy armor should match the AC given in D&D Beyond, in that it should not include the dex modifier in the calculation.

Screenshots Correct AC dndbeyond_correct_ac Incorrect AC dm_screen_incorrect_ac

Desktop (please complete the following information):