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

sc2: Adding settings to control the mapping of race to colour in the launcher GUI #361

Merged
merged 6 commits into from
Dec 3, 2024

Conversation

MatthewMarinets
Copy link

@MatthewMarinets MatthewMarinets commented Dec 3, 2024

What is this fixing or adding?

New sc2 settings in host.yaml that allow controlling the colour of buttons in the GUI. I find the colours quite dark and murky and like some over-the-top colours myself. I admit the preset colours may be a little too bright, but I'm not sure how to get a good set of colours that allow for bright highlights while keeping the overall colours sane to look at. It may be that the best route is to allow additional configuration, like different colours for no-builds or fine control over borders.

How was this tested?

Changed the colours around a lot and restarted the client.

If this makes graphical changes, please attach screenshots.

image
image

image
image

image

image

image

default_colours = {
'TERRAN': (0.0838, 0.2898, 0.2346),
'ZERG': (0.345, 0.22425, 0.12765),
'PROTOSS': (0.18975, 0.2415, 0.345),
Copy link
Author

@MatthewMarinets MatthewMarinets Dec 3, 2024

Choose a reason for hiding this comment

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

Working with settings is weird; attempting to pass in or import SC2Race caused the client startup to hang for some reason. So I just ended up working with strings, which I'm not fully happy about.

These defaults are the old colours multiplied by 0.345 (the colour component value of the grey button graphic).

@Ziktofel
Copy link
Owner

Ziktofel commented Dec 3, 2024

colour -> color

@Ziktofel Ziktofel merged commit 47754d9 into Ziktofel:sc2-next Dec 3, 2024
12 checks passed
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.

2 participants