-
-
Notifications
You must be signed in to change notification settings - Fork 0
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
Remove gold/silver/bronze icons from rerun cups #66
Comments
Just to clarify, the colors don't show your best division, it's just based on the div number. Either way, the most obvious solution to the confusion is probably just to remove all rerun entries from the table if the checkbox is unchecked. Probably will require some thinking about whether the API will need to be adjusted as well (to prevent the "Load more" button from doing nothing if only reruns get loaded and none of them get shown). |
Are you sure? My best rerun divs now happen to be 2, 3, 4, but before I got those I had 4 marked gold, 8 marked silver, and I think 9 marked bronze, so it seems like it is best divs overall including reruns |
Hmm maybe something changed actually, because this player's best div (asdfghjkl294) is only 5, and nothing is colored. I could've sworn my best divs before were colored as such, not just divs 1-4 |
Just checked the history, I'm pretty sure it's always been like this.
Ye, I think the other colors are just a little misleading - seeing gold or silver on division 2 or 3 is not really intuitive (especially cause we're showing gold for division rank 1). Maybe a generic highlight for the top X divisions (like 2 and 3?) could still be nice to see at a glance which cups were in the top divs - or just a normal gold/silver/bronze for the top three to make it more obvious what the colors mean. Not sure how I feel about the idea of treating the reruns differently when it comes to the coloring tbh, it's not easy to communicate the difference clearly. Of course reruns have significantly fewer players, but I think I'd prefer visual consistency over technically being more accurate by e.g. making the colors depend on the achieved div's percentage based on the cup's total number of divs. Maybe @codecat has some input too? |
Sorry, I should have clarified - I like having the full D1-4 styling for main COTDs, but I think it should get axed for lower divs. I'll never get div 1, but seeing the couple of D3s/4s stick out on the profile is kind of neat. I agree it's kind of a pain as far as consistency, though. I had a thought to add a separate color palate for reruns (e.g., a purple D1 instead of the green for those), but I'm not sure that's actually a good idea. Some kind of a by-percentage system could work out, though - like a "D1 is author, >= 10% is gold, >=25% is silver, >=50% is bronze" would be a nice option to have. |
Ye, fair points. I just don't like the idea of one div 7 having some colored highlight, and another being grey for no obvious reason. Hoping that people will bother to compare the player numbers isn't amazing UX. For the colors, maybe we could keep the same color scheme for reruns but just make it darker/less saturated? Worth playing around with I think, there might be some intuitive solutions that still stay mostly consistent. |
Perhaps we should pick some other color, unrelated to the Author/Gold/Silver/Bronze scheme, and then only show it for non-rerun div1. (Optionally a different color for rerun div1?) That way we keep consistency, and potentially make it clearer. |
Had another look at this now that reruns are hidden from the table when they're not explicitly included. |
I'd argue that getting division 2 in a rerun is not really an achievement, so having the indicator that it was my best division is not at all helpful. It should at least be toggleable with the "include reruns" checkbox, or maybe a separate checkbox for indicating best divisions.
The text was updated successfully, but these errors were encountered: