Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sneak suit to-hit penalties are not shown on infantry in hex map #6077

Open
7 tasks done
BlueThing00 opened this issue Oct 7, 2024 · 1 comment
Open
7 tasks done

Comments

@BlueThing00
Copy link

Prerequisites and Pre-Issue Checklist

  • I'm reporting the issue to the correct repository:

  • MegaMek

  • MegaMekLab

  • MekHQ

  • I've tested the issue against at least the latest MILESTONE version

  • I've asked on the MegaMek Discord about the error

  • I've reviewed the BattleTech rules and MegaMek documentation, and I've confirmed that something isn't working as intended.

  • I've searched the Github tracker and haven't found the issue listed

Severity *

Medium (Gameplay Limitation): Non-core functionality is impaired, providing a suboptimal but playable experience.

Brief Description *

image

I attached a saved game replicating this. The infantry with the sneak suit displays the same to-hit as the one without on the hex view.

The to-hit numbers look correct when actually targeting the infantry (and are correct when resolving attacks).

Steps to Reproduce

Load the attached saved games, or field appropriate infantry.

Operating System *

Windows 11

Java Version *

17

MegaMek Suite Version *

v0.50.0

Custom MegaMek Version

No response

Attach Files

bug_infantry_to_hit.sav.gz

Final Checklist

  • I've checked to make sure that this issue has not already been filed
  • I'm reporting only one issue in this ticket for clarity and focus
@gsparks3
Copy link
Collaborator

gsparks3 commented Oct 7, 2024

Any range-based modifiers are not included in the hexmap modifier display (the blue numbers) by design, as they would vary depending on the weapon selected. However, the camo modifiers apply to all attacks, and therefore should be included, so that portion is a valid issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants