{"action":"created","issue":{"url":"https://api.github.com/repos/zurb/foundation/issues/6313","labels_url":"https://api.github.com/repos/zurb/foundation/issues/6313/labels{/name}","comments_url":"https://api.github.com/repos/zurb/foundation/issues/6313/comments","events_url":"https://api.github.com/repos/zurb/foundation/issues/6313/events","html_url":"https://github.com/zurb/foundation/pull/6313","id":58440808,"number":6313,"title":"Remove non-namespaced JS events","user":{"login":"javiercejudo","id":1095851,"avatar_url":"https://avatars.githubusercontent.com/u/1095851?v=3","gravatar_id":"","url":"https://api.github.com/users/javiercejudo","html_url":"https://github.com/javiercejudo","followers_url":"https://api.github.com/users/javiercejudo/followers","following_url":"https://api.github.com/users/javiercejudo/following{/other_user}","gists_url":"https://api.github.com/users/javiercejudo/gists{/gist_id}","starred_url":"https://api.github.com/users/javiercejudo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/javiercejudo/subscriptions","organizations_url":"https://api.github.com/users/javiercejudo/orgs","repos_url":"https://api.github.com/users/javiercejudo/repos","events_url":"https://api.github.com/users/javiercejudo/events{/privacy}","received_events_url":"https://api.github.com/users/javiercejudo/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-02-21T04:02:57Z","updated_at":"2015-05-14T12:46:56Z","closed_at":"2015-02-24T16:48:46Z","pull_request":{"url":"https://api.github.com/repos/zurb/foundation/pulls/6313","html_url":"https://github.com/zurb/foundation/pull/6313","diff_url":"https://github.com/zurb/foundation/pull/6313.diff","patch_url":"https://github.com/zurb/foundation/pull/6313.patch"},"body":"Fully remove non-namespaced JS events. I didn't really know what to say in the docs' deprecation notices. Let me know if you'd like to change the wording.\r\n\r\nI noticed only abide has tests regarding events. I didn't want to add tests for that in this PR, but might add tests separately.\r\n\r\nAnd an open question: is this necessary? https://github.com/zurb/foundation/blob/624c2e35facc1d5b98bc3cacf8d4ef936f427678/js/foundation/foundation.reveal.js#L179 Maybe the check for namespace in fefc24c03b64438835f2859d18c248b9a723484e can be removed as well.\r\n\r\ncloses #5330"},"comment":{"url":"https://api.github.com/repos/zurb/foundation/issues/comments/102027152","html_url":"https://github.com/zurb/foundation/pull/6313#issuecomment-102027152","issue_url":"https://api.github.com/repos/zurb/foundation/issues/6313","id":102027152,"user":{"login":"javiercejudo","id":1095851,"avatar_url":"https://avatars.githubusercontent.com/u/1095851?v=3","gravatar_id":"","url":"https://api.github.com/users/javiercejudo","html_url":"https://github.com/javiercejudo","followers_url":"https://api.github.com/users/javiercejudo/followers","following_url":"https://api.github.com/users/javiercejudo/following{/other_user}","gists_url":"https://api.github.com/users/javiercejudo/gists{/gist_id}","starred_url":"https://api.github.com/users/javiercejudo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/javiercejudo/subscriptions","organizations_url":"https://api.github.com/users/javiercejudo/orgs","repos_url":"https://api.github.com/users/javiercejudo/repos","events_url":"https://api.github.com/users/javiercejudo/events{/privacy}","received_events_url":"https://api.github.com/users/javiercejudo/received_events","type":"User","site_admin":false},"created_at":"2015-05-14T12:46:56Z","updated_at":"2015-05-14T12:46:56Z","body":"At http://foundation.zurb.com/docs/components/abide.html, adding \r\n```js\r\n$('#testForm').on('invalid.fndtn.abide', function() { alert(0); });\r\n$('#testForm').on('valid.fndtn.abide', function() { alert(1); });\r\n```\r\nin the console and manipulating the first form seems to do what it should, and it is using foundation 5.5.2. It'd be great to see a JSFiddle/CodePen or similar showing the issue."}}