Results 1 to 5 of 5

Thread: [FIXED][3.0rc1] Element.scroll only scrolls to the left

    Thank you for reporting this bug. We will make it our priority to review this report.
  1. #1
    Ext JS Premium Member
    Join Date
    Nov 2007
    Location
    Richardson, Texas
    Posts
    21
    Vote Rating
    0
      0  

    Default [FIXED][3.0rc1] Element.scroll only scrolls to the left

    It appears that the rewrite for the Element.scroll method was not finished in 3.0rc1. As you can see below, it only includes code for scrolling to the left:
    Code:
         scroll : function(direction, distance, animate){
             if(this.isScrollable()){
                 var el = this.dom,
                     l = el.scrollLeft, t = el.scrollTop,
                     w = el.scrollWidth, h = el.scrollHeight,
                     cw = el.clientWidth, ch = el.clientHeight,
                     scrolled = false,                                  
                     l = Math.min(l + distance, w-cw),
                     r = Math.max(l - distance, 0),
                     t = Math.max(t - distance, 0),
                     b = Math.min(t + distance, h-ch),
                     hash = {
                        l : l,
                        left : l,
                        r : r,
                        right : r,
                        t : t,
                        top : t,
                        up : t,
                        b : b, 
                        bottom : b,
                        down : b         
                     };
                     
                 direction = direction.toLowerCase();
                 
                 if (v = hash[direction]) {
                     this.scrollTo("left", v, this.preanim(arguments, 2));
                     scrolled = true;
                  }
                 return scrolled;
             }

  2. #2
    Ext JS Premium Member
    Join Date
    Nov 2007
    Location
    Richardson, Texas
    Posts
    21
    Vote Rating
    0
      0  

    Default [3.0RC1][revision 3880] Element.scroll is still broken

    Element.scroll is still broken. I reported this almost two weeks ago, but it doesn't appear that any effort has been made to address it.

    http://www.extjs.com/forum/showthread.php?t=66303

    The method appears to be incomplete, as if the programmer was interrupted before finishing it, or got hungry and went off to go get a sandwich and forgot about it. Or maybe a meteor struck him in the middle of it.

  3. #3
    Sencha Premium User evant's Avatar
    Join Date
    Apr 2007
    Location
    Sydney, Australia
    Posts
    19,128
    Vote Rating
    976
      0  

    Default

    Just bump the initial thread, no need for another. Thanks.
    Twitter - @evantrimboli
    Former Sencha framework engineer, available for consulting.
    As of 2017-09-22 I am not employed by Sencha, all subsequent posts are my own and do not represent Sencha in any way.

  4. #4
    Sencha Premium User evant's Avatar
    Join Date
    Apr 2007
    Location
    Sydney, Australia
    Posts
    19,128
    Vote Rating
    976
      0  

    Default

    Fixed in SVN, thanks.
    Twitter - @evantrimboli
    Former Sencha framework engineer, available for consulting.
    As of 2017-09-22 I am not employed by Sencha, all subsequent posts are my own and do not represent Sencha in any way.

  5. #5
    Sencha User Animal's Avatar
    Join Date
    Mar 2007
    Location
    Bédoin/Nottingham
    Posts
    30,874
    Vote Rating
    82
      0  

    Default

    Quote Originally Posted by jacombs View Post
    Element.scroll is still broken. I reported this almost two weeks ago, but it doesn't appear that any effort has been made to address it.

    http://www.extjs.com/forum/showthread.php?t=66303

    The method appears to be incomplete, as if the programmer was interrupted before finishing it, or got hungry and went off to go get a sandwich and forgot about it. Or maybe a meteor struck him in the middle of it.


    I predict you manage developers!

Posting Permissions

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