Changes between Initial Version and Version 1 of Ticket #9358, comment 4


Ignore:
Timestamp:
Jun 5, 2013, 7:02:34 PM (9 years ago)
Author:
lsching17
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #9358, comment 4

    initial v1  
    1 I created a patch for jquery-ui.js (1.10.3) and problem seems fixed, but do not know whether it is good.
    2 
    3 http://jsbin.com/iqeyat/18/
    4 http://jsbin.com/iqeyat/17/
    5 
    6 {{{
    7 --- new.js
    8 +++ old.js
    9 @@ -567,37 +567,29 @@
    10         _init: $.noop,
    11  
    12         destroy: function() {
    13 -               try{
    14 -                       this._destroy();
    15 -               } finally {
    16 -                       // we can probably remove the unbind calls in 2.0
    17 -                       // all event bindings should go through this._on()
    18 -                       this.element
    19 -                               .unbind( this.eventNamespace )
    20 -                               // 1.9 BC for #7810
    21 -                               // TODO remove dual storage
    22 -                               .removeData( this.widgetName )
    23 -                               .removeData( this.widgetFullName )
    24 -                               // support: jquery <1.6.3
    25 -                               // http://bugs.jquery.com/ticket/9413
    26 -                               .removeData( $.camelCase( this.widgetFullName ) );
    27 -                       this.widget()
    28 -                               .unbind( this.eventNamespace )
    29 -                               .removeAttr( "aria-disabled" )
    30 -                               .removeClass(
    31 -                                       this.widgetFullName + "-disabled " +
    32 -                                       "ui-state-disabled" );
    33 -
    34 -                       // clean up events and states
    35 -                       this.bindings.unbind( this.eventNamespace );
    36 -                       this.hoverable.removeClass( "ui-state-hover" );
    37 -                       this.focusable.removeClass( "ui-state-focus" );
    38 -
    39 -                       //clean up properties
    40 -                       this.element=null;
    41 -                       this.options=null;
    42 -
    43 -               }
    44 +               this._destroy();
    45 +               // we can probably remove the unbind calls in 2.0
    46 +               // all event bindings should go through this._on()
    47 +               this.element
    48 +                       .unbind( this.eventNamespace )
    49 +                       // 1.9 BC for #7810
    50 +                       // TODO remove dual storage
    51 +                       .removeData( this.widgetName )
    52 +                       .removeData( this.widgetFullName )
    53 +                       // support: jquery <1.6.3
    54 +                       // http://bugs.jquery.com/ticket/9413
    55 +                       .removeData( $.camelCase( this.widgetFullName ) );
    56 +               this.widget()
    57 +                       .unbind( this.eventNamespace )
    58 +                       .removeAttr( "aria-disabled" )
    59 +                       .removeClass(
    60 +                               this.widgetFullName + "-disabled " +
    61 +                               "ui-state-disabled" );
    62 +
    63 +               // clean up events and states
    64 +               this.bindings.unbind( this.eventNamespace );
    65 +               this.hoverable.removeClass( "ui-state-hover" );
    66 +               this.focusable.removeClass( "ui-state-focus" );
    67         },
    68         _destroy: $.noop,
    69  
    70 @@ -2495,10 +2487,6 @@
    71  
    72                 //Initialize the mouse interaction
    73                 this._mouseInit();
    74 -
    75 -               //clean up to avoid closure leakage
    76 -               that = null,
    77 -               o = null;
    78  
    79         },
    80  
    81 
    82 }}}
     1please delete