← Back to team overview

widelands-dev team mailing list archive

Re: [Merge] lp:~widelands-dev/widelands/cleanup-statistics-plotter into lp:widelands

 

> 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.
 
-- 
https://code.launchpad.net/~widelands-dev/widelands/cleanup-statistics-plotter/+merge/365138
Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/cleanup-statistics-plotter into lp:widelands.


References