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

Add option to show unit images in TO&E #5024

Merged
merged 2 commits into from
Oct 11, 2024

Conversation

WeaverThree
Copy link
Contributor

Another Weaver-annoying low-hanging fruit, this one adds an option to show the images of the actual units in the TO&E rather than people's portraits. The portraits do nothing to help me identify which units I'm looking at. Plus half the time I'm playing with portrait generation disabled so everyone is a silhouette.

2024-10-10_170051

@codecov-commenter
Copy link

codecov-commenter commented Oct 11, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 10.46%. Comparing base (c2c2f4b) to head (c7fcda5).
Report is 61 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #5024      +/-   ##
============================================
- Coverage     10.46%   10.46%   -0.01%     
- Complexity     6022     6026       +4     
============================================
  Files           950      950              
  Lines        133437   133481      +44     
  Branches      19383    19393      +10     
============================================
+ Hits          13967    13969       +2     
- Misses       118124   118163      +39     
- Partials       1346     1349       +3     

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

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
Copy link
Member

This has picked up conflicts.

@WeaverThree
Copy link
Contributor Author

Conflicts resolved

@IllianiCBT IllianiCBT merged commit a619e8a into MegaMek:master Oct 11, 2024
4 checks passed
@WeaverThree WeaverThree deleted the wvr-toe-icons branch October 16, 2024 17:24
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