[patch][ccla] Avoid _Templated dependency on charting legend

[cjolif]

charting Legend is extending _Templated. However it can be very easily re-written without that dependency which will save download time in a mobile context and will fit with dojox.mobile good practice of not including _Templated for mobile deployment.

This attached patch is replacing the template by a buildRendering implementation.

[cjolif]

patch

[bill]

Please attach patch files, not full files. Also, the buildRendering() you have could be more succinctly written with something like:

buildRendering: function(){
	this.domNode = this.legendNode = dojo.create("table", 
		{role: "group", "aria-label": "chart legend", "class": "dojoxLegendNode"});
	this.legendBody = dojo.create("tbody", this.legendNode);
},

BTW, the legendNode attribute in this class is unnecessary since it's the same as this.domNode.

[Eugene Lazutkin]

Please attach a patch taking Bill's comments into consideration.

I'll treat the unnecessary dependence on _Templated as a defect.

[cjolif]

Sorry I inadvertently attached the full file instead of the patch file. I'll take Bill's comment into consideration and attached a new patch. Please note that removing the unnecessary legendNode attribute also forces to patch SelectableLegend?.js as it was using this property.

I kept the this.inherited(arguments) call removed by Bill because parent method does something with CSS baseClass and I guess this can be useful in some cases.

Let me know if further modification are needed.

[Chris Mitchell]

patch appears to be out of date with trunk. please update

[cjolif]

The new attachment should work on trunk.

[Chris Mitchell]

I'm still getting 1 reject applying this patch to trunk:

applying patch 12268-nontemplatedlegend...

(Stripping trailing CRs from patch.) patching file dojox/charting/widget/Legend.js Hunk #1 FAILED at 1. 1 out of 3 hunks FAILED -- saving rejects to file dojox/charting/widget/Legend.js.rej (Stripping trailing CRs from patch.) patching file dojox/charting/widget/SelectableLegend.js

[Chris Mitchell]

Disregard, latest patch working on trunk. user error.

[cjolif]

Chris, the latest patch indeed applies on trunk but I think I found an issue with it at runtime. When porting from old trunk to new trunk I forgot to re-apply a fix I included compare to bill's in the first patch. That is:

this.legendBody = dojo.create("tbody", null, this.legendNode);

not

this.legendBody = dojo.create("tbody", this.legendNode);

I'll re-attached a fixed patch in a second.

[Chris Mitchell]

(In [24511]) fixes #12268 Avoid _Templated dependency on charting legend. !strict

[Chris Mitchell]

reopening...had out of date patch file

[Chris Mitchell]

fixed in [24537]