dragster
dragster copied to clipboard
bensmithett
Add support for older versions of IE
@catmanjan I'm taking the progressive enhancement approach here. I mostly use these events to add/remove styling classes to indicate a user has dragged over an element. It's nice UI polish, but it's not vital to drag and drop. You can still drop on a dropzone that doesn't change its appearance on dragover.
I basically treat the events like I treat :hover in the world of touch devices. Nice & useful if you have an input device that supports them, never vital though.
Do you have a use case where these events are vital in old IE? (Will reopen if so)
What exactly is your reason for closing?
My use case is adding a hover style when dragging/dropping a file onto an element. It doesn't work in IE, therefore it is vital? :confused:
@catmanjan at this point it's an enhancement, I've never promised old IE support. Reopened & labeled as such :smiley:
Old IE doesn't support DOM event constructors, and I don't want to bloat the library with cross-browser event junk to support a use-case I haven't had a need for.
There's a pull request in for a jQuery plugin version here that I haven't had a chance to play with yet: https://github.com/bensmithett/dragster/pull/2/files
That feels like a much nicer way to handle IE support - can you try it out & see if it solves your problems as is?
jQuery does help with IE support but that PR is buggy.
I will maintain a seperate repo with a jquery.dragster project, for anyone else seeing this, it is also available on Nuget, making version control easier for your MVC folk.
https://github.com/catmanjan/jquery-dragster
https://www.nuget.org/packages/jQuery.Dragster/1.0.0
@catmanjan any chance you can list off the bugs you found in the other jquery plugin PR?
Was suffering the same problem as issue #5.
Also does not pass in global jQuery namespace correctly, causing issues with other incorrect plugins.
Following Pull Request is working with IE9, IE10, IE11 and Edge: https://github.com/christophermina/dragster
