1. #11
    Ext JS Premium Member
    Join Date
    Aug 2007
    Location
    Austin, Texas
    Posts
    24
    Vote Rating
    0
    smathis is on a distinguished road

      0  

    Default


    The following code appears to have fixed my issue...

    Code:
    Ext.override(Ext.tree.TreeEditor, {
        triggerEdit : function(node, defer){
            this.completeEdit();
            if(node.attributes.editable !== false){
                this.editNode = node;
                if(this.tree.autoScroll){
                  this.tree.body.scrollChildIntoView(node.ui.getEl());
                }
                this.autoEditTimer = this.startEdit.defer(this.editDelay, this, [node.ui.textNode, node.text]);
                return false;
            }
        }
    });
    I'm not sure why using 'scrollChildIntoView' doesn't jump versus why 'scrollIntoView' does. But for what it's worth I'm not seeing the page jump when I sub them out.

    If anyone else runs into the same problem that I did, try out scrollChildIntoView.

    Thanks again, Brian, for all your help.

  2. #12
    Sencha User
    Join Date
    Apr 2012
    Location
    Austin, Texas
    Posts
    2
    Vote Rating
    0
    brian.moeskau is an unknown quantity at this point

      0  

    Default


    Hmmm. Considering that this is the code for Element.scrollChildIntoView:

    Code:
        scrollChildIntoView : function(child, hscroll){
            Ext.fly(child, '_scrollChildIntoView').scrollIntoView(this, hscroll);
        },
    ...I'm not really sure why it would make any difference. scrollChildIntoView is a private convenience function for Element that uses an internal flyweight Element, but all scroll logic is in scrollIntoView.

  3. #13
    Sencha User
    Join Date
    Apr 2012
    Location
    Austin, Texas
    Posts
    2
    Vote Rating
    0
    brian.moeskau is an unknown quantity at this point

      0  

    Default


    Fixed in SVN.

  4. #14

    Default


    Did this fix make it into 2.2 by any chance?

  5. #15
    Sencha User ThorstenSuckow's Avatar
    Join Date
    Sep 2007
    Location
    Aachen, Germany
    Posts
    597
    Vote Rating
    2
    ThorstenSuckow is on a distinguished road

      0  

    Default


    Quote Originally Posted by Arthur.Blake View Post
    Did this fix make it into 2.2 by any chance?

    Having the same issue as described above (jumpy page) with Ext2.2. Brian, is the fix in the 2.2 branch or was it already shipped with 2.2?

  6. #16
    Ext User
    Join Date
    May 2007
    Posts
    16
    Vote Rating
    0
    nico is on a distinguished road

      0  

    Default


    The last fix (using scrollChildIntoView) ist not in Ext 2.2

    For the moment I am using smathis' override.
    Thx for that!

  7. #17
    Ext JS Premium Member
    Join Date
    Aug 2007
    Location
    Austin, Texas
    Posts
    24
    Vote Rating
    0
    smathis is on a distinguished road

      0  

    Default


    I've verified that this is still a problem in Ext 3.0 as well. The fix still appears to work.

    Why does this issue have "Fixed" in the title? It doesn't seem to have ever been implemented in the source...

  8. #18
    Sencha - Community Support Team mystix's Avatar
    Join Date
    Mar 2007
    Location
    Singapore
    Posts
    6,236
    Vote Rating
    5
    mystix will become famous soon enough

      0  

    Default


    Quote Originally Posted by smathis View Post
    I've verified that this is still a problem in Ext 3.0 as well. The fix still appears to work.

    Why does this issue have "Fixed" in the title? It doesn't seem to have ever been implemented in the source...
    it would be good if you could create a new thread in the 3.x Bugs forum with a link back to this thread?
    the devs normally do not monitor threads tagged [FIXED].

    thanks.

    p.s. no idea why this thread was tagged [FIXED] if things are as you say.

  9. #19
    Touch Premium Member
    Join Date
    Oct 2009
    Location
    Sydney, Australia
    Posts
    12
    Vote Rating
    0
    felenka is on a distinguished road

      0  

    Default


    I have the same problem with jumpy tree, and fixes above didn't help, it still jumps
    The only solution that really works for me is to set autoScroll to false
    I use ExtJS 2.2
    So, I don't think this bug could be set to FIXED actually

  10. #20
    Sencha - Community Support Team jsakalos's Avatar
    Join Date
    Apr 2007
    Location
    Slovakia
    Posts
    27,523
    Vote Rating
    378
    jsakalos has a brilliant future jsakalos has a brilliant future jsakalos has a brilliant future jsakalos has a brilliant future jsakalos has a brilliant future jsakalos has a brilliant future jsakalos has a brilliant future jsakalos has a brilliant future jsakalos has a brilliant future jsakalos has a brilliant future jsakalos has a brilliant future

      0  

    Default


    What if you upgrade to the latest 2.x release? (Ext 2.3.0 at the moment.)
    Jozef Sakalos, aka Saki

    Education, extensions and services for developers at new http://extjs.eu
    News: Grid Search Plugin, ExtJS 5 Complex Data Binding using MVVM