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

Clarify map dimensions in scenario displays #5040

Merged
merged 1 commit into from
Oct 13, 2024

Conversation

WeaverThree
Copy link
Contributor

Minor tweak to show which is width and which is height, because we always get confused here and it matters for cross-the-map objectives.

2024-10-12_160445

@codecov-commenter
Copy link

codecov-commenter commented Oct 12, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 10.43%. Comparing base (3e8168d) to head (dd31e60).
Report is 15 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #5040      +/-   ##
============================================
- Coverage     10.43%   10.43%   -0.01%     
- Complexity     6022     6024       +2     
============================================
  Files           952      952              
  Lines        133810   133873      +63     
  Branches      19432    19429       -3     
============================================
+ Hits          13961    13966       +5     
- Misses       118498   118559      +61     
+ Partials       1351     1348       -3     

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

@HammerGS HammerGS merged commit 6b68f97 into MegaMek:master Oct 13, 2024
4 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.

3 participants