Thank you for reporting this bug. We will make it our priority to review this report.
  1. #1
    Sencha User
    Join Date
    Jun 2008
    Posts
    70
    Vote Rating
    0
    uwolfer is on a distinguished road

      0  

    Default [FIXED-515][3.1.1 rc] Ext.Toolbar#enableOverflow (patch)

    [FIXED-515][3.1.1 rc] Ext.Toolbar#enableOverflow (patch)


    The documentation tells that Ext.Toolbar#enableOverflow is false by default. This is not true; it's not initialized at all. This has the result that the check "this.container.enableOverflow === false" returns false and performance expensive dom operations are executed (Firebug profiler tells about 10% in my case).

    The following override fixes this issue:

    Code:
    Ext.override(Ext.Toolbar, {
        enableOverflow: false
    });
    Tocco AG - technology meets spirit
    http://www.tocco.ch

  2. #2
    Sencha User
    Join Date
    Jun 2008
    Posts
    70
    Vote Rating
    0
    uwolfer is on a distinguished road

      0  

    Default


    It looks like you have changed something in this area for 3.1.1 RC. Though it does not fix the issue in a global way. Why don't you use my override, which would fix it?
    Tocco AG - technology meets spirit
    http://www.tocco.ch

  3. #3
    Sencha - Ext JS Dev Team evant's Avatar
    Join Date
    Apr 2007
    Location
    Sydney, Australia
    Posts
    16,935
    Vote Rating
    635
    evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute

      0  

    Default


    Jamie did exactly that, in rev 5978.
    Evan Trimboli
    Sencha Developer
    Twitter - @evantrimboli
    Don't be afraid of the source code!

  4. #4
    Sencha User Jamie Avins's Avatar
    Join Date
    Mar 2007
    Location
    Redwood City, California
    Posts
    3,661
    Vote Rating
    20
    Jamie Avins is a jewel in the rough Jamie Avins is a jewel in the rough Jamie Avins is a jewel in the rough

      0  

    Default


    Thanks Evan, I forgot to post it is now in SVN 5978.

Thread Participants: 2