lila icon indicating copy to clipboard operation
lila copied to clipboard

Enhancement: Icons in the profile charts should match time/variant icons

Open azariak opened this issue 1 year ago • 6 comments

The icons on the profile charts should not be arbitrary, instead they should be the same icons used throughout the site including on the left panel of the profile page. This would make the charts more readable and intuitive.

image

azariak avatar Jan 09 '24 04:01 azariak

If you are then it won't match the color of the chart

M-DinhHoangViet avatar Jan 09 '24 05:01 M-DinhHoangViet

If you are then it won't match the color of the chart

Agreed - but flair icons for t/c and variants could be used? Given that we can edit the colour of each line of the charts, am not familiar with chart.js stuff

TheForkPower avatar Jan 09 '24 07:01 TheForkPower

We should maintain the colours on the chart, and just colour the time/variant icons from their current white, to the colour on the chart for each time/variant.

azariak avatar Jan 09 '24 15:01 azariak

controversial!

TheForkPower avatar Jan 10 '24 18:01 TheForkPower

Hello, I took a shot at implementing your suggestion. While I do think that the tooltip looks better, I think that the point visibility majorly suffers. Keeping the markers while only changing the tooltip is bad (see https://github.com/lichess-org/lila/issues/13655). Therefore, I will not push forward with it.

With Icons Without Icons
with without

allanjoseph98 avatar Jan 11 '24 05:01 allanjoseph98

That's a good point, although I there might be ways of improving visibility in both of your images above.

  1. I think the icons should be in front of the lines. For example with the Correspondence yellow icon it is behind the blue line which makes it harder to see.
  2. If a solid or semi-transparent white\grey coloured square was behind the icons in the chart visibility for the chart with icons would dramatically improve
  3. I am not totally sure about this one but perhaps the icons could be staggered a bit. For example the classical and crazyhouse icons overlap. Would it be possible to, say, move the rapid icon a bit earlier in the line in the case of overlaps? Would this improve visibility or cause visual discomfort?

Not sure if this is trivial or very hard/impossible with chart.js though so not sure how realistic an implementation would be

azariak avatar Jan 12 '24 15:01 azariak