{"action":"created","issue":{"url":"https://api.github.com/repos/tmpvar/jsdom/issues/1007","labels_url":"https://api.github.com/repos/tmpvar/jsdom/issues/1007/labels{/name}","comments_url":"https://api.github.com/repos/tmpvar/jsdom/issues/1007/comments","events_url":"https://api.github.com/repos/tmpvar/jsdom/issues/1007/events","html_url":"https://github.com/tmpvar/jsdom/pull/1007","id":55079128,"number":1007,"title":"Only uncheck radios in same form","user":{"login":"zpao","id":8445,"avatar_url":"https://avatars.githubusercontent.com/u/8445?v=3","gravatar_id":"","url":"https://api.github.com/users/zpao","html_url":"https://github.com/zpao","followers_url":"https://api.github.com/users/zpao/followers","following_url":"https://api.github.com/users/zpao/following{/other_user}","gists_url":"https://api.github.com/users/zpao/gists{/gist_id}","starred_url":"https://api.github.com/users/zpao/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zpao/subscriptions","organizations_url":"https://api.github.com/users/zpao/orgs","repos_url":"https://api.github.com/users/zpao/repos","events_url":"https://api.github.com/users/zpao/events{/privacy}","received_events_url":"https://api.github.com/users/zpao/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-01-21T21:24:17Z","updated_at":"2015-01-21T22:04:37Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/tmpvar/jsdom/pulls/1007","html_url":"https://github.com/tmpvar/jsdom/pull/1007","diff_url":"https://github.com/tmpvar/jsdom/pull/1007.diff","patch_url":"https://github.com/tmpvar/jsdom/pull/1007.patch"},"body":"When selecting another radio element, only other radios in the same form should be unchecked.\r\n\r\nSee http://jsbin.com/daqoge/1/ as an example. This is taken pretty much directly from a test we have in React where we're ensuring we maintain browser behavior. In order to make it pass in jest we actually \"fix\" this in our (ancient) jsdom env by overridding the setter for `HTMLInputElement.checked` (https://github.com/facebook/jest/blob/master/src/lib/jsdom-compat.js#L245-L284).\r\n\r\nI think this is the right fix but there's a lot of code I didn't read. I also didn't write a test - I'll look into that process but if anybody more familiar wants to jump in and help, that would be appreciated."},"comment":{"url":"https://api.github.com/repos/tmpvar/jsdom/issues/comments/70931165","html_url":"https://github.com/tmpvar/jsdom/pull/1007#issuecomment-70931165","issue_url":"https://api.github.com/repos/tmpvar/jsdom/issues/1007","id":70931165,"user":{"login":"zpao","id":8445,"avatar_url":"https://avatars.githubusercontent.com/u/8445?v=3","gravatar_id":"","url":"https://api.github.com/users/zpao","html_url":"https://github.com/zpao","followers_url":"https://api.github.com/users/zpao/followers","following_url":"https://api.github.com/users/zpao/following{/other_user}","gists_url":"https://api.github.com/users/zpao/gists{/gist_id}","starred_url":"https://api.github.com/users/zpao/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zpao/subscriptions","organizations_url":"https://api.github.com/users/zpao/orgs","repos_url":"https://api.github.com/users/zpao/repos","events_url":"https://api.github.com/users/zpao/events{/privacy}","received_events_url":"https://api.github.com/users/zpao/received_events","type":"User","site_admin":false},"created_at":"2015-01-21T22:04:37Z","updated_at":"2015-01-21T22:04:37Z","body":"Note: I did confirm that the new test fails in master without the rest of my patch."}}