Module talk:Sandbox/Wnt/Plotter

Comments
Very nice.

some constructive criticism and suggestions:
 * style: the code does not adhere to style guide (esp. spaces in parens - ( a ) and not (a) ). not a huge deal - just mentioned it in passing.
 * i do not like the "Plot" (aka "main", linechart) syntax. i'd rather see the data for the line as a single parameter, and the different points as numbered pairs in parenthesis. this way, you could do something like this (note that:
 * Pie chart: same difference. i'd like to see the params either as three series (one for values, one for names and one for colors), or a single parameter of ordered trinities. e.g. (130, #ff6622, Napa Valley]])
 * Pie: please lose the assertion that values are supplied as percents. if my data is 4000, 4000, 5000 and 8000 we should be able to draw a chart (and writ the correct values in the legends).
 * Bar chart: (you get the point)
 * All: may be nice to allow making the different objects links. for plot you get it for free if you allow icon per point, like outlined above - the editor needs to set "link=" for the image. for Bars, you do it by inserting a transparent, large enough image to the DIV, set the div style with "overflow=hidden", and use the link= attribute of the image. pie might be a bit tricky, but it can be accomplished by embedding the transparent image inside an "imagemap" tag. it's not entirely trivial, but it's not that hard also.

if you decide to use the (a,b) (a,b) syntax for data pairs, note the %b of gmatch: you can do something like

peace - קיפודנחש (aka kipod) (talk) 21:53, 5 April 2013 (UTC)