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 FlatLaf GUi scaling to MHQ #4990

Merged
merged 7 commits into from
Oct 11, 2024
Merged

Conversation

SJuliez
Copy link
Member

@SJuliez SJuliez commented Oct 5, 2024

This adds FlatLaf's GUI scaling to MHQ. It shares the scale value with MM and MML, i.e. changing it in one program changes it for the others too. The scale is set in the MekHQ options, display. This PR also removes hardcoded GUI sizes where they interfere with scaling. There are certain to be some mis-sized elements left but generally it seems to work well.

@codecov-commenter
Copy link

codecov-commenter commented Oct 5, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 10.44%. Comparing base (64f8158) to head (db2252b).
Report is 25 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #4990      +/-   ##
============================================
- Coverage     10.46%   10.44%   -0.02%     
+ Complexity     6022     6009      -13     
============================================
  Files           951      950       -1     
  Lines        133345   133437      +92     
  Branches      19378    19383       +5     
============================================
- Hits          13955    13944      -11     
- Misses       118041   118142     +101     
- Partials       1349     1351       +2     

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

@repligator repligator added the GUI label Oct 8, 2024
@SJuliez SJuliez marked this pull request as ready for review October 10, 2024 20:03
@HammerGS HammerGS merged commit 71dbfe4 into MegaMek:master Oct 11, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants