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

resolve color active/inactive colors similars #4096

Open
wants to merge 1 commit into
base: master
from

Conversation

@newfrenchy83
Copy link
Contributor

commented May 28, 2019

This is an resolve to #4094 issue.

and the result is here:
colorsspecials
where backstab (coup dans le dos)is inactive and poison(empoisonnement) is active.

resolve color active/inactive colors similars
This is an resolve to #4094 issue.

@jostephd jostephd added this to the 1.14.9 milestone May 29, 2019

@jostephd jostephd added the UI label Jun 10, 2019

@Wedge009

This comment has been minimized.

Copy link
Member

commented Aug 14, 2019

This change makes sense to me but looking at some of the discussion from the issues linked in #4094, there was mention of a decision to deliberately make the text visually similar. If it's okay to reverse that decision, then there shouldn't be any reason not to merge this, right?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.