-
-
Notifications
You must be signed in to change notification settings - Fork 377
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 score lead line to winrate graph #823
base: master
Are you sure you want to change the base?
Conversation
Apparently corresponds quite well to #755 |
Thank you for beginning work on this :)
Sorry, ignore the above. I think I just had a bad webpack cache or something |
Hi, It's a pleasure to contribute! |
I've tested this and found some issues:
|
It's possible this is outside the scope of this PR, but the other features related to the win rate should be replicated for the score lead, namely, the displays in the graph showing the numerical value, as well as the difference bars in the graph. For the difference bars it might make sense to have an option to choose whether the bars correspond to win rate or score lead, and even to hide them altogether. |
Actually, having all information simultaneously displayed might be too cluttered. Perhaps the graph should have information corresponding to only one of win rate or score lead at any given time, with an option to toggle between them. Then the same colors could be used for the lines and bars in either scenario. |
This feature is a high priority for me so I went ahead and fixed the issues myself in #828 . |
Early PR at this stage, would welcome some suggestions to make cleaner. and for how to do the i18n correctly.
Content of this PR:
SBKS
<-- Welcoming a better suggestion for the node property nameShow Score Lead Graph
Example image: