_Widget not converted

[mmaxwell]

_Widget is not converted to dijit/_WidgetBase.

[brianarn]

This sort of conversion seems like it may potentially be unsafe, given that _Widget brings in some of the automatic onClick etc camelcased DOM events. Does seem like it might be a good idea to issue some sort of notice/warning when using _Widget (or _Templated) though.

[mmaxwell]

In the source of _Widget, it states "Old base class for widgets. New widgets should extend dijit/_WidgetBase instead". If the goal of this tool is to help in converting from older versions of dojo to 2.0, it seems like this should be what we want to do.

[kfranqueiro]

In this case it's not about moving forward, it's about not breaking existing code. As Brian stated, _Widget contains stuff that _WidgetBase does not, so upconverting from _Widget to _WidgetBase might cause regressions in people's code.

[brianarn]

I do like the idea of having some sort of flag or way to issue a warning, though, as a handy tip.

[wkeese]

FYI, conversion of _Widget to _WidgetBase is pretty complicated:

• Besides _WidgetBase you may need to pull in other mixins, ex: _CssStateMixin. Hard to analyze which of those mixins a _Widget subclass is depending on.
• Need to upgrade usage of old API's like this.connect(...) to this.own(on(...)).

Ideally you could do the conversion but it's a lot of work.

[mmaxwell]

I believe we are holding off on this piece until _Widget goes away.