Skip to content

next: Support use different color to indicate move blunder in variation#413

Merged
featurecat merged 2 commits intofeaturecat:nextfrom
zsalch:next_colornode
Nov 8, 2018
Merged

next: Support use different color to indicate move blunder in variation#413
featurecat merged 2 commits intofeaturecat:nextfrom
zsalch:next_colornode

Conversation

@zsalch
Copy link
Copy Markdown
Contributor

@zsalch zsalch commented Oct 31, 2018

Base on #412
Closes #389

Color node:
image

With comment node:
image

  1. Config:
    "blunder-winrate-thresholds": [-30,-20,-10,-5,5,10],
    "blunder-node-colors":[[255,0,0],[0,255,0],[0,0,255],[255,255,0],[0,255,255],[255,0,255]],
    "comment-node-color":[0,0,255,255],

  2. Ctrl+T for toggle the node color with the comment

  3. Press ‘Y' for switch node color display

    default (show color) -> only black color -> only white color

@featurecat
Copy link
Copy Markdown
Owner

LGTM

@featurecat featurecat merged commit 94c5a96 into featurecat:next Nov 8, 2018
@featurecat
Copy link
Copy Markdown
Owner

By the way, you forgot to add the three config values to the config file. So I had to add them to get it to work. I fixed it. You should try deleting the config file as a test before making a PR, to see if a new feature still works.

@featurecat
Copy link
Copy Markdown
Owner

Actually apparently the node color is part of the theme. I'm not sure how I got it to work but it no longer works as of now. It's not part of the default theme which is problematic.

@zsalch
Copy link
Copy Markdown
Contributor Author

zsalch commented Nov 8, 2018

You are right. Only these three options have no default value. I missed them.

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