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

Parts In Use - Filter Mothballed and Spare Part Quality #5055

Merged
merged 9 commits into from
Oct 18, 2024

Conversation

WeaverThree
Copy link
Contributor

@WeaverThree WeaverThree commented Oct 16, 2024

This PR depends on #5052 - the changes should look much smaller once that one is applied.

Implements #5026.

Adds options to the Parts In Use dialog to ignore parts on mothballed units and/or ignore spare parts under a given quality. The latter will be much more useful once we get random part quality implemented, but I decided to cut this out as a complete unit before then since it's all UI.

It also makes the dialog a little bigger and tweaks the sizes of columns a little, since I find this window needs fiddling with every time it's opened, hopefully this will lessen that.

Example Images:

2024-10-16_150449
2024-10-16_150454
2024-10-16_150503

@codecov-commenter
Copy link

codecov-commenter commented Oct 16, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 10.42%. Comparing base (e72f7fd) to head (0b4d2c4).
Report is 34 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #5055      +/-   ##
============================================
- Coverage     10.44%   10.42%   -0.02%     
+ Complexity     6039     6033       -6     
============================================
  Files           953      953              
  Lines        133945   134106     +161     
  Branches      19431    19434       +3     
============================================
- Hits          13990    13986       -4     
- Misses       118611   118772     +161     
- Partials       1344     1348       +4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@WeaverThree
Copy link
Contributor Author

#5052 was merged so I foded master back into this so it only shows relevant changes now

Copy link
Collaborator

@IllianiCBT IllianiCBT left a comment

Choose a reason for hiding this comment

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

Great work as always, just some minor points of feedback.

MekHQ/src/mekhq/campaign/Campaign.java Outdated Show resolved Hide resolved
MekHQ/src/mekhq/campaign/Campaign.java Outdated Show resolved Hide resolved
MekHQ/src/mekhq/campaign/Campaign.java Outdated Show resolved Hide resolved
MekHQ/src/mekhq/campaign/Campaign.java Outdated Show resolved Hide resolved
MekHQ/src/mekhq/campaign/Campaign.java Outdated Show resolved Hide resolved
MekHQ/src/mekhq/campaign/Campaign.java Show resolved Hide resolved
MekHQ/src/mekhq/gui/dialog/PartsReportDialog.java Outdated Show resolved Hide resolved
Copy link
Collaborator

@IllianiCBT IllianiCBT left a comment

Choose a reason for hiding this comment

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

LGTM

@HammerGS HammerGS merged commit 63ac932 into MegaMek:master Oct 18, 2024
4 checks passed
@WeaverThree WeaverThree deleted the wvr-partsinuse-options-b branch October 18, 2024 01:13
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.

4 participants