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

Cleaning up a few erroneous Campaign GUI properties #2656

Merged

Conversation

Windchild292
Copy link
Contributor

The removed lines were unused, and the dialog tags were the only ones to load dialogs without the ...

@Windchild292 Windchild292 added (RFE) Enhancement Requests for Enhancement, new features or implementations GUI labels May 29, 2021
@Windchild292 Windchild292 self-assigned this May 29, 2021
@codecov
Copy link

codecov bot commented May 29, 2021

Codecov Report

Merging #2656 (feb4112) into master (2fa74ec) will decrease coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #2656      +/-   ##
============================================
- Coverage     10.78%   10.77%   -0.01%     
+ Complexity     3824     3823       -1     
============================================
  Files           703      703              
  Lines         97926    97926              
  Branches      16272    16272              
============================================
- Hits          10557    10556       -1     
  Misses        85991    85991              
- Partials       1378     1379       +1     
Impacted Files Coverage Δ
...ekhq/campaign/universe/RandomFactionGenerator.java 58.40% <0.00%> (-0.40%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2fa74ec...feb4112. Read the comment docs.

@@ -75,23 +75,19 @@ miCargoReport.text=Cargo Report...
# View Menu
menuView.text=View
miShowHistoricalReportLog.text=Historical Daily Report Log
miDetachLog.text=Detach Daily Report Log
miAttachLog.text=Attach Daily Report Log
Copy link
Member

Choose a reason for hiding this comment

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

We really ought to look at making the daily log detachable again. It's such a pain in the ass to have to keep going back to the command center tab for it.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I concur, and will open up an issue to open it in its own modeless window.

@Windchild292 Windchild292 merged commit add047d into MegaMek:master May 31, 2021
@Windchild292 Windchild292 deleted the dev_Windchild_CampaignGUIProperties branch May 31, 2021 01:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
GUI (RFE) Enhancement Requests for Enhancement, new features or implementations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants