[dojo-contributors] dmandrioli@gmail.com

I have not looked at it in depth yet (though I read the Google Doc and took
a quick look at a few of the JS files at GitHub), but a couple of things
jump out at me...

1. I'm not sure why you are beginning with dijit/_WidgetBase as an
underpinning; it means that Dijit is *required* for use of these gauges.
Given the move to baseless, I wonder if that is a good idea; perhaps
instead, you might consider the approach Charting took, which begins with
essentially standalone usages, and provides a way of creating/using charts
via Dijit.

2. Speaking of baseless...I notice that you are still "enforcing" the use
of "dojox" as a method of organization (particularly in the way that you
have been using the declare statements). I'd *highly* suggest taking a
look at dgrid (https://github.com/SitePen/dgrid/ ) does this; dgrid is
written in such a way as to *not* force any particular namespace container,
and should probably be used going forward as the definitive
example/template of how to structure a Dojo-based package.

That's all for now; if I have a chance this week, I will grab a copy and
play with it a bit for other criticisms.

Cheers--
Tom

2011/11/9 Emmanuel Tissandier <etissandier at gmail.com>