Skip to content

Comments

Fixes #1076. Infantry armor panel now only shows custom armors if rules level set to Advanced or higher.#7970

Merged
HammerGS merged 5 commits intoMegaMek:mainfrom
TenkawaBC:InfantryCustomArmorFix
Feb 15, 2026
Merged

Fixes #1076. Infantry armor panel now only shows custom armors if rules level set to Advanced or higher.#7970
HammerGS merged 5 commits intoMegaMek:mainfrom
TenkawaBC:InfantryCustomArmorFix

Conversation

@TenkawaBC
Copy link
Contributor

This is a very old bug report, but it was still valid, and this implements the fix. Fixes #1076

… if rules level set to Advanced or higher.
@TenkawaBC TenkawaBC requested a review from a team as a code owner February 8, 2026 06:36
@codecov
Copy link

codecov bot commented Feb 8, 2026

Codecov Report

❌ Patch coverage is 0% with 22 lines in your changes missing coverage. Please review.
✅ Project coverage is 29.85%. Comparing base (5f34714) to head (2b20dc9).
⚠️ Report is 19 commits behind head on main.

Files with missing lines Patch % Lines
...lient/ui/dialogs/customMek/InfantryArmorPanel.java 0.00% 22 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff            @@
##               main    #7970   +/-   ##
=========================================
  Coverage     29.85%   29.85%           
- Complexity    16880    16882    +2     
=========================================
  Files          3144     3144           
  Lines        303517   303518    +1     
  Branches      53035    53036    +1     
=========================================
+ Hits          90612    90615    +3     
+ Misses       203523   203514    -9     
- Partials       9382     9389    +7     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Fixes #1076 by restricting the Infantry Armor UI so that custom armor kits/options are only available when the game rules tech level is Advanced or higher, preventing Advanced-only armor options from appearing under Standard tech settings.

Changes:

  • Conditionally renders/populates the infantry armor kit + related advanced armor controls based on SimpleTechLevel.
  • Moves specialization checkbox rendering to occur regardless of tech level.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@HammerGS HammerGS merged commit 0eee9b5 into MegaMek:main Feb 15, 2026
9 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.

Infantry Armor Available In Standard Tech

2 participants