search

Solaris-Skunk-Werks / solarisskunkwerks

66 stars 26 forks source link

Mixing ATMs and MMLs only gives you one of their lists of ammo types in weapon box #314

Open mrpopogod opened 12 months ago

mrpopogod commented 12 months ago

Describe the bug A mech running MMLs or ATMs gets all the ammo types listed in the attack box for easy understanding of damage and range. If you combine both on a mech only the first one in the list will have its ammo types listed out.

To Reproduce Steps to reproduce the behavior:

  1. Create a new mech, Experimental, Dark Ages, Mixed Tech
  2. Add an MML and an ATM launcher to the mech.
  3. Print Preview - see that only the top launcher in the list has its ammo listed out (either SRM/LRM or Std/HE/ER)

Expected behavior First ATM and first MML entry should list out all ammo types with their ranges/damages.

Additional context When a mech has multiple ATMs/MMLs only the first one gets the enumeration to save space (e.g. the Turkina Z). It seems that check should be done on a per-type basis, rather than a global "did we enumerate once?".