{"action":"created","comment":{"url":"https://api.github.com/repos/Microsoft/cordova-simulate/pulls/comments/85578666","pull_request_review_id":6288951,"id":85578666,"diff_hunk":"@@ -28,6 +31,15 @@ function showDialog(dialogId, cb) {\n \n     currentDialogId = dialogId;\n \n+    // Disable the panels if they're not already\n+    if (!panelsDisabled) {\n+        activeElement = document.activeElement;\n+        Array.prototype.forEach.call(getPanels(), function (panel) {","path":"src/modules/sim-host/dialog.js","position":17,"original_position":17,"commit_id":"6a4e9ce91b750b28bb97213f9943c53902c31126","original_commit_id":"6a4e9ce91b750b28bb97213f9943c53902c31126","user":{"login":"MSLaguana","id":14098340,"avatar_url":"https://avatars.githubusercontent.com/u/14098340?v=3","gravatar_id":"","url":"https://api.github.com/users/MSLaguana","html_url":"https://github.com/MSLaguana","followers_url":"https://api.github.com/users/MSLaguana/followers","following_url":"https://api.github.com/users/MSLaguana/following{/other_user}","gists_url":"https://api.github.com/users/MSLaguana/gists{/gist_id}","starred_url":"https://api.github.com/users/MSLaguana/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MSLaguana/subscriptions","organizations_url":"https://api.github.com/users/MSLaguana/orgs","repos_url":"https://api.github.com/users/MSLaguana/repos","events_url":"https://api.github.com/users/MSLaguana/events{/privacy}","received_events_url":"https://api.github.com/users/MSLaguana/received_events","type":"User","site_admin":false},"body":"What is the benefit of this over `getPanels().forEach(function (panel) {` ?","created_at":"2016-10-28T17:52:05Z","updated_at":"2016-10-28T17:52:05Z","html_url":"https://github.com/Microsoft/cordova-simulate/pull/182#discussion_r85578666","pull_request_url":"https://api.github.com/repos/Microsoft/cordova-simulate/pulls/182","_links":{"self":{"href":"https://api.github.com/repos/Microsoft/cordova-simulate/pulls/comments/85578666"},"html":{"href":"https://github.com/Microsoft/cordova-simulate/pull/182#discussion_r85578666"},"pull_request":{"href":"https://api.github.com/repos/Microsoft/cordova-simulate/pulls/182"}}},"pull_request":{"url":"https://api.github.com/repos/Microsoft/cordova-simulate/pulls/182","id":91422898,"html_url":"https://github.com/Microsoft/cordova-simulate/pull/182","diff_url":"https://github.com/Microsoft/cordova-simulate/pull/182.diff","patch_url":"https://github.com/Microsoft/cordova-simulate/pull/182.patch","issue_url":"https://api.github.com/repos/Microsoft/cordova-simulate/issues/182","number":182,"state":"open","locked":false,"title":"Disable panels when dialog showing.","user":{"login":"TimBarham","id":9665847,"avatar_url":"https://avatars.githubusercontent.com/u/9665847?v=3","gravatar_id":"","url":"https://api.github.com/users/TimBarham","html_url":"https://github.com/TimBarham","followers_url":"https://api.github.com/users/TimBarham/followers","following_url":"https://api.github.com/users/TimBarham/following{/other_user}","gists_url":"https://api.github.com/users/TimBarham/gists{/gist_id}","starred_url":"https://api.github.com/users/TimBarham/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/TimBarham/subscriptions","organizations_url":"https://api.github.com/users/TimBarham/orgs","repos_url":"https://api.github.com/users/TimBarham/repos","events_url":"https://api.github.com/users/TimBarham/events{/privacy}","received_events_url":"https://api.github.com/users/TimBarham/received_events","type":"User","site_admin":false},"body":"","created_at":"2016-10-28T17:48:15Z","updated_at":"2016-10-28T17:52:05Z","closed_at":null,"merged_at":null,"merge_commit_sha":"5c17eadcf588b310a9002c648b09c4f284716ba3","assignee":null,"assignees":[],"milestone":null,"commits_url":"https://api.github.com/repos/Microsoft/cordova-simulate/pulls/182/commits","review_comments_url":"https://api.github.com/repos/Microsoft/cordova-simulate/pulls/182/comments","review_comment_url":"https://api.github.com/repos/Microsoft/cordova-simulate/pulls/comments{/number}","comments_url":"https://api.github.com/repos/Microsoft/cordova-simulate/issues/182/comments","statuses_url":"https://api.github.com/repos/Microsoft/cordova-simulate/statuses/6a4e9ce91b750b28bb97213f9943c53902c31126","head":{"label":"Microsoft:tbarham/modal-dialog","ref":"tbarham/modal-dialog","sha":"6a4e9ce91b750b28bb97213f9943c53902c31126","user":{"login":"Microsoft","id":6154722,"avatar_url":"https://avatars.githubusercontent.com/u/6154722?v=3","gravatar_id":"","url":"https://api.github.com/users/Microsoft","html_url":"https://github.com/Microsoft","followers_url":"https://api.github.com/users/Microsoft/followers","following_url":"https://api.github.com/users/Microsoft/following{/other_user}","gists_url":"https://api.github.com/users/Microsoft/gists{/gist_id}","starred_url":"https://api.github.com/users/Microsoft/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Microsoft/subscriptions","organizations_url":"https://api.github.com/users/Microsoft/orgs","repos_url":"https://api.github.com/users/Microsoft/repos","events_url":"https://api.github.com/users/Microsoft/events{/privacy}","received_events_url":"https://api.github.com/users/Microsoft/received_events","type":"Organization","site_admin":false},"repo":{"id":43615334,"name":"cordova-simulate","full_name":"Microsoft/cordova-simulate","owner":{"login":"Microsoft","id":6154722,"avatar_url":"https://avatars.githubusercontent.com/u/6154722?v=3","gravatar_id":"","url":"https://api.github.com/users/Microsoft","html_url":"https://github.com/Microsoft","followers_url":"https://api.github.com/users/Microsoft/followers","following_url":"https://api.github.com/users/Microsoft/following{/other_user}","gists_url":"https://api.github.com/users/Microsoft/gists{/gist_id}","starred_url":"https://api.github.com/users/Microsoft/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Microsoft/subscriptions","organizations_url":"https://api.github.com/users/Microsoft/orgs","repos_url":"https://api.github.com/users/Microsoft/repos","events_url":"https://api.github.com/users/Microsoft/events{/privacy}","received_events_url":"https://api.github.com/users/Microsoft/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/Microsoft/cordova-simulate","description":"Simulates your Apache Cordova application in the browser.","fork":false,"url":"https://api.github.com/repos/Microsoft/cordova-simulate","forks_url":"https://api.github.com/repos/Microsoft/cordova-simulate/forks","keys_url":"https://api.github.com/repos/Microsoft/cordova-simulate/keys{/key_id}","collaborators_url":"https://api.github.com/repos/Microsoft/cordova-simulate/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/Microsoft/cordova-simulate/teams","hooks_url":"https://api.github.com/repos/Microsoft/cordova-simulate/hooks","issue_events_url":"https://api.github.com/repos/Microsoft/cordova-simulate/issues/events{/number}","events_url":"https://api.github.com/repos/Microsoft/cordova-simulate/events","assignees_url":"https://api.github.com/repos/Microsoft/cordova-simulate/assignees{/user}","branches_url":"https://api.github.com/repos/Microsoft/cordova-simulate/branches{/branch}","tags_url":"https://api.github.com/repos/Microsoft/cordova-simulate/tags","blobs_url":"https://api.github.com/repos/Microsoft/cordova-simulate/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/Microsoft/cordova-simulate/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/Microsoft/cordova-simulate/git/refs{/sha}","trees_url":"https://api.github.com/repos/Microsoft/cordova-simulate/git/trees{/sha}","statuses_url":"https://api.github.com/repos/Microsoft/cordova-simulate/statuses/{sha}","languages_url":"https://api.github.com/repos/Microsoft/cordova-simulate/languages","stargazers_url":"https://api.github.com/repos/Microsoft/cordova-simulate/stargazers","contributors_url":"https://api.github.com/repos/Microsoft/cordova-simulate/contributors","subscribers_url":"https://api.github.com/repos/Microsoft/cordova-simulate/subscribers","subscription_url":"https://api.github.com/repos/Microsoft/cordova-simulate/subscription","commits_url":"https://api.github.com/repos/Microsoft/cordova-simulate/commits{/sha}","git_commits_url":"https://api.github.com/repos/Microsoft/cordova-simulate/git/commits{/sha}","comments_url":"https://api.github.com/repos/Microsoft/cordova-simulate/comments{/number}","issue_comment_url":"https://api.github.com/repos/Microsoft/cordova-simulate/issues/comments{/number}","contents_url":"https://api.github.com/repos/Microsoft/cordova-simulate/contents/{+path}","compare_url":"https://api.github.com/repos/Microsoft/cordova-simulate/compare/{base}...{head}","merges_url":"https://api.github.com/repos/Microsoft/cordova-simulate/merges","archive_url":"https://api.github.com/repos/Microsoft/cordova-simulate/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/Microsoft/cordova-simulate/downloads","issues_url":"https://api.github.com/repos/Microsoft/cordova-simulate/issues{/number}","pulls_url":"https://api.github.com/repos/Microsoft/cordova-simulate/pulls{/number}","milestones_url":"https://api.github.com/repos/Microsoft/cordova-simulate/milestones{/number}","notifications_url":"https://api.github.com/repos/Microsoft/cordova-simulate/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/Microsoft/cordova-simulate/labels{/name}","releases_url":"https://api.github.com/repos/Microsoft/cordova-simulate/releases{/id}","deployments_url":"https://api.github.com/repos/Microsoft/cordova-simulate/deployments","created_at":"2015-10-03T21:17:05Z","updated_at":"2016-10-26T19:22:25Z","pushed_at":"2016-10-28T17:48:15Z","git_url":"git://github.com/Microsoft/cordova-simulate.git","ssh_url":"git@github.com:Microsoft/cordova-simulate.git","clone_url":"https://github.com/Microsoft/cordova-simulate.git","svn_url":"https://github.com/Microsoft/cordova-simulate","homepage":"","size":1166,"stargazers_count":82,"watchers_count":82,"language":"HTML","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":28,"mirror_url":null,"open_issues_count":20,"forks":28,"open_issues":20,"watchers":82,"default_branch":"master"}},"base":{"label":"Microsoft:master","ref":"master","sha":"6bd19e63d6f8dc6636ce9ac144c014caa0c0a611","user":{"login":"Microsoft","id":6154722,"avatar_url":"https://avatars.githubusercontent.com/u/6154722?v=3","gravatar_id":"","url":"https://api.github.com/users/Microsoft","html_url":"https://github.com/Microsoft","followers_url":"https://api.github.com/users/Microsoft/followers","following_url":"https://api.github.com/users/Microsoft/following{/other_user}","gists_url":"https://api.github.com/users/Microsoft/gists{/gist_id}","starred_url":"https://api.github.com/users/Microsoft/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Microsoft/subscriptions","organizations_url":"https://api.github.com/users/Microsoft/orgs","repos_url":"https://api.github.com/users/Microsoft/repos","events_url":"https://api.github.com/users/Microsoft/events{/privacy}","received_events_url":"https://api.github.com/users/Microsoft/received_events","type":"Organization","site_admin":false},"repo":{"id":43615334,"name":"cordova-simulate","full_name":"Microsoft/cordova-simulate","owner":{"login":"Microsoft","id":6154722,"avatar_url":"https://avatars.githubusercontent.com/u/6154722?v=3","gravatar_id":"","url":"https://api.github.com/users/Microsoft","html_url":"https://github.com/Microsoft","followers_url":"https://api.github.com/users/Microsoft/followers","following_url":"https://api.github.com/users/Microsoft/following{/other_user}","gists_url":"https://api.github.com/users/Microsoft/gists{/gist_id}","starred_url":"https://api.github.com/users/Microsoft/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Microsoft/subscriptions","organizations_url":"https://api.github.com/users/Microsoft/orgs","repos_url":"https://api.github.com/users/Microsoft/repos","events_url":"https://api.github.com/users/Microsoft/events{/privacy}","received_events_url":"https://api.github.com/users/Microsoft/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/Microsoft/cordova-simulate","description":"Simulates your Apache Cordova application in the browser.","fork":false,"url":"https://api.github.com/repos/Microsoft/cordova-simulate","forks_url":"https://api.github.com/repos/Microsoft/cordova-simulate/forks","keys_url":"https://api.github.com/repos/Microsoft/cordova-simulate/keys{/key_id}","collaborators_url":"https://api.github.com/repos/Microsoft/cordova-simulate/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/Microsoft/cordova-simulate/teams","hooks_url":"https://api.github.com/repos/Microsoft/cordova-simulate/hooks","issue_events_url":"https://api.github.com/repos/Microsoft/cordova-simulate/issues/events{/number}","events_url":"https://api.github.com/repos/Microsoft/cordova-simulate/events","assignees_url":"https://api.github.com/repos/Microsoft/cordova-simulate/assignees{/user}","branches_url":"https://api.github.com/repos/Microsoft/cordova-simulate/branches{/branch}","tags_url":"https://api.github.com/repos/Microsoft/cordova-simulate/tags","blobs_url":"https://api.github.com/repos/Microsoft/cordova-simulate/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/Microsoft/cordova-simulate/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/Microsoft/cordova-simulate/git/refs{/sha}","trees_url":"https://api.github.com/repos/Microsoft/cordova-simulate/git/trees{/sha}","statuses_url":"https://api.github.com/repos/Microsoft/cordova-simulate/statuses/{sha}","languages_url":"https://api.github.com/repos/Microsoft/cordova-simulate/languages","stargazers_url":"https://api.github.com/repos/Microsoft/cordova-simulate/stargazers","contributors_url":"https://api.github.com/repos/Microsoft/cordova-simulate/contributors","subscribers_url":"https://api.github.com/repos/Microsoft/cordova-simulate/subscribers","subscription_url":"https://api.github.com/repos/Microsoft/cordova-simulate/subscription","commits_url":"https://api.github.com/repos/Microsoft/cordova-simulate/commits{/sha}","git_commits_url":"https://api.github.com/repos/Microsoft/cordova-simulate/git/commits{/sha}","comments_url":"https://api.github.com/repos/Microsoft/cordova-simulate/comments{/number}","issue_comment_url":"https://api.github.com/repos/Microsoft/cordova-simulate/issues/comments{/number}","contents_url":"https://api.github.com/repos/Microsoft/cordova-simulate/contents/{+path}","compare_url":"https://api.github.com/repos/Microsoft/cordova-simulate/compare/{base}...{head}","merges_url":"https://api.github.com/repos/Microsoft/cordova-simulate/merges","archive_url":"https://api.github.com/repos/Microsoft/cordova-simulate/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/Microsoft/cordova-simulate/downloads","issues_url":"https://api.github.com/repos/Microsoft/cordova-simulate/issues{/number}","pulls_url":"https://api.github.com/repos/Microsoft/cordova-simulate/pulls{/number}","milestones_url":"https://api.github.com/repos/Microsoft/cordova-simulate/milestones{/number}","notifications_url":"https://api.github.com/repos/Microsoft/cordova-simulate/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/Microsoft/cordova-simulate/labels{/name}","releases_url":"https://api.github.com/repos/Microsoft/cordova-simulate/releases{/id}","deployments_url":"https://api.github.com/repos/Microsoft/cordova-simulate/deployments","created_at":"2015-10-03T21:17:05Z","updated_at":"2016-10-26T19:22:25Z","pushed_at":"2016-10-28T17:48:15Z","git_url":"git://github.com/Microsoft/cordova-simulate.git","ssh_url":"git@github.com:Microsoft/cordova-simulate.git","clone_url":"https://github.com/Microsoft/cordova-simulate.git","svn_url":"https://github.com/Microsoft/cordova-simulate","homepage":"","size":1166,"stargazers_count":82,"watchers_count":82,"language":"HTML","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":28,"mirror_url":null,"open_issues_count":20,"forks":28,"open_issues":20,"watchers":82,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/Microsoft/cordova-simulate/pulls/182"},"html":{"href":"https://github.com/Microsoft/cordova-simulate/pull/182"},"issue":{"href":"https://api.github.com/repos/Microsoft/cordova-simulate/issues/182"},"comments":{"href":"https://api.github.com/repos/Microsoft/cordova-simulate/issues/182/comments"},"review_comments":{"href":"https://api.github.com/repos/Microsoft/cordova-simulate/pulls/182/comments"},"review_comment":{"href":"https://api.github.com/repos/Microsoft/cordova-simulate/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/Microsoft/cordova-simulate/pulls/182/commits"},"statuses":{"href":"https://api.github.com/repos/Microsoft/cordova-simulate/statuses/6a4e9ce91b750b28bb97213f9943c53902c31126"}}}}
{"actor":{"display_login":"MSLaguana"}}