-
Notifications
You must be signed in to change notification settings - Fork 170
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
Parts In Use - Filter Mothballed and Spare Part Quality #5055
Conversation
MekHQ/src/mekhq/campaign/universe/generators/companyGenerators/AbstractCompanyGenerator.java
Outdated
Show resolved
Hide resolved
Codecov ReportAll modified and coverable lines are covered by tests ✅
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. |
#5052 was merged so I foded master back into this so it only shows relevant changes now |
There was a problem hiding this 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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
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: