Code review comment for lp:~widelands-dev/widelands/cleanup-statistics-plotter

Revision history for this message
Toni Förster (stonerl) wrote :

> maybe there is a way to display both?

I tried this as well, but there isn't enough space. We cannot move the
y-axes label to the right-hand side, because the labels value can be a
anything from a one digit number up to a five digit number. To meet
these requirements, the space to the right-hand side would have to be
either variable or 50 maybe 70 pixel wide.

Either solution would look very awkward.

The numbers are centred atop and beneath the y-axes for single digit
numbers. All additional decimal numbers are prefixed

Since the code is shared between all plots, I think the way it is now is
the best compromise. Aesthetically pleasing, informative, easy to
understand and it fits all graphs.

Furthermore, the economy health graph is the only differential graph and
therefore, needs to be handled a little different from the others.

« Back to merge proposal