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.

Sign up for GitHub

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

Jump to bottom

Crit cell display improvements #1382

Merged

neoancient merged 1 commit into MegaMek:master from SJuliez:crit-cell-imp

Jan 16, 2024

Member

SJuliez commented Jan 11, 2024

This slightly changes crit cell display (not for record sheets, only for construction) to

not show "1 shot" for Coolant Pods (that are implemented as AmmoType)
use the equipment short name if the standard name is longer than 22 characters (magic number). This affects only a handful of equipment like AES or HarJel III. The tooltip still shows the full equipment name


          do not write "shots" for coolant pods, use short name when name is long

d36a492

neoancient approved these changes

View reviewed changes

neoancient merged commit 4f3aad9 into MegaMek:master

3 of 4 checks passed

SJuliez deleted the crit-cell-imp branch

March 9, 2024 09:23

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet