Here, a patch to add the "chdlp" option[1] (legend's position).
It introduces the variable: $chart['#legend_position'] (possible values for Google Chart: b, t, tv, bv, r, l - one value)

Could it be integrated on a future version of Chart ?

[1] http://code.google.com/intl/fr/apis/chart/labels.html#chart_legend

CommentFileSizeAuthor
#2 chart.module-legend-position.patch1.57 KBawolfey
chart.patch960 bytessullix

Comments

boombatower’s picture

boombatower commented
Status: Active » Needs work

Would seem that the positions should be turned into constants.

awolfey’s picture

awolfey commented
Status: Needs work » Needs review
StatusFileSize
new chart.module-legend-position.patch1.57 KB

Here's a patch against HEAD with positions as constants.

boombatower’s picture

boombatower commented
Status: Needs review » Fixed 
+++ chart.module	7 Oct 2009 21:10:16 -0000
@@ -58,6 +58,16 @@
+* Legend Position
+*/

Need spaced in.

+++ chart.module	7 Oct 2009 21:10:16 -0000
@@ -289,6 +299,7 @@
+  _chart_append('chdlp',$chart['#legend_position'],        $data);

A bit off, needs a space.

Made changes and committed, thanks.

I'm on crack. Are you, too?

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.