Treebeard constructor is unintuitive

[sloria]

The Treebeard constructor/intializer function, runTB is a bit unintuitive because it doesn't really return a new Treebeard instance; intstead, it returns a mithril module: https://github.com/caneruguz/treebeard/blob/93ff6df2f4e230db33736eb1850de4e471daa5db/scripts/grid.js#L2406

This leads to a number of "gotchas":

var treebeard = new Treebeard({...});
console.log(treebeard.options);  // ERROR

[caneruguz]

It should probably be this instead

    ...
    m.module(document.getElementById(finalOptions.divID), tb);
    return tb.controller;
    ...

What do you think?

[sloria]

Given that Treebeard is based on mithril, I think the most intuitive interface would be to define Treebeard as a component:

var App = {
    view: function(ctrl) {
        return m('div#myGrid', {}, [
            m.component(Treebeard, {
                toolbarComponent: MyToolbar,
                resolveItemURL: function(item) {...}
            });
        ])
    }
}

m.mount(document.getElementById('app'), App);