Changes between Initial Version and Version 1 of Ticket #9498


Ignore:
Timestamp:
Aug 15, 2013, 11:04:16 AM (6 years ago)
Author:
Scott González
Comment:

This is not a bug, but a feature request. Can you please explain the real use case for this?

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #9498

    • Property Owner set to barhorn
    • Property Status changed from new to pending
    • Property Type changed from bug to feature
  • Ticket #9498 – Description

    initial v1  
    77in jquery.ui.resizable.js around line 253 (jQueryUI v1.10.3), we should loop over the handles object instead of just grabbing the first one.  ie, change:
    88
    9 {{{
    10   _mouseCapture: function(event) {
    11     var i, handle,
    12       capture = false;
    13 
    14     for (i in this.handles) {
    15       handle = $(this.handles[i])[0];
    16       if (handle === event.target || $.contains(handle, event.target)) {
    17         capture = true;
    18       }
    19     }
    20 
    21     return !this.options.disabled && capture;
    22   },
    23 }}}
    24 
    25 to
    26 
    27 
    28 {{{
    29   _mouseCapture: function(event) {
    30     var i, handle,
    31       capture = false;
    32 
    33     for (i in this.handles) {
    34       $(this.handles[i]).each(function(idxHandle,handle) {
    35         if (handle === event.target || $.contains(handle, event.target)) {
    36           capture = true;
    37           return false; //get out of the $().each loop
    38         }
    39       });
    40     }
    41 
    42     return !this.options.disabled && capture;
    43   },
    44 }}}
     9::removed::