PDA

View Full Version : [FIXED-524][3.1.1-rc] EditorGrid field blur bug



clesquir
3 Feb 2010, 6:34 AM
Ext version tested:
* Ext 3.1.1-rc

css used:
* only default ext-all.css

Browser versions tested against:
* FF3.5.7 (firebug 1.4.5 installed)

Operating System:
* Ubuntu 9.10

Description:
* When entering in editing mode in the textfield, then tab to go the next textfield row. If you then click outside of the grid, it is usually unrendering the textfield that was in edit mode. Now it stays in editing mode so the value is not changed for that field.

Test Case:
Use the example ext-3.1.1-rc\examples\grid\edit-grid.html modifying the columnModel in edit-grid.js file like this:



var cm = new Ext.grid.ColumnModel({
// specify any defaults for each column
defaults: {
sortable: true // columns are not sortable by default
},
columns: [
{
id: 'common',
header: 'Common Name',
dataIndex: 'common',
width: 220,
// use shorthand alias defined above
editor: new fm.TextField({
allowBlank: false
})
}, {
header: 'Light',
dataIndex: 'light',
width: 130
}, {
header: 'Price',
dataIndex: 'price',
width: 70,
align: 'right',
renderer: 'usMoney'
}, {
header: 'Available',
dataIndex: 'availDate',
width: 95,
renderer: formatDate
}
]
});

mystix
3 Feb 2010, 8:51 AM
verified in the ext-3.1.1-rc release and in svn.

basically remove all editors except for the first column's TextField editor, click in any cell in the first column to commence editing, tab once to move the editor to the next row (because all other columns have no editors), then click outside the grid -- the active editor does not complete the edit.

Jamie Avins
3 Feb 2010, 10:01 AM
Fixed in svn 5983.

clesquir
3 Feb 2010, 10:12 AM
Will it be available with Ext 3.1.1 or is it too late?

Jamie Avins
3 Feb 2010, 11:06 AM
It will go in 3.1.1.