{"action":"created","comment":{"url":"https://api.github.com/repos/codice/alliance/pulls/comments/81185774","pull_request_review_id":2184650,"id":81185774,"diff_hunk":"@@ -9,17 +9,77 @@\n  * <http://www.gnu.org/licenses/lgpl.html>.\n  *\n  **/\n-.chipping-pane {\n-    color:white;\n-    background-image:linear-gradient(to bottom,#252525,#252525);\n+\n+html,\n+body {\n+  height: 100%;\n+  width: 100%;\n+  color: #ffffff;\n+  background: #28282c;\n+}\n+body {\n+  position: relative;\n+  -webkit-font-smoothing: antialiased;\n+}\n+a,\n+button {\n+  border: none;\n+  outline: none;\n+  background: none;\n+  padding: 0px;\n+  border-radius: 1px;\n+}\n+.btn {\n+    width: 49%;\n+}\n+.btn:hover {\n+    color: white;\n+}\n+\n+/* Download Button */\n+a.is-primary {\n+  color: white;\n+  background: #3c6dd5;\n+  font-size: 2rem;\n+  padding: 0px;\n+  line-height: 44px;\n+  height: 44px;\n+}\n+a.is-primary:hover {\n+  background: #3662c0;\n+  box-shadow: 0px 0px 2px #2d52a0;\n+}\n+a.is-primary:focus {\n+  background: #2d52a0;\n+  border: 2px solid #1e376b;\n+}\n+a.is-primary.is-disabled,\n+a.is-primary:disabled {\n+  background: #9eb6ea;\n+  cursor: not-allowed;\n }\n \n-.chip-image {\n-    background-color:#2a9fd6;\n-    color:white;\n+/* Reset Button */\n+button.is-neutral {\n+  color: #ffffff;\n+  background: rgba(255, 255, 255, 0.05);\n+  font-size: 2rem;\n+  padding: 0px;\n+  line-height: 44px;\n+  height: 44px;\n+  background: rgba(0, 0, 0, 0);","path":"catalog/imaging/imaging-actionprovider-chip/src/main/webapp/css/style.css","position":67,"original_position":68,"commit_id":"8f5b5d6fb5c605e0a3bd6178fefb804c61d9d8d9","original_commit_id":"b14f4f3910b6f8bcf86af66fe39222b1d152ba2b","user":{"login":"lcrosenbu","id":16792154,"avatar_url":"https://avatars.githubusercontent.com/u/16792154?v=3","gravatar_id":"","url":"https://api.github.com/users/lcrosenbu","html_url":"https://github.com/lcrosenbu","followers_url":"https://api.github.com/users/lcrosenbu/followers","following_url":"https://api.github.com/users/lcrosenbu/following{/other_user}","gists_url":"https://api.github.com/users/lcrosenbu/gists{/gist_id}","starred_url":"https://api.github.com/users/lcrosenbu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lcrosenbu/subscriptions","organizations_url":"https://api.github.com/users/lcrosenbu/orgs","repos_url":"https://api.github.com/users/lcrosenbu/repos","events_url":"https://api.github.com/users/lcrosenbu/events{/privacy}","received_events_url":"https://api.github.com/users/lcrosenbu/received_events","type":"User","site_admin":false},"body":"Removed white background rule `background: rgba(255, 255, 255, 0.05);` to leave the black background.","created_at":"2016-09-29T17:07:27Z","updated_at":"2016-09-29T17:07:27Z","html_url":"https://github.com/codice/alliance/pull/152#discussion_r81185774","pull_request_url":"https://api.github.com/repos/codice/alliance/pulls/152","_links":{"self":{"href":"https://api.github.com/repos/codice/alliance/pulls/comments/81185774"},"html":{"href":"https://github.com/codice/alliance/pull/152#discussion_r81185774"},"pull_request":{"href":"https://api.github.com/repos/codice/alliance/pulls/152"}}},"pull_request":{"url":"https://api.github.com/repos/codice/alliance/pulls/152","id":86228356,"html_url":"https://github.com/codice/alliance/pull/152","diff_url":"https://github.com/codice/alliance/pull/152.diff","patch_url":"https://github.com/codice/alliance/pull/152.patch","issue_url":"https://api.github.com/repos/codice/alliance/issues/152","number":152,"state":"open","locked":false,"title":"CAL-143 Incorporate Catalog UI CSS Styling into Image Chipping Application","user":{"login":"lcrosenbu","id":16792154,"avatar_url":"https://avatars.githubusercontent.com/u/16792154?v=3","gravatar_id":"","url":"https://api.github.com/users/lcrosenbu","html_url":"https://github.com/lcrosenbu","followers_url":"https://api.github.com/users/lcrosenbu/followers","following_url":"https://api.github.com/users/lcrosenbu/following{/other_user}","gists_url":"https://api.github.com/users/lcrosenbu/gists{/gist_id}","starred_url":"https://api.github.com/users/lcrosenbu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lcrosenbu/subscriptions","organizations_url":"https://api.github.com/users/lcrosenbu/orgs","repos_url":"https://api.github.com/users/lcrosenbu/repos","events_url":"https://api.github.com/users/lcrosenbu/events{/privacy}","received_events_url":"https://api.github.com/users/lcrosenbu/received_events","type":"User","site_admin":false},"body":"#### What does this PR do?\r\n - Applies minimal CSS styling to the Alliance image chipping app, for consistency with the Catalog UI. \r\n - Added header to provide some instruction to the user on how to use the app. \r\n - Disabled the `Download` button until a bounding box selection is made, to prevent a server error.\r\n - Changed the names of the buttons to be more descriptive.\r\n - Added custom cursor when user is drawing the bounding box.  \r\n\r\n#### Who is reviewing it (please choose AT LEAST two reviewers that need to approve the PR before it can get merged; if a component team is listed, at least one of its members needs to approve)?\r\n@andrewkfiedler @bdeining @codice/ui \r\n\r\n#### Choose 2 committers to review/merge the PR (please choose ONLY two committers from below, delete the rest).\r\n@jlcsmith\r\n@pklinef\r\n\r\n#### How should this be tested?\r\n1. Build and run Alliance.\r\n2. Ingest an image NITF file.\r\n3. Choose `Chip Image` action provider on ingested NITF.\r\n4. Verify the `Download` and `Reset` buttons are consistent with Catalog UI font/color styling.\r\n5. Regression test that image chip can be selected and downloaded from both the action provider popup and the new window.\r\n\r\n#### Any background context you want to provide?\r\nUntil the Catalog UI is merged into DDF, the styles from the Catalog UI CSS have been manually applied to the image chipping application CSS. Additional UX enhancements have been recommended and are beyond the scope of this ticket.\r\n\r\n#### What are the relevant tickets?\r\nhttps://codice.atlassian.net/browse/CAL-143\r\n\r\n#### Screenshots (if appropriate)\r\n![screen shot 2016-09-26 at 9 50 53 am](https://cloud.githubusercontent.com/assets/16792154/18841385/c9015884-83ce-11e6-85b2-f23a81c6dd1a.png)\r\n\r\n#### Checklist:\r\n- [ ] Documentation Updated\r\n- [ ] Update / Add Unit Tests\r\n- [ ] Update / Add Integration Tests\r\n","created_at":"2016-09-21T19:43:23Z","updated_at":"2016-09-29T17:07:27Z","closed_at":null,"merged_at":null,"merge_commit_sha":"a410ddf76b018ed79adce14e4bf703a7b09d3c27","assignee":null,"assignees":[],"milestone":null,"commits_url":"https://api.github.com/repos/codice/alliance/pulls/152/commits","review_comments_url":"https://api.github.com/repos/codice/alliance/pulls/152/comments","review_comment_url":"https://api.github.com/repos/codice/alliance/pulls/comments{/number}","comments_url":"https://api.github.com/repos/codice/alliance/issues/152/comments","statuses_url":"https://api.github.com/repos/codice/alliance/statuses/8f5b5d6fb5c605e0a3bd6178fefb804c61d9d8d9","head":{"label":"lcrosenbu:CAL-143","ref":"CAL-143","sha":"8f5b5d6fb5c605e0a3bd6178fefb804c61d9d8d9","user":{"login":"lcrosenbu","id":16792154,"avatar_url":"https://avatars.githubusercontent.com/u/16792154?v=3","gravatar_id":"","url":"https://api.github.com/users/lcrosenbu","html_url":"https://github.com/lcrosenbu","followers_url":"https://api.github.com/users/lcrosenbu/followers","following_url":"https://api.github.com/users/lcrosenbu/following{/other_user}","gists_url":"https://api.github.com/users/lcrosenbu/gists{/gist_id}","starred_url":"https://api.github.com/users/lcrosenbu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lcrosenbu/subscriptions","organizations_url":"https://api.github.com/users/lcrosenbu/orgs","repos_url":"https://api.github.com/users/lcrosenbu/repos","events_url":"https://api.github.com/users/lcrosenbu/events{/privacy}","received_events_url":"https://api.github.com/users/lcrosenbu/received_events","type":"User","site_admin":false},"repo":{"id":60187605,"name":"alliance","full_name":"lcrosenbu/alliance","owner":{"login":"lcrosenbu","id":16792154,"avatar_url":"https://avatars.githubusercontent.com/u/16792154?v=3","gravatar_id":"","url":"https://api.github.com/users/lcrosenbu","html_url":"https://github.com/lcrosenbu","followers_url":"https://api.github.com/users/lcrosenbu/followers","following_url":"https://api.github.com/users/lcrosenbu/following{/other_user}","gists_url":"https://api.github.com/users/lcrosenbu/gists{/gist_id}","starred_url":"https://api.github.com/users/lcrosenbu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lcrosenbu/subscriptions","organizations_url":"https://api.github.com/users/lcrosenbu/orgs","repos_url":"https://api.github.com/users/lcrosenbu/repos","events_url":"https://api.github.com/users/lcrosenbu/events{/privacy}","received_events_url":"https://api.github.com/users/lcrosenbu/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/lcrosenbu/alliance","description":"","fork":true,"url":"https://api.github.com/repos/lcrosenbu/alliance","forks_url":"https://api.github.com/repos/lcrosenbu/alliance/forks","keys_url":"https://api.github.com/repos/lcrosenbu/alliance/keys{/key_id}","collaborators_url":"https://api.github.com/repos/lcrosenbu/alliance/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/lcrosenbu/alliance/teams","hooks_url":"https://api.github.com/repos/lcrosenbu/alliance/hooks","issue_events_url":"https://api.github.com/repos/lcrosenbu/alliance/issues/events{/number}","events_url":"https://api.github.com/repos/lcrosenbu/alliance/events","assignees_url":"https://api.github.com/repos/lcrosenbu/alliance/assignees{/user}","branches_url":"https://api.github.com/repos/lcrosenbu/alliance/branches{/branch}","tags_url":"https://api.github.com/repos/lcrosenbu/alliance/tags","blobs_url":"https://api.github.com/repos/lcrosenbu/alliance/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/lcrosenbu/alliance/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/lcrosenbu/alliance/git/refs{/sha}","trees_url":"https://api.github.com/repos/lcrosenbu/alliance/git/trees{/sha}","statuses_url":"https://api.github.com/repos/lcrosenbu/alliance/statuses/{sha}","languages_url":"https://api.github.com/repos/lcrosenbu/alliance/languages","stargazers_url":"https://api.github.com/repos/lcrosenbu/alliance/stargazers","contributors_url":"https://api.github.com/repos/lcrosenbu/alliance/contributors","subscribers_url":"https://api.github.com/repos/lcrosenbu/alliance/subscribers","subscription_url":"https://api.github.com/repos/lcrosenbu/alliance/subscription","commits_url":"https://api.github.com/repos/lcrosenbu/alliance/commits{/sha}","git_commits_url":"https://api.github.com/repos/lcrosenbu/alliance/git/commits{/sha}","comments_url":"https://api.github.com/repos/lcrosenbu/alliance/comments{/number}","issue_comment_url":"https://api.github.com/repos/lcrosenbu/alliance/issues/comments{/number}","contents_url":"https://api.github.com/repos/lcrosenbu/alliance/contents/{+path}","compare_url":"https://api.github.com/repos/lcrosenbu/alliance/compare/{base}...{head}","merges_url":"https://api.github.com/repos/lcrosenbu/alliance/merges","archive_url":"https://api.github.com/repos/lcrosenbu/alliance/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/lcrosenbu/alliance/downloads","issues_url":"https://api.github.com/repos/lcrosenbu/alliance/issues{/number}","pulls_url":"https://api.github.com/repos/lcrosenbu/alliance/pulls{/number}","milestones_url":"https://api.github.com/repos/lcrosenbu/alliance/milestones{/number}","notifications_url":"https://api.github.com/repos/lcrosenbu/alliance/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/lcrosenbu/alliance/labels{/name}","releases_url":"https://api.github.com/repos/lcrosenbu/alliance/releases{/id}","deployments_url":"https://api.github.com/repos/lcrosenbu/alliance/deployments","created_at":"2016-06-01T15:11:16Z","updated_at":"2016-06-01T15:11:20Z","pushed_at":"2016-09-29T16:50:40Z","git_url":"git://github.com/lcrosenbu/alliance.git","ssh_url":"git@github.com:lcrosenbu/alliance.git","clone_url":"https://github.com/lcrosenbu/alliance.git","svn_url":"https://github.com/lcrosenbu/alliance","homepage":null,"size":25661,"stargazers_count":0,"watchers_count":0,"language":"Java","has_issues":false,"has_downloads":true,"has_wiki":false,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"codice:master","ref":"master","sha":"cc649bd7500da5af034e1b8b1a5180b5c33d69d0","user":{"login":"codice","id":2386734,"avatar_url":"https://avatars.githubusercontent.com/u/2386734?v=3","gravatar_id":"","url":"https://api.github.com/users/codice","html_url":"https://github.com/codice","followers_url":"https://api.github.com/users/codice/followers","following_url":"https://api.github.com/users/codice/following{/other_user}","gists_url":"https://api.github.com/users/codice/gists{/gist_id}","starred_url":"https://api.github.com/users/codice/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/codice/subscriptions","organizations_url":"https://api.github.com/users/codice/orgs","repos_url":"https://api.github.com/users/codice/repos","events_url":"https://api.github.com/users/codice/events{/privacy}","received_events_url":"https://api.github.com/users/codice/received_events","type":"Organization","site_admin":false},"repo":{"id":52804523,"name":"alliance","full_name":"codice/alliance","owner":{"login":"codice","id":2386734,"avatar_url":"https://avatars.githubusercontent.com/u/2386734?v=3","gravatar_id":"","url":"https://api.github.com/users/codice","html_url":"https://github.com/codice","followers_url":"https://api.github.com/users/codice/followers","following_url":"https://api.github.com/users/codice/following{/other_user}","gists_url":"https://api.github.com/users/codice/gists{/gist_id}","starred_url":"https://api.github.com/users/codice/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/codice/subscriptions","organizations_url":"https://api.github.com/users/codice/orgs","repos_url":"https://api.github.com/users/codice/repos","events_url":"https://api.github.com/users/codice/events{/privacy}","received_events_url":"https://api.github.com/users/codice/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/codice/alliance","description":"","fork":false,"url":"https://api.github.com/repos/codice/alliance","forks_url":"https://api.github.com/repos/codice/alliance/forks","keys_url":"https://api.github.com/repos/codice/alliance/keys{/key_id}","collaborators_url":"https://api.github.com/repos/codice/alliance/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/codice/alliance/teams","hooks_url":"https://api.github.com/repos/codice/alliance/hooks","issue_events_url":"https://api.github.com/repos/codice/alliance/issues/events{/number}","events_url":"https://api.github.com/repos/codice/alliance/events","assignees_url":"https://api.github.com/repos/codice/alliance/assignees{/user}","branches_url":"https://api.github.com/repos/codice/alliance/branches{/branch}","tags_url":"https://api.github.com/repos/codice/alliance/tags","blobs_url":"https://api.github.com/repos/codice/alliance/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/codice/alliance/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/codice/alliance/git/refs{/sha}","trees_url":"https://api.github.com/repos/codice/alliance/git/trees{/sha}","statuses_url":"https://api.github.com/repos/codice/alliance/statuses/{sha}","languages_url":"https://api.github.com/repos/codice/alliance/languages","stargazers_url":"https://api.github.com/repos/codice/alliance/stargazers","contributors_url":"https://api.github.com/repos/codice/alliance/contributors","subscribers_url":"https://api.github.com/repos/codice/alliance/subscribers","subscription_url":"https://api.github.com/repos/codice/alliance/subscription","commits_url":"https://api.github.com/repos/codice/alliance/commits{/sha}","git_commits_url":"https://api.github.com/repos/codice/alliance/git/commits{/sha}","comments_url":"https://api.github.com/repos/codice/alliance/comments{/number}","issue_comment_url":"https://api.github.com/repos/codice/alliance/issues/comments{/number}","contents_url":"https://api.github.com/repos/codice/alliance/contents/{+path}","compare_url":"https://api.github.com/repos/codice/alliance/compare/{base}...{head}","merges_url":"https://api.github.com/repos/codice/alliance/merges","archive_url":"https://api.github.com/repos/codice/alliance/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/codice/alliance/downloads","issues_url":"https://api.github.com/repos/codice/alliance/issues{/number}","pulls_url":"https://api.github.com/repos/codice/alliance/pulls{/number}","milestones_url":"https://api.github.com/repos/codice/alliance/milestones{/number}","notifications_url":"https://api.github.com/repos/codice/alliance/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/codice/alliance/labels{/name}","releases_url":"https://api.github.com/repos/codice/alliance/releases{/id}","deployments_url":"https://api.github.com/repos/codice/alliance/deployments","created_at":"2016-02-29T16:10:33Z","updated_at":"2016-06-14T21:40:26Z","pushed_at":"2016-09-29T16:50:40Z","git_url":"git://github.com/codice/alliance.git","ssh_url":"git@github.com:codice/alliance.git","clone_url":"https://github.com/codice/alliance.git","svn_url":"https://github.com/codice/alliance","homepage":null,"size":26208,"stargazers_count":2,"watchers_count":2,"language":"Java","has_issues":false,"has_downloads":true,"has_wiki":false,"has_pages":false,"forks_count":29,"mirror_url":null,"open_issues_count":4,"forks":29,"open_issues":4,"watchers":2,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/codice/alliance/pulls/152"},"html":{"href":"https://github.com/codice/alliance/pull/152"},"issue":{"href":"https://api.github.com/repos/codice/alliance/issues/152"},"comments":{"href":"https://api.github.com/repos/codice/alliance/issues/152/comments"},"review_comments":{"href":"https://api.github.com/repos/codice/alliance/pulls/152/comments"},"review_comment":{"href":"https://api.github.com/repos/codice/alliance/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/codice/alliance/pulls/152/commits"},"statuses":{"href":"https://api.github.com/repos/codice/alliance/statuses/8f5b5d6fb5c605e0a3bd6178fefb804c61d9d8d9"}}}}
{"actor":{"display_login":"lcrosenbu"}}