{"action":"closed","issue":{"url":"https://api.github.com/repos/taye/interact.js/issues/367","repository_url":"https://api.github.com/repos/taye/interact.js","labels_url":"https://api.github.com/repos/taye/interact.js/issues/367/labels{/name}","comments_url":"https://api.github.com/repos/taye/interact.js/issues/367/comments","events_url":"https://api.github.com/repos/taye/interact.js/issues/367/events","html_url":"https://github.com/taye/interact.js/issues/367","id":142431322,"number":367,"title":"Firefox 45.0.1 throws \"TypeError: this.activeDrops.dropzones is null\"","user":{"login":"mzxgiant","id":225599,"avatar_url":"https://avatars.githubusercontent.com/u/225599?v=3","gravatar_id":"","url":"https://api.github.com/users/mzxgiant","html_url":"https://github.com/mzxgiant","followers_url":"https://api.github.com/users/mzxgiant/followers","following_url":"https://api.github.com/users/mzxgiant/following{/other_user}","gists_url":"https://api.github.com/users/mzxgiant/gists{/gist_id}","starred_url":"https://api.github.com/users/mzxgiant/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mzxgiant/subscriptions","organizations_url":"https://api.github.com/users/mzxgiant/orgs","repos_url":"https://api.github.com/users/mzxgiant/repos","events_url":"https://api.github.com/users/mzxgiant/events{/privacy}","received_events_url":"https://api.github.com/users/mzxgiant/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2016-03-21T18:24:04Z","updated_at":"2016-08-10T14:59:15Z","closed_at":"2016-08-10T14:59:15Z","body":"On line 2455 of interact.js, an error is thrown in my implementation of interact.js (which has a LI being dragged into a single absolutely-positioned DIV with no content). This seems to be some odd behavior of Firefox, as the debugger suggests it is ignoring loop bounds and firing the drop events multiple times for some reason. Changing 2455 to the code below (to validate we have dropzones to fire on) fixes the problem, but I did not put this in as a PR because it seems like an unnecessarily dirty fix.\r\n\r\n`if (dropEvents.deactivate && this.activeDrops.dropzones != null)\r\n{\r\nthis.fireActiveDrops(dropEvents.deactivate); \r\n}`\r\n\r\n<bountysource-plugin>\r\n---\r\nWant to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/32110726-firefox-45-0-1-throws-typeerror-this-activedrops-dropzones-is-null?utm_campaign=plugin&utm_content=tracker%2F518286&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F518286&utm_medium=issues&utm_source=github).\r\n</bountysource-plugin>"}}
{"actor":{"display_login":"taye"}}