#322 new
kristysnet

Sortable.create() breaks onclick in IE9

Reported by kristysnet | April 7th, 2011 @ 09:20 PM

When running Sortable.create() on an element in internet explorer 9, it causes all onclick events on that element (and any child elements) to be ignored.

Comments and changes to this ticket

  • Stephen

    Stephen May 9th, 2011 @ 09:46 PM

    We are also seeing this behavior in IE9. No click events work inside the sortable element.

  • azul

    azul August 2nd, 2011 @ 03:20 PM

    Which version of prototype are you using?

    We are seeing similar issues but we are using Prototype 1.6.1 - only prototype 1.7 is supposed to support ie9

  • Franky

    Franky August 13th, 2011 @ 09:14 PM

    I have the same problem with the Sortable and a onclick event at the elements. I figured out, that if more than one tab in IE9 is opened the onclick event works.

  • Naveenkumar Manthena

    Naveenkumar Manthena November 11th, 2011 @ 03:59 PM

    Hi..

    The following fix worked for me

    I added new method in dragdrop.js

    isElementOrChildOfElement: function(el, tagname){

      if(el.tagName.toUpperCase()==tagname) {
          return true;
      }
      var ancestors = $(el).ancestors();
      for(var i=0; i<ancestors.length; i++) {
          if(ancestors[i].tagName.toUpperCase()==tagname){
              return true;
          }
      }
      return false;
    

    },

    and calling isElementOrChildOfElement method in initDrag, because i have image inside , so to avoid drag for any element of type or child of
    I am using this method.

    initDrag: function(event) {

    if(typeof Draggable._dragging[this.element] != 'undefined' &&
      Draggable._dragging[this.element]) return;
    if(Event.isLeftClick(event)) {    
      // abort on form elements, fixes a Firefox issue
      var src = Event.element(event);
      if((tag_name = src.tagName.toUpperCase()) && (
        tag_name=='INPUT' ||
        tag_name=='SELECT' ||
        tag_name=='OPTION' ||
        tag_name=='BUTTON' ||
        tag_name=='TEXTAREA' || 
        this.isElementOrChildOfElement(src, 'A'))) return;
    
      var pointer = [Event.pointerX(event), Event.pointerY(event)];
      var pos     = Position.cumulativeOffset(this.element);
      this.offset = [0,1].map( function(i) { return (pointer[i] - pos[i]) });
    
      Draggables.activate(this);
      Event.stop(event);
    }
    

    },

  • jimmy

    jimmy February 28th, 2012 @ 11:53 AM

    Have recently come accross an issue which may be at the root of this one. What i've found is that the mousemove event is triggered when clicking a draggable in IE9. So maybe that has something to do with it. I've also found that having more than one tab open in the same browser will fix the issue like Franky mentioned above.

    I have submitted a ticket here: #343.

  • Uposend255

    Uposend255 April 11th, 2018 @ 01:37 PM

    The financial and also brokerage firm business is based in New York City. https://etradelogin.org We have actually supplied a quick ways to activation overview.

  • Duccol469

    Duccol469 April 12th, 2018 @ 12:10 PM

    Go into the e-mail address connected to your account to get a new Customer ID. merrilllynchlogin.org Remain gotten in touch with your personal financial consultant always.

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.

New-ticket Create new ticket

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile ยป

script.aculo.us is an open-source JavaScript framework for visual effects and interface behaviours.
<br/><b>Source available from github</b>
The Git repository resides at:
<a href="http://github.com/madrobby/scriptaculous">http://github.com/madrobby/scriptaculous</a>
<br/>Check out the current development trunk with:
<code>git clone git://github.com/madrobby/scriptaculous.git</code>
<br/>As <b>script.aculo.us 1.xx is feature-frozen</b>, this development trunk is for <b>bugfixes only</b>.
<br/>New development should happen only for
<b>script.aculo.us 2</b>.
<br/><b>Creating a bug report</b>
When creating a bug report, be sure to include as much relevant information as possible. Post a an example that shows off the problem. Preferably, <b>alter the unit tests</b> and show through either changed or added tests how the expected behavior is not occuring.

People watching this ticket

Referenced by

Pages