Fixed for IE7 and IE8

[jasonruesch]

I made some changes if the browser is IE8 or below to handle correctly attaching the mouse events and identifying the correct mouse button clicked.

This is regarding issue #12.

[kiezasrac]

Thank you very much, you saved me a bunch of time! Works like a charm on IE8.

[jasonruesch]

I'm glad it was helpful for someone else other than myself. :)

On Jun 10, 2013, at 3:50 PM, kiezasrac [email protected] wrote:

Thank you very much, you saved me a bunch of time! Works like a charm on IE8.

— Reply to this email directly or view it on GitHub.

[estmatic]

Suggestion: I would drop the user agent browser detection... jQuery removed it for a reason. Attaching the events to $(document) should work on all browsers and jQuery normalizes event.which so that it's consistent across browsers (event.which === 1 // left-mouse click).

[andrewdisney]

I am not able to get this to work in IE 8 on Windows 7. I really like this plugin! But I cannot get the demo page to work in IE 8.

[iladarsda]

Thanks for sharing!

[pealife]

I am also not able to get this to work in IE 8 on Windows 7. Please can anyone help? I am having this bug:

Error message: Object doesn't support this property or method Sortable.js, line 96 character 9 --CODE--
// Define events _customEvents.forEach(function (name) { options[name] = _bind(this, options[name] || noop); _on(el, name.substr(2).toLowerCase(), options[name]); });

Thanks