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

show movement for all entity types on unit tool tip #4130

Merged
merged 2 commits into from
Jan 26, 2023

Conversation

kuronekochomusuke
Copy link
Collaborator

@kuronekochomusuke kuronekochomusuke commented Jan 22, 2023

  • show movement mode for all entity types on unit tool tip. to make it easier to see what mode unit is in. most useful for units that can convert modes.
  • correct issue of movement reduced bomb indicator incorrectly showing on LAMs.

image

image

@kuronekochomusuke kuronekochomusuke added Bug (RFE) Enhancement Requests for Enhancement, new features or implementations GUI User Interface labels Jan 22, 2023
Copy link
Contributor

@Windchild292 Windchild292 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks like there's an unintended change included in this.

@@ -1130,7 +1130,7 @@ public void clearBombChoices() {
@Override
public int reduceMPByBombLoad(int t) {
// bombs don't impact movement
return 0;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That goes against the comment, and doesn't appear related?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the movement it not impacted so, so it should return the same value.

@Windchild292
Copy link
Contributor

I'm just blind apparently, but does that cause issues with the actual movement given the very obvious comment above the changed LAM line.

@Windchild292 Windchild292 merged commit 0206995 into MegaMek:master Jan 26, 2023
@kuronekochomusuke kuronekochomusuke deleted the showMovementMode branch March 3, 2023 23:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug GUI User Interface (RFE) Enhancement Requests for Enhancement, new features or implementations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants