8000 Improvement: Expanded Briefing Room Tutorial to Include Support Points & Reinforcing Scenarios; Fixed Formatting by IllianiBird · Pull Request #6911 · MegaMek/mekhq · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Improvement: Expanded Briefing Room Tutorial to Include Support Points & Reinforcing Scenarios; Fixed Formatting #6911

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

Merged
merged 4 commits into from
May 8, 2025

Conversation

IllianiBird
Copy link
Collaborator

The formatting was fine when tested, but became wonky once merged most likely due to some merge condition. Either way, it's fixed now.

…s & Reinforcing Scenarios; Fixed Formatting.
@IllianiBird IllianiBird self-assigned this May 7, 2025
@IllianiBird IllianiBird added Bug GUI UX User experience Severity: Medium Issues described as medium severity as per the new issue form labels May 7, 2025
Copy link
codecov bot commented May 7, 2025

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 11.71%. Comparing base (e1ba6b7) to head (428ebff).
Report is 42 commits behind head on master.

Files with missing lines Patch % Lines
MekHQ/src/mekhq/gui/view/MissionViewPanel.java 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #6911      +/-   ##
============================================
- Coverage     11.97%   11.71%   -0.26%     
+ Complexity     6785     6673     -112     
============================================
  Files          1101     1101              
  Lines        141414   141413       -1     
  Branches      21901    21901              
============================================
- Hits          16937    16571     -366     
- Misses       122677   123141     +464     
+ Partials       1800     1701      -99     

☔ 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.

@HammerGS HammerGS requested a review from Copilot May 7, 2025 23:51
Copy link
Contributor
@Copilot 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

This PR expands the tutorial content to include Support Points and Reinforcing Scenarios details while also addressing formatting issues in the MissionViewPanel.

  • Adjusts UI layout in MissionViewPanel.java by modifying gridBagConstraints and removing an unnecessary horizontal spacer.
  • Extends the tutorial text in ContractViewPanel.properties to provide additional guidance on support mechanics in StratCon.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
MekHQ/src/mekhq/gui/view/MissionViewPanel.java Adjusts UI layout settings and updates header comments for correct attribution.
MekHQ/resources/mekhq/resources/ContractViewPanel.properties Adds expanded tutorial content for Support Points and Reinforcing Scenarios.
Comments suppressed due to low confidence (1)

MekHQ/src/mekhq/gui/view/MissionViewPanel.java:173

  • Verify that increasing the weightx value from 1.0 to 2.0 aligns with the intended UI layout behavior across different window sizes.
            gridBagConstraints.weightx = 2.0;

@HammerGS HammerGS merged commit 483df2e into MegaMek:master May 8, 2025
6 checks passed
@IllianiBird IllianiBird deleted the tutorialFormatting branch June 7, 2025 22:46
Sign 6FEE up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug GUI Severity: Medium Issues described as medium severity as per the new issue form UX User experience
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0