{"action":"created","comment":{"url":"https://api.github.com/repos/concord-consortium/flow-server/pulls/comments/206342594","pull_request_review_id":141739361,"id":206342594,"node_id":"MDI0OlB1bGxSZXF1ZXN0UmV2aWV3Q29tbWVudDIwNjM0MjU5NA==","diff_hunk":"@@ -0,0 +1,58 @@\n+//\n+// A widget to make app fullscreen\n+//\n+var FullScreenWidget = function() {\n+\n+    var fullscreenContainer = jQuery('#fullscreen-widget');\n+    var fullscreenButton  = jQuery('<div>', {id:'fullscreen-button'} );\n+    fullscreenButton.appendTo(fullscreenContainer);\n+    var fullscreen = false;\n+    fullscreenButton.click(function(e) {\n+        if (screenfull.enabled) {\n+            screenfull.toggle();\n+        }\n+    });\n+    //\n+    // function to enter or exit fullscreen\n+    //\n+    function requestFullScreen(element) {\n+        // Supports most browsers and their versions.\n+        var requestMethod = element.requestFullScreen || element.webkitRequestFullScreen || element.mozRequestFullScreen || element.msRequestFullScreen;\n+\n+        if (requestMethod) { // Native full screen.\n+            requestMethod.call(element);\n+        } else if (typeof window.ActiveXObject !== \"undefined\") { // Older IE.\n+            var wscript = new ActiveXObject(\"WScript.Shell\");\n+            if (wscript !== null) {\n+                wscript.SendKeys(\"{F11}\");\n+            }\n+        }\n+\n+        fullscreen = !fullscreen;\n+        if(fullscreen){\n+            $('#fullscreen-button').addClass(\"exit\");\n+        }\n+        else{\n+            $('#fullscreen-button').removeClass(\"exit\");\n+        }\n+    }\n+\n+    /*!\n+    * screenfull\n+    * v3.3.2 - 2017-10-27\n+    * (c) Sindre Sorhus; MIT License\n+    */","path":"static/flow/views/fullscreen-widget.js","position":44,"original_position":44,"commit_id":"7574d852ee96ddb7bcfa8e94949cbbca313e3b55","original_commit_id":"7574d852ee96ddb7bcfa8e94949cbbca313e3b55","user":{"login":"ekosmin","id":3824039,"node_id":"MDQ6VXNlcjM4MjQwMzk=","avatar_url":"https://avatars1.githubusercontent.com/u/3824039?v=4","gravatar_id":"","url":"https://api.github.com/users/ekosmin","html_url":"https://github.com/ekosmin","followers_url":"https://api.github.com/users/ekosmin/followers","following_url":"https://api.github.com/users/ekosmin/following{/other_user}","gists_url":"https://api.github.com/users/ekosmin/gists{/gist_id}","starred_url":"https://api.github.com/users/ekosmin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ekosmin/subscriptions","organizations_url":"https://api.github.com/users/ekosmin/orgs","repos_url":"https://api.github.com/users/ekosmin/repos","events_url":"https://api.github.com/users/ekosmin/events{/privacy}","received_events_url":"https://api.github.com/users/ekosmin/received_events","type":"User","site_admin":false},"body":"This seems like a slightly sketchy way to include a library, is this standard across the project?","created_at":"2018-07-30T22:28:00Z","updated_at":"2018-07-30T22:37:46Z","html_url":"https://github.com/concord-consortium/flow-server/pull/71#discussion_r206342594","pull_request_url":"https://api.github.com/repos/concord-consortium/flow-server/pulls/71","author_association":"NONE","_links":{"self":{"href":"https://api.github.com/repos/concord-consortium/flow-server/pulls/comments/206342594"},"html":{"href":"https://github.com/concord-consortium/flow-server/pull/71#discussion_r206342594"},"pull_request":{"href":"https://api.github.com/repos/concord-consortium/flow-server/pulls/71"}}},"pull_request":{"url":"https://api.github.com/repos/concord-consortium/flow-server/pulls/71","id":204695251,"node_id":"MDExOlB1bGxSZXF1ZXN0MjA0Njk1MjUx","html_url":"https://github.com/concord-consortium/flow-server/pull/71","diff_url":"https://github.com/concord-consortium/flow-server/pull/71.diff","patch_url":"https://github.com/concord-consortium/flow-server/pull/71.patch","issue_url":"https://api.github.com/repos/concord-consortium/flow-server/issues/71","number":71,"state":"open","locked":false,"title":"159281049 add full screen button dataflow","user":{"login":"mklewandowski","id":5126913,"node_id":"MDQ6VXNlcjUxMjY5MTM=","avatar_url":"https://avatars0.githubusercontent.com/u/5126913?v=4","gravatar_id":"","url":"https://api.github.com/users/mklewandowski","html_url":"https://github.com/mklewandowski","followers_url":"https://api.github.com/users/mklewandowski/followers","following_url":"https://api.github.com/users/mklewandowski/following{/other_user}","gists_url":"https://api.github.com/users/mklewandowski/gists{/gist_id}","starred_url":"https://api.github.com/users/mklewandowski/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mklewandowski/subscriptions","organizations_url":"https://api.github.com/users/mklewandowski/orgs","repos_url":"https://api.github.com/users/mklewandowski/repos","events_url":"https://api.github.com/users/mklewandowski/events{/privacy}","received_events_url":"https://api.github.com/users/mklewandowski/received_events","type":"User","site_admin":false},"body":"Fullscreen is optionally shown via a URL parameter (hence the changes in ext.py, flow-app.js, and flow-app.html).  You'll also see some whitespace clean up in these modules which makes up a bulk of the diff.  SVG files and fullscreen implementation in fullscreen-widget was taken from existing implementation in SEPA app as per Sam's recommendation (and trying to ensure unified look of tools in CC apps).","created_at":"2018-07-30T05:06:59Z","updated_at":"2018-07-30T22:37:46Z","closed_at":null,"merged_at":null,"merge_commit_sha":"a1f27d7a20cb1d3b85c4883ccc7f5d4415b4f1ef","assignee":null,"assignees":[],"requested_reviewers":[],"requested_teams":[],"labels":[],"milestone":null,"commits_url":"https://api.github.com/repos/concord-consortium/flow-server/pulls/71/commits","review_comments_url":"https://api.github.com/repos/concord-consortium/flow-server/pulls/71/comments","review_comment_url":"https://api.github.com/repos/concord-consortium/flow-server/pulls/comments{/number}","comments_url":"https://api.github.com/repos/concord-consortium/flow-server/issues/71/comments","statuses_url":"https://api.github.com/repos/concord-consortium/flow-server/statuses/7574d852ee96ddb7bcfa8e94949cbbca313e3b55","head":{"label":"concord-consortium:159281049-add-full-screen-button-dataflow","ref":"159281049-add-full-screen-button-dataflow","sha":"7574d852ee96ddb7bcfa8e94949cbbca313e3b55","user":{"login":"concord-consortium","id":319219,"node_id":"MDEyOk9yZ2FuaXphdGlvbjMxOTIxOQ==","avatar_url":"https://avatars1.githubusercontent.com/u/319219?v=4","gravatar_id":"","url":"https://api.github.com/users/concord-consortium","html_url":"https://github.com/concord-consortium","followers_url":"https://api.github.com/users/concord-consortium/followers","following_url":"https://api.github.com/users/concord-consortium/following{/other_user}","gists_url":"https://api.github.com/users/concord-consortium/gists{/gist_id}","starred_url":"https://api.github.com/users/concord-consortium/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/concord-consortium/subscriptions","organizations_url":"https://api.github.com/users/concord-consortium/orgs","repos_url":"https://api.github.com/users/concord-consortium/repos","events_url":"https://api.github.com/users/concord-consortium/events{/privacy}","received_events_url":"https://api.github.com/users/concord-consortium/received_events","type":"Organization","site_admin":false},"repo":{"id":83187691,"node_id":"MDEwOlJlcG9zaXRvcnk4MzE4NzY5MQ==","name":"flow-server","full_name":"concord-consortium/flow-server","owner":{"login":"concord-consortium","id":319219,"node_id":"MDEyOk9yZ2FuaXphdGlvbjMxOTIxOQ==","avatar_url":"https://avatars1.githubusercontent.com/u/319219?v=4","gravatar_id":"","url":"https://api.github.com/users/concord-consortium","html_url":"https://github.com/concord-consortium","followers_url":"https://api.github.com/users/concord-consortium/followers","following_url":"https://api.github.com/users/concord-consortium/following{/other_user}","gists_url":"https://api.github.com/users/concord-consortium/gists{/gist_id}","starred_url":"https://api.github.com/users/concord-consortium/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/concord-consortium/subscriptions","organizations_url":"https://api.github.com/users/concord-consortium/orgs","repos_url":"https://api.github.com/users/concord-consortium/repos","events_url":"https://api.github.com/users/concord-consortium/events{/privacy}","received_events_url":"https://api.github.com/users/concord-consortium/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/concord-consortium/flow-server","description":"The Manylabs flow system lets people create data flow diagrams that interface with sensors and actuators.","fork":false,"url":"https://api.github.com/repos/concord-consortium/flow-server","forks_url":"https://api.github.com/repos/concord-consortium/flow-server/forks","keys_url":"https://api.github.com/repos/concord-consortium/flow-server/keys{/key_id}","collaborators_url":"https://api.github.com/repos/concord-consortium/flow-server/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/concord-consortium/flow-server/teams","hooks_url":"https://api.github.com/repos/concord-consortium/flow-server/hooks","issue_events_url":"https://api.github.com/repos/concord-consortium/flow-server/issues/events{/number}","events_url":"https://api.github.com/repos/concord-consortium/flow-server/events","assignees_url":"https://api.github.com/repos/concord-consortium/flow-server/assignees{/user}","branches_url":"https://api.github.com/repos/concord-consortium/flow-server/branches{/branch}","tags_url":"https://api.github.com/repos/concord-consortium/flow-server/tags","blobs_url":"https://api.github.com/repos/concord-consortium/flow-server/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/concord-consortium/flow-server/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/concord-consortium/flow-server/git/refs{/sha}","trees_url":"https://api.github.com/repos/concord-consortium/flow-server/git/trees{/sha}","statuses_url":"https://api.github.com/repos/concord-consortium/flow-server/statuses/{sha}","languages_url":"https://api.github.com/repos/concord-consortium/flow-server/languages","stargazers_url":"https://api.github.com/repos/concord-consortium/flow-server/stargazers","contributors_url":"https://api.github.com/repos/concord-consortium/flow-server/contributors","subscribers_url":"https://api.github.com/repos/concord-consortium/flow-server/subscribers","subscription_url":"https://api.github.com/repos/concord-consortium/flow-server/subscription","commits_url":"https://api.github.com/repos/concord-consortium/flow-server/commits{/sha}","git_commits_url":"https://api.github.com/repos/concord-consortium/flow-server/git/commits{/sha}","comments_url":"https://api.github.com/repos/concord-consortium/flow-server/comments{/number}","issue_comment_url":"https://api.github.com/repos/concord-consortium/flow-server/issues/comments{/number}","contents_url":"https://api.github.com/repos/concord-consortium/flow-server/contents/{+path}","compare_url":"https://api.github.com/repos/concord-consortium/flow-server/compare/{base}...{head}","merges_url":"https://api.github.com/repos/concord-consortium/flow-server/merges","archive_url":"https://api.github.com/repos/concord-consortium/flow-server/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/concord-consortium/flow-server/downloads","issues_url":"https://api.github.com/repos/concord-consortium/flow-server/issues{/number}","pulls_url":"https://api.github.com/repos/concord-consortium/flow-server/pulls{/number}","milestones_url":"https://api.github.com/repos/concord-consortium/flow-server/milestones{/number}","notifications_url":"https://api.github.com/repos/concord-consortium/flow-server/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/concord-consortium/flow-server/labels{/name}","releases_url":"https://api.github.com/repos/concord-consortium/flow-server/releases{/id}","deployments_url":"https://api.github.com/repos/concord-consortium/flow-server/deployments","created_at":"2017-02-26T06:18:09Z","updated_at":"2018-07-26T14:45:05Z","pushed_at":"2018-07-30T18:20:44Z","git_url":"git://github.com/concord-consortium/flow-server.git","ssh_url":"git@github.com:concord-consortium/flow-server.git","clone_url":"https://github.com/concord-consortium/flow-server.git","svn_url":"https://github.com/concord-consortium/flow-server","homepage":null,"size":1120,"stargazers_count":4,"watchers_count":4,"language":"JavaScript","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":3,"mirror_url":null,"archived":false,"open_issues_count":4,"license":{"key":"mit","name":"MIT License","spdx_id":"MIT","url":"https://api.github.com/licenses/mit","node_id":"MDc6TGljZW5zZTEz"},"forks":3,"open_issues":4,"watchers":4,"default_branch":"master"}},"base":{"label":"concord-consortium:master","ref":"master","sha":"06e9c08efdc0164582139f6040af5f0c2b7fb3ee","user":{"login":"concord-consortium","id":319219,"node_id":"MDEyOk9yZ2FuaXphdGlvbjMxOTIxOQ==","avatar_url":"https://avatars1.githubusercontent.com/u/319219?v=4","gravatar_id":"","url":"https://api.github.com/users/concord-consortium","html_url":"https://github.com/concord-consortium","followers_url":"https://api.github.com/users/concord-consortium/followers","following_url":"https://api.github.com/users/concord-consortium/following{/other_user}","gists_url":"https://api.github.com/users/concord-consortium/gists{/gist_id}","starred_url":"https://api.github.com/users/concord-consortium/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/concord-consortium/subscriptions","organizations_url":"https://api.github.com/users/concord-consortium/orgs","repos_url":"https://api.github.com/users/concord-consortium/repos","events_url":"https://api.github.com/users/concord-consortium/events{/privacy}","received_events_url":"https://api.github.com/users/concord-consortium/received_events","type":"Organization","site_admin":false},"repo":{"id":83187691,"node_id":"MDEwOlJlcG9zaXRvcnk4MzE4NzY5MQ==","name":"flow-server","full_name":"concord-consortium/flow-server","owner":{"login":"concord-consortium","id":319219,"node_id":"MDEyOk9yZ2FuaXphdGlvbjMxOTIxOQ==","avatar_url":"https://avatars1.githubusercontent.com/u/319219?v=4","gravatar_id":"","url":"https://api.github.com/users/concord-consortium","html_url":"https://github.com/concord-consortium","followers_url":"https://api.github.com/users/concord-consortium/followers","following_url":"https://api.github.com/users/concord-consortium/following{/other_user}","gists_url":"https://api.github.com/users/concord-consortium/gists{/gist_id}","starred_url":"https://api.github.com/users/concord-consortium/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/concord-consortium/subscriptions","organizations_url":"https://api.github.com/users/concord-consortium/orgs","repos_url":"https://api.github.com/users/concord-consortium/repos","events_url":"https://api.github.com/users/concord-consortium/events{/privacy}","received_events_url":"https://api.github.com/users/concord-consortium/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/concord-consortium/flow-server","description":"The Manylabs flow system lets people create data flow diagrams that interface with sensors and actuators.","fork":false,"url":"https://api.github.com/repos/concord-consortium/flow-server","forks_url":"https://api.github.com/repos/concord-consortium/flow-server/forks","keys_url":"https://api.github.com/repos/concord-consortium/flow-server/keys{/key_id}","collaborators_url":"https://api.github.com/repos/concord-consortium/flow-server/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/concord-consortium/flow-server/teams","hooks_url":"https://api.github.com/repos/concord-consortium/flow-server/hooks","issue_events_url":"https://api.github.com/repos/concord-consortium/flow-server/issues/events{/number}","events_url":"https://api.github.com/repos/concord-consortium/flow-server/events","assignees_url":"https://api.github.com/repos/concord-consortium/flow-server/assignees{/user}","branches_url":"https://api.github.com/repos/concord-consortium/flow-server/branches{/branch}","tags_url":"https://api.github.com/repos/concord-consortium/flow-server/tags","blobs_url":"https://api.github.com/repos/concord-consortium/flow-server/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/concord-consortium/flow-server/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/concord-consortium/flow-server/git/refs{/sha}","trees_url":"https://api.github.com/repos/concord-consortium/flow-server/git/trees{/sha}","statuses_url":"https://api.github.com/repos/concord-consortium/flow-server/statuses/{sha}","languages_url":"https://api.github.com/repos/concord-consortium/flow-server/languages","stargazers_url":"https://api.github.com/repos/concord-consortium/flow-server/stargazers","contributors_url":"https://api.github.com/repos/concord-consortium/flow-server/contributors","subscribers_url":"https://api.github.com/repos/concord-consortium/flow-server/subscribers","subscription_url":"https://api.github.com/repos/concord-consortium/flow-server/subscription","commits_url":"https://api.github.com/repos/concord-consortium/flow-server/commits{/sha}","git_commits_url":"https://api.github.com/repos/concord-consortium/flow-server/git/commits{/sha}","comments_url":"https://api.github.com/repos/concord-consortium/flow-server/comments{/number}","issue_comment_url":"https://api.github.com/repos/concord-consortium/flow-server/issues/comments{/number}","contents_url":"https://api.github.com/repos/concord-consortium/flow-server/contents/{+path}","compare_url":"https://api.github.com/repos/concord-consortium/flow-server/compare/{base}...{head}","merges_url":"https://api.github.com/repos/concord-consortium/flow-server/merges","archive_url":"https://api.github.com/repos/concord-consortium/flow-server/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/concord-consortium/flow-server/downloads","issues_url":"https://api.github.com/repos/concord-consortium/flow-server/issues{/number}","pulls_url":"https://api.github.com/repos/concord-consortium/flow-server/pulls{/number}","milestones_url":"https://api.github.com/repos/concord-consortium/flow-server/milestones{/number}","notifications_url":"https://api.github.com/repos/concord-consortium/flow-server/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/concord-consortium/flow-server/labels{/name}","releases_url":"https://api.github.com/repos/concord-consortium/flow-server/releases{/id}","deployments_url":"https://api.github.com/repos/concord-consortium/flow-server/deployments","created_at":"2017-02-26T06:18:09Z","updated_at":"2018-07-26T14:45:05Z","pushed_at":"2018-07-30T18:20:44Z","git_url":"git://github.com/concord-consortium/flow-server.git","ssh_url":"git@github.com:concord-consortium/flow-server.git","clone_url":"https://github.com/concord-consortium/flow-server.git","svn_url":"https://github.com/concord-consortium/flow-server","homepage":null,"size":1120,"stargazers_count":4,"watchers_count":4,"language":"JavaScript","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":3,"mirror_url":null,"archived":false,"open_issues_count":4,"license":{"key":"mit","name":"MIT License","spdx_id":"MIT","url":"https://api.github.com/licenses/mit","node_id":"MDc6TGljZW5zZTEz"},"forks":3,"open_issues":4,"watchers":4,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/concord-consortium/flow-server/pulls/71"},"html":{"href":"https://github.com/concord-consortium/flow-server/pull/71"},"issue":{"href":"https://api.github.com/repos/concord-consortium/flow-server/issues/71"},"comments":{"href":"https://api.github.com/repos/concord-consortium/flow-server/issues/71/comments"},"review_comments":{"href":"https://api.github.com/repos/concord-consortium/flow-server/pulls/71/comments"},"review_comment":{"href":"https://api.github.com/repos/concord-consortium/flow-server/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/concord-consortium/flow-server/pulls/71/commits"},"statuses":{"href":"https://api.github.com/repos/concord-consortium/flow-server/statuses/7574d852ee96ddb7bcfa8e94949cbbca313e3b55"}},"author_association":"COLLABORATOR"}}
{"actor":{"display_login":"ekosmin"}}