[patch] AMD refactoring of dojox.layout.TableContainer

[Kitson Kelly]

Attached is a refactoring of dojox.layout.TableContainer. In addition the test_TableContainer.html contained an odd situation where it expressly included the JS file in addition to the dojo.require, which causes some issues when converting to AMD declaration.

[Kenneth G. Franqueiro]

I'm attaching a new patch for the js file; since this is not a truly baseless conversion, it would be more appropriate to be requiring "dojo" and "dijit" as dependencies. (Also fixed indentation of requires.)

[Kitson Kelly]

Ok, I think I did a full baseless conversion on it now. It seems to be working. The only thing I am not 100% on is how the extension of dijit._Widget is supposed to work in the baseless world.

[Kitson Kelly]

Ignore patch 3... It was the same as the original. This one...

[Kenneth G. Franqueiro]

I'm attaching another new patch to make delegate and _Widget references also baseless. (The process for _Widget is the same as you already did for _LayoutWidget.)

[Kenneth G. Franqueiro]

New patch on top of .4 with baseless declare and _Widget

[Mathevet julien]

Seem to be partially done in #13897 : (In [26554] )

You could merge header to use this header patch and then close it.

[Kenneth G. Franqueiro]

In [27238]:

TableContainer?: dijit._Widget -> baseless _WidgetBase, use dom-style and remove _base/html dep, array -> arrayUtil, LayoutWidget? -> _LayoutWidget. Refs #13897, fixes #13831

[Kenneth G. Franqueiro]

In [27239]:

TableContainer?: backport [27238] to 1.7 branch. Refs #13897, refs #13831

[Kenneth G. Franqueiro]

As moogle pointed out, this was actually "half-fixed" in a commit before 1.7.0 went out, thus I've backported the completion of the conversion for 1.7.1 as well.