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 customization for colors used throughout MekHQ #1480 #1491

Merged

Conversation

sixlettervariables
Copy link
Contributor

This allows customization (without a UX at the moment) for colors throughout MekHQ, providing the backend work to #1480. Each property has a main and alternate color, currently used for backgrounds and foregrounds respectively. The values are saved in mmconf/mekhq.preferences as a pair of colors separated by a pipe, e.g. #ff0000|#000000. Each of the colors provides a default value based on our original colors.

@lgtm-com
Copy link

lgtm-com bot commented Feb 16, 2020

This pull request introduces 2 alerts and fixes 1 when merging c997c91 into d1c4c5f - view on LGTM.com

new alerts:

  • 2 for Spurious Javadoc @param tags

fixed alerts:

  • 1 for Dereferenced variable may be null


@Override
protected void dispose() {
// TODO Auto-generated method stub
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this TODO needed?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Unlikely.

@lgtm-com
Copy link

lgtm-com bot commented Feb 17, 2020

This pull request fixes 1 alert when merging fc48f6a into d1c4c5f - view on LGTM.com

fixed alerts:

  • 1 for Dereferenced variable may be null

Copy link
Member

@NickAragua NickAragua left a comment

Choose a reason for hiding this comment

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

Can't wait for all the NPEs. ;)

@sixlettervariables sixlettervariables merged commit ff0612c into MegaMek:master Feb 22, 2020
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