[cla][patch]add tooltip to dojox gfx

[liyang]

Tooltip is available in dijit but not in gfx, this patch adds tooltip support to gfx, so you can:

var rect = surface.createRect({x:0,y:0,width:100,height:100}).setFill("black");
rect.showTooltip("hello gfx tooltip");

Patch of source code and test case have been submitted to IBM internal issue tracker and shall be uploaded here soon by IBM committer.

[peller]

It is my understanding that this functionality -- even tooltips with rich text -- is already available in Dojo 1.4, but I'm not sure if it's specific to charting.

[peller]

patch from liyang (ibm, ccla)

[peller]

patch from liyang (ibm, ccla)

[bill]

See #4721. There's already a dojox.charting.action2d.Tooltip which may be overlapping with this functionality. I'm not sure why #4721 is still open though...

[liyang]

A general GFX tooltip will be appreciated, as besides charting, more widgets (like Map, Diagrammer) are being developed on top of GFX, and they all need tooltip. Better to have a general GFX tooltip first and let the higher level widgets to reuse it.

[bill]

Makes sense to me. I was thinking that the charting tooltip could leverage the GFX tooltip, rather than vice-versa, since charts are made of GFX objects.

Although looking at the charting tooltip code not sure how much can be delegated to a GFX tooltip, maybe some of the aroundRect calculations.

[peller]

It should also be noted that this code (both the proposed gfx patch and the existing charting code) has a dijit dependency, presumably on dijit css and js. Is this the only dijit dependency in these packages?

[bill]

You can now use dijit tooltips against GFX objects, so no need for a separate widget in GFX. Actually I think at one point we did have a separate widget, but it's gone now.