Documentation Bugs - Ext 2.x

[jay@moduscreate.com]

Ext.field.msgTarget
Should be:

msgTarget : String

The location where error text should display in a form layout. Should be one of the following values (defaults to 'qtip'):

Or something similar

[mjlecomte]

[implemented]

Ext.grid.ColumnModel

The code samples at the top are excellent. I didn't realize you could do the config like that.

That said, the first method (ColumnModel constructor) seems like it needs to be updated as it only mentions the array option for the config object.

ColumnModel( Object config )
Parameters:

config : Object
An Array of column config objects or specify columns config property along with config properties for the superclass. See this class's config objects for details.
Returns:

As far as I see the only other config options available would be:

• listeners
• defaultWidth
• defaultSortable

So why not just specify those? It's not as obvious for noobs, especially when "listeners" is not listed and the other two are listed as "properties" not "configs".

[fins]

the comment of config "enableDragDrop" maybe has a mistake.
now it's :

    /**
     * @cfg {Boolean} <p>enableDragDrop True to enable dragging of the selected rows of the GridPanel.</p>
... 

the "<p>" should be after enableDragDrop , I think.
Like this :

    /**
     * @cfg {Boolean} enableDragDrop <p>True to enable dragging of the selected rows of the GridPanel.</p>
... 

[Condor]

[implemented]

TabPanel is missing docs for the itemTpl config option.

ps. The Ext 3.0 getTemplateArgs docs even link to the non-existent itemTpl docs!

[jay@moduscreate.com]

What's up with:

for Class Ext.dd.PanelProxy

 // Internal developer documentation -- will not show up in API docs A custom drag proxy implementation specific to Ext.Panels. This class is primarily used internally for the Panel's drag drop implementation, and should never need to be created directly.

[watrboy00]

Class Ext.layout.Accordion, in the example both animate and activeOnTop are true but the docs inform you that you should not do that.

var accordion = new Ext.Panel({
    title: 'Accordion Layout',
    layout:'accordion',
    defaults: {
        // applied to each contained panel
        bodyStyle: 'padding:15px'
    },
    layoutConfig: {
        // layout-specific configs go here
        titleCollapse: false,
        animate: true,
        activeOnTop: true
    },
    items: [{
        title: 'Panel 1',
        html: '<p>Panel content!</p>'
    },{
        title: 'Panel 2',
        html: '<p>Panel content!</p>'
    },{
        title: 'Panel 3',
        html: '<p>Panel content!</p>'
    }]
});

activeOnTop : Boolean
True to swap the position of each panel as it is expanded so that it becomes the first item in the container, false to keep the panels in the rendered order. This is NOT compatible with "animate:true" (defaults to false).

animate : Boolean
True to slide the contained panels open and closed during expand/collapse using animation, false to open and close directly with no animation (defaults to false). Note: to defer to the specific config setting of each contained panel for this property, set this to undefined at the layout level.

[mikegiddens]

[implemented]
It seems there is no reference to:
enableKeyEvents

I see in Class Ext.form.TextField a comment in
keydown
keyup
Keypress input field event. This event only fires if enableKeyEvents is set to true.

so this should be noted in the cfg section that this is an option that can be set.

[mjlecomte]

[implemented]

It seems there is no reference to:
enableKeyEvents

I see in Class Ext.form.TextField a comment in
keydown
keyup
Keypress input field event. This event only fires if enableKeyEvents is set to true.

so this should be noted in the cfg section that this is an option that can be set.

Thanks Mike, already mentioned here BTW: http://extjs.com/forum/showthread.ph...408#post243408

It's been implemented in svn already.

[Animal]

On a style note, there are a few occurences of "it's" creeping in when the word should be "its".

<span class="english-pedant">"It's" is shorthand for "it is". An apostophe generally indicates letters have been dropped. "Its" is the same kind of word as "His" and "Hers"

So "they're" means "they are". As opposed to "their" or "there".

And "You're" means "you are". As opposed to "your"</span>

[jay@moduscreate.com]

Ah, you English and your correct english!

Just kidding. To be honest, my wife is a huge stickler for these types of errors. Good thing she doesn't proof read for me .