Documentation Bugs - Ext 2.x

[jan1234]

The default for the configuration option selModel is not Ext.grid.RowSelectionModel but: Ext.grid.CellSelectionModel

[splintor]

In Element.addListener, in hte last "shorthand syntax" sample:

el.on({
    'click' : this.onClick,
    'mouseover' : this.onMouseOver,
    'mouseout' : this.onMouseOut
    scope: this
});

a comma is missing after onMouseOut.

Also, I guess both in this sample and in the previous one, apostrophe can be removed from the event names, as they are specified as object attributes and not as string parameters.

Regards,
splintor

[Jul]

Jul, none of the constructor documentation sections specifies a return value - check all the other constructors.
The formatting problems are fixed in SVN.

Thanks, Animal. I didn't notice it missing from the other constructors. Oh how I wish we could build the docs directly so we could see what's actually in SVN.

[aconran]

Ext.data.Store's baseParams should be marked as a public property indicating that it can be changed after it has been instantiated.

[sj137]

links to source code eg combo.js seem to be redirected to the homepage...

[punkrider]

In Element.addListener, in hte last "shorthand syntax" sample:

el.on({
    'click' : this.onClick,
    'mouseover' : this.onMouseOver,
    'mouseout' : this.onMouseOut
    scope: this
});

a comma is missing after onMouseOut.

Also, I guess both in this sample and in the previous one, apostrophe can be removed from the event names, as they are specified as object attributes and not as string parameters.

Regards,
splintor

Expanding on another doc bug note in this thread...

In the addListener examples under Element there are a bunch of commas missing after the fn declares. (this.onClick, this.onMouseOver, this.onMouseOut)

*** Current Version ***

el.on({
    'click' : {
        fn: this.onClick
        scope: this,
        delay: 100
    },
    'mouseover' : {
        fn: this.onMouseOver
        scope: this
    },
    'mouseout' : {
        fn: this.onMouseOut
        scope: this
    }
});

Or a shorthand syntax:
Code:

el.on({
    'click' : this.onClick,
    'mouseover' : this.onMouseOver,
    'mouseout' : this.onMouseOut
    scope: this
});

*** Fixed Version***

Currently:
el.on({
    'click' : {
        fn: this.onClick,
        scope: this,
        delay: 100
    },
    'mouseover' : {
        fn: this.onMouseOver,
        scope: this
    },
    'mouseout' : {
        fn: this.onMouseOut,
        scope: this
    }
});

Or a shorthand syntax:
Code:

el.on({
    'click' : this.onClick,
    'mouseover' : this.onMouseOver,
    'mouseout' : this.onMouseOut,
    scope: this
});

[aconran]

Provider's documentation needs to be updated from 1.x -> 2.0.

[thesilentman]

[Ext2.0][Build 1423]
tabpanel docu misses the 'closable' and 'iconCls' config options.

[paradawks]

XTemplate example for {[ ]} inline code shows access to data values directly, but you actually have to use values.field. I'm guessing it's a documentation error, though I suppose it could be the implementation that's wrong. See the "company" line in the documentation example below:

var tpl = new Ext.XTemplate(
    '<p>Name: {name}</p>',
    '<p>Company: {[company.toUpperCase() + ', ' + title]}</p>',
    '<p>Kids: ',
    '<tpl for="kids">',
       '<div class="{[xindex % 2 === 0 ? "even" : "odd"]}">,
        '{name}',
        '</div>',
    '</tpl></p>'
);
tpl.overwrite(panel.body, data);

Should be:

var tpl = new Ext.XTemplate(
    '<p>Name: {name}</p>',
    '<p>Company: {[values.company.toUpperCase() + ', ' + values.title]}</p>',
    '<p>Kids: ',
    '<tpl for="kids">',
       '<div class="{[xindex % 2 === 0 ? "even" : "odd"]}">,
        '{name}',
        '</div>',
    '</tpl></p>'
);
tpl.overwrite(panel.body, data);

[thesilentman]

[Build 1425] The example on JSONReader`s Metadata/Metachange is not showing up in docs. It's in the JSONReader.js file (Lines 35-80). Was this added after the latest public build with which the current docs were created?

Greets,
Frank