[CLOSED][3.??] RowEditor css weird behavior

[jackjia]

Please see the attachment for the weird row editor. I did link the roweditor.css

[jackjia]

I fixed the problem by move the this.verifyLayout(true); function call after show(). This will make sure that in positionButtons will be called.

startEditing: function(rowIndex, doFocus){
     .........
     //       this.verifyLayout(true);                    //move this line to below
            if(!this.isVisible()){
                this.setPagePosition(Ext.fly(row).getXY());
            } else{
                this.el.setXY(Ext.fly(row).getXY(), {duration:0.15});
            }
            if(!this.isVisible()){
                this.show().doLayout();
            }
            this.verifyLayout(true);                      //verifyLayout function moved to here so that the clientHeight can be calculated correctly
            if(doFocus !== false){
                this.doFocus.defer(this.focusDelay, this);
            }
}

[aconran]

Are you able to reproduce a testcase that triggers this issue?

[jackjia]

To reproduce the error, remove the editor for email field in the example provided for roweditor. Resize the email column. See attachment for detail.

[mjlecomte]

It would be helpful if you posted in accordance with this thread:
http://extjs.com/forum/showthread.php?t=71015

I'm unable to reproduce the behavior described. What version of Ext and what browser are you having issues with?

[jackjia]

There is nothing to do with accordance. Check my previous screen shot with RowEditor.html example provided in the Ext examples. In that example, all I did is remove the editor for email field. Resize the email column and make sue it is small enough. You can see from the screen shoot the update button is right on top of the row.

It would be helpful if you posted in accordance with this thread:
http://extjs.com/forum/showthread.php?t=71015

I'm unable to reproduce the behavior described. What version of Ext and what browser are you having issues with?

[mjlecomte]

I have already done as you explained previously, and again, based on that in the browser I tested with, I see no such behavior, based on the Ext version I am using.

[jackjia]

I am using RC3 and Firefox 3.0. I also tested it in IE6, it has the same behavior.

You have to resize the email column to make sure it is small enough.

I have already done as you explained previously, and again, based on that in the browser I tested with, I see no such behavior, based on the Ext version I am using.

[mjlecomte]

Can you try it again with 3.0.0 final?

[jackjia]

Can you try it again with 3.0.0 final?

You are right. It has been fixed in the final. I didn't realize the final has been released.

Great jobs! Thanks