Thank you for reporting this bug. We will make it our priority to review this report.
  1. #1
    Ext User
    Join Date
    Feb 2009
    Posts
    26
    Vote Rating
    0
    jackjia is on a distinguished road

      0  

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

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


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

  2. #2
    Ext User
    Join Date
    Feb 2009
    Posts
    26
    Vote Rating
    0
    jackjia is on a distinguished road

      0  

    Default bug fix in RowEditor.js

    bug fix in RowEditor.js


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

    Code:
    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);
                }
    }
    Attached Images

  3. #3
    Sencha - Architect Dev Team aconran's Avatar
    Join Date
    Mar 2007
    Posts
    8,882
    Vote Rating
    101
    aconran is a splendid one to behold aconran is a splendid one to behold aconran is a splendid one to behold aconran is a splendid one to behold aconran is a splendid one to behold aconran is a splendid one to behold

      0  

    Default


    Are you able to reproduce a testcase that triggers this issue?
    Aaron Conran
    @aconran
    Sencha Architect Development Team

  4. #4
    Ext User
    Join Date
    Feb 2009
    Posts
    26
    Vote Rating
    0
    jackjia is on a distinguished road

      0  

    Default Reproduce the error

    Reproduce the error


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

  5. #5
    Ext User
    Join Date
    Jul 2007
    Location
    Florida
    Posts
    9,996
    Vote Rating
    5
    mjlecomte will become famous soon enough mjlecomte will become famous soon enough

      0  

    Default


    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?

  6. #6
    Ext User
    Join Date
    Feb 2009
    Posts
    26
    Vote Rating
    0
    jackjia is on a distinguished road

      0  

    Default


    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.

    Quote Originally Posted by mjlecomte View Post
    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?

  7. #7
    Ext User
    Join Date
    Jul 2007
    Location
    Florida
    Posts
    9,996
    Vote Rating
    5
    mjlecomte will become famous soon enough mjlecomte will become famous soon enough

      0  

    Default


    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.

  8. #8
    Ext User
    Join Date
    Feb 2009
    Posts
    26
    Vote Rating
    0
    jackjia is on a distinguished road

      0  

    Default


    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.

    Quote Originally Posted by mjlecomte View Post
    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.

  9. #9
    Ext User
    Join Date
    Jul 2007
    Location
    Florida
    Posts
    9,996
    Vote Rating
    5
    mjlecomte will become famous soon enough mjlecomte will become famous soon enough

      0  

    Default


    Can you try it again with 3.0.0 final?

  10. #10
    Ext User
    Join Date
    Feb 2009
    Posts
    26
    Vote Rating
    0
    jackjia is on a distinguished road

      0  

    Default Fixed. Thanks

    Fixed. Thanks


    Quote Originally Posted by mjlecomte View Post
    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

Thread Participants: 2

film izle

hd film izle

film sitesi

takipci kazanma sitesi

takipci kazanma sitesi

güzel olan herşey

takipci alma sitesi

komik eğlenceli videolar