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

Fix for 3095 - no +1 TMM for jumping WiGE #4912

Merged

Conversation

Sleet01
Copy link
Collaborator

@Sleet01 Sleet01 commented Nov 20, 2023

Fixes issue #3095, but only for non-airborne WiGE that jump. Airborne jumping WiGE are already getting a +1 TMM from being airborne and these are not cumulative with each other (TW pg 307, ATTACK MODIFIERS TABLE).

Also fixes an issue where immobile / crashing WiGE are treated as other land vehicles and sink immediately; according to TW pg 55 they float and treat water as clear terrain, acting as naval vessels when landed on water, and so do not sink due to being immobile (q.v.).

Close #3095.

Fixes issue MegaMek#3095, but only for non-airborne WiGE that jump.
Airborne jumping WiGE are already getting a +1 TMM from being airborne.

Also fixes an issue where immobile / crashing WiGE are treated as other
land vehicles and sink immediately; in fact by TW pg 55 they float and
treat water as clear terrain and do not sink due to being immoble (q.v.)

Close MegaMek#3095.
@neoancient neoancient merged commit d0ae1e8 into MegaMek:master Nov 22, 2023
4 checks passed
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

Successfully merging this pull request may close these issues.

Jumped WIGE does not get to-hit modifier
3 participants