Page 2 of 3 FirstFirst 123 LastLast
Results 11 to 20 of 27

Thread: [2.1][FIXED] TreePanel Text Box Display Postion Bug

  1. #11
    Ext JS Premium Member
    Join Date
    Aug 2007
    Location
    Austin, Texas
    Posts
    24
    Vote Rating
    0
      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
    4
    Vote Rating
    1
      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
    4
    Vote Rating
    1
      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
    604
    Vote Rating
    3
      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
      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
      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 User mystix's Avatar
    Join Date
    Mar 2007
    Location
    Singapore
    Posts
    6,236
    Vote Rating
    5
      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
      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,664
    Vote Rating
    403
      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 MultiSearch Plugin, Grid MultiSort Plugin, Configuring ViewModel Hierarchy


Page 2 of 3 FirstFirst 123 LastLast

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts
  •