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

Less permissive color validator #8380

Merged
merged 1 commit into from
Jun 8, 2015
Merged

Conversation

reaperrr
Copy link
Contributor

@reaperrr reaperrr commented Jun 5, 2015

Enforces larger difference to terrain color and higher minimum lumination.

Closes #7902.

@Mailaender
Copy link
Member

👍 The new values seem to yield better results.

@obrakmann
Copy link
Contributor

That's better, but it still allows setting this:

screenshot from 2015-06-05 22 26 40

I played around a bit and got satisfying results with 0xA0 as threshold (and 0x80 for lower bound). That would allow setting colors like below, but that's about the closest you get:

screenshot from 2015-06-05 22 30 52

@reaperrr
Copy link
Contributor Author

reaperrr commented Jun 8, 2015

Updated, changed the values to 0x70 for threshold and 0x80 for lower bound. As mentioned on irc, the higher the threshold, the more difficult it becomes to choose a GDI-ish color on TD desert maps, so I tried to find a middle-ground. This should still be significantly better than the values I used initially, maybe enough that players trying to exploit colors might get annoyed about the rejected colors and give up eventually (or settle for less advantageous colors).

@obrakmann
Copy link
Contributor

👍

obrakmann added a commit that referenced this pull request Jun 8, 2015
@obrakmann obrakmann merged commit 5f45de1 into OpenRA:bleed Jun 8, 2015
@obrakmann
Copy link
Contributor

Changelog

@xan2622
Copy link
Contributor

xan2622 commented Jun 8, 2015

Is it normal that the "color chooser" hides the swatches after picking a color?

screenshot001

@Mailaender
Copy link
Member

No, that is a glitch: #4451. I am not sure, but #8425 might fix it.

@reaperrr reaperrr deleted the color-valid-tweak branch July 16, 2015 22:53
@AoAGeneral AoAGeneral mentioned this pull request Oct 31, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants