PDA

View Full Version : [OPEN-726] Row editor bugs



minspect
13 Mar 2010, 11:44 PM
Win7, Firefox 3.6, extjs 3.2 beta

1. Checkboxes are not centered. See last column in
http://www.extjs.com/deploy/ext-3.2-beta/examples/grid/row-editor.html

2. Row editor does not resize if grid resizes.

Stju
14 Mar 2010, 12:36 AM
3. Each next editor field is too big, resulting in wrong position(clearly visible ifthere are more than 5-6 columns). Last editor gets truncated (see attachment);
4. If first column is non-editable, text is without padding (see attachment);

minspect
14 Mar 2010, 8:19 AM
@Stju
Check http://www.extjs.com/forum/showthread.php?t=94196 for bug #3.

Stju
14 Mar 2010, 8:25 AM
Yes, that Fix #3, thanks!

minspect
25 Mar 2010, 1:51 PM
I see none of this bugs are fixed in extjs 3.2 RC.

RowEditor bug #5 is related to Google Chrome 4 (Win 7)
Goto http://www.extjs.com/deploy/ext-3.2-rc/examples/grid/row-editor.html
and try few times double clicking on random rows. RowEditor is displayed, editor content appears selected for a moment and then selection disappears. It's annoying, and I believe that this behavior would scare the average user.

Jamie Avins
25 Mar 2010, 2:14 PM
This is why it is still currently a UX and not in the main code yet. We will be spending some time on the ux area soon and the RowEditor in particular to hopefully bring it up to a quality level high enough to include in the main code.

minspect
28 Jul 2010, 9:55 PM
It's a pity that none of this bugs is fixed in this 3.3 beta release.

tymarats
22 Dec 2010, 7:18 AM
From what I've seen, bug #2 is not RowEditor's issue, but the bug in Ext.BoxComponent. From Ext.BoxComponent.setSize() it is clear that the width and height properties of the component are set only if boxReady is false (i.e. if the box has not been rendered yet). Thus, after resize, placement calculations for BoxLayout.onLayout() are off. Possible solution would be to place:



this.width = aw;
this.height = ah;somewhere near the end of that method (maybe before the final return).