Chart: resize broken with one arguments (width)

[Mathevet julien]

chart resize method doesn't work if you specified only width. Here CLA patch:

Index: Chart.js
===================================================================
--- Chart.js    (révision 30449)
+++ Chart.js    (copie de travail)
@@ -635,8 +635,7 @@
                                // case 0, do not resize the div, just the surface
                                case 1:
                                        // argument, override node box
-                                       box = lang.mixin({}, width);
-                                       domGeom.setMarginBox(this.node, box);
+                                       domGeom.setMarginBox(this.node, {w: width});
                                        break;
                                case 2:
                                        box = {w: width, h: height};

[cjolif]

moogle, I think (even if I agree this is not very clear) the intend was that when only a single argument is used the resize take that argument as a box definition not a single width.

See the method description:

"If no width/height or box is provided, resize the surface to the marginBox of the chart."

(or box means you can pass the box as parameter)

That also what Dijit is advertising:

​http://livedocs.dojotoolkit.org/dijit/layout#resizing

Based on that I would tend to close that as invalid but make the doc clearer. Except if you have some references to some doc / examples that shows that my interpretation is incorrect?

[Mathevet julien]

ok you're right. Set ticket to invalid. Just need to update width param doc. Sorry about to loose your time. I was disturbed by width param.

[cjolif]

In [30516]:

refs #16654. Improve the resize method doc and clean a little bit the code.

[cjolif]

In [30626]:

refs #16654. Adjust widget/Chart.js to latest changes in Chart.js

[cjolif]

In [30652]:

refs #16654. Make the correct widget/Chart.js fix this time.