{"action":"created","comment":{"url":"https://api.github.com/repos/codeforamerica/jail-dashboard/pulls/comments/94646421","pull_request_review_id":15178175,"id":94646421,"diff_hunk":"@@ -16,6 +16,7 @@\n @import \"charts/population_capacity\";\n \n .tidy {\n-  height: 600px;\n-  overflow: scroll;\n+  height: 400px;\n+  margin-bottom: 3em;","path":"app/assets/stylesheets/application.scss","position":7,"original_position":7,"commit_id":"7cdb3ea7c6f09a3ff3328c9b0d0b9100caf6c11c","original_commit_id":"7cdb3ea7c6f09a3ff3328c9b0d0b9100caf6c11c","user":{"login":"tmaybe","id":546665,"avatar_url":"https://avatars.githubusercontent.com/u/546665?v=3","gravatar_id":"","url":"https://api.github.com/users/tmaybe","html_url":"https://github.com/tmaybe","followers_url":"https://api.github.com/users/tmaybe/followers","following_url":"https://api.github.com/users/tmaybe/following{/other_user}","gists_url":"https://api.github.com/users/tmaybe/gists{/gist_id}","starred_url":"https://api.github.com/users/tmaybe/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tmaybe/subscriptions","organizations_url":"https://api.github.com/users/tmaybe/orgs","repos_url":"https://api.github.com/users/tmaybe/repos","events_url":"https://api.github.com/users/tmaybe/events{/privacy}","received_events_url":"https://api.github.com/users/tmaybe/received_events","type":"User","site_admin":false},"body":"@jennifermarie I added a bottom margin to tidy so the _People who didn't post bond_ table wouldn't end at the bottom of the page.","created_at":"2017-01-04T19:18:30Z","updated_at":"2017-01-04T19:18:30Z","html_url":"https://github.com/codeforamerica/jail-dashboard/pull/120#discussion_r94646421","pull_request_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/pulls/120","_links":{"self":{"href":"https://api.github.com/repos/codeforamerica/jail-dashboard/pulls/comments/94646421"},"html":{"href":"https://github.com/codeforamerica/jail-dashboard/pull/120#discussion_r94646421"},"pull_request":{"href":"https://api.github.com/repos/codeforamerica/jail-dashboard/pulls/120"}}},"pull_request":{"url":"https://api.github.com/repos/codeforamerica/jail-dashboard/pulls/120","id":100163490,"html_url":"https://github.com/codeforamerica/jail-dashboard/pull/120","diff_url":"https://github.com/codeforamerica/jail-dashboard/pull/120.diff","patch_url":"https://github.com/codeforamerica/jail-dashboard/pull/120.patch","issue_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/issues/120","number":120,"state":"open","locked":false,"title":"prevent overly-long tables ","user":{"login":"jennifermarie","id":1070220,"avatar_url":"https://avatars.githubusercontent.com/u/1070220?v=3","gravatar_id":"","url":"https://api.github.com/users/jennifermarie","html_url":"https://github.com/jennifermarie","followers_url":"https://api.github.com/users/jennifermarie/followers","following_url":"https://api.github.com/users/jennifermarie/following{/other_user}","gists_url":"https://api.github.com/users/jennifermarie/gists{/gist_id}","starred_url":"https://api.github.com/users/jennifermarie/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jennifermarie/subscriptions","organizations_url":"https://api.github.com/users/jennifermarie/orgs","repos_url":"https://api.github.com/users/jennifermarie/repos","events_url":"https://api.github.com/users/jennifermarie/events{/privacy}","received_events_url":"https://api.github.com/users/jennifermarie/received_events","type":"User","site_admin":false},"body":"Closes #112 \r\n\r\n### Problem\r\nThe tables for the lists of (filterable) people / people who didn't post bond were excessively long in unfiltered state.\r\n\r\n### Solution\r\nSet a height of 400 for each section, with scroll on overflow-y. The visible scrollbar should make it clear to the user that there are more people in the table, but the page will not be so long as to obscure additional sections.","created_at":"2017-01-04T18:22:35Z","updated_at":"2017-01-04T19:18:30Z","closed_at":null,"merged_at":null,"merge_commit_sha":"5cf36d24879b511c93af7ce52a34e6733dff37d9","assignee":{"login":"tmaybe","id":546665,"avatar_url":"https://avatars.githubusercontent.com/u/546665?v=3","gravatar_id":"","url":"https://api.github.com/users/tmaybe","html_url":"https://github.com/tmaybe","followers_url":"https://api.github.com/users/tmaybe/followers","following_url":"https://api.github.com/users/tmaybe/following{/other_user}","gists_url":"https://api.github.com/users/tmaybe/gists{/gist_id}","starred_url":"https://api.github.com/users/tmaybe/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tmaybe/subscriptions","organizations_url":"https://api.github.com/users/tmaybe/orgs","repos_url":"https://api.github.com/users/tmaybe/repos","events_url":"https://api.github.com/users/tmaybe/events{/privacy}","received_events_url":"https://api.github.com/users/tmaybe/received_events","type":"User","site_admin":false},"assignees":[{"login":"tmaybe","id":546665,"avatar_url":"https://avatars.githubusercontent.com/u/546665?v=3","gravatar_id":"","url":"https://api.github.com/users/tmaybe","html_url":"https://github.com/tmaybe","followers_url":"https://api.github.com/users/tmaybe/followers","following_url":"https://api.github.com/users/tmaybe/following{/other_user}","gists_url":"https://api.github.com/users/tmaybe/gists{/gist_id}","starred_url":"https://api.github.com/users/tmaybe/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tmaybe/subscriptions","organizations_url":"https://api.github.com/users/tmaybe/orgs","repos_url":"https://api.github.com/users/tmaybe/repos","events_url":"https://api.github.com/users/tmaybe/events{/privacy}","received_events_url":"https://api.github.com/users/tmaybe/received_events","type":"User","site_admin":false}],"milestone":null,"commits_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/pulls/120/commits","review_comments_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/pulls/120/comments","review_comment_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/pulls/comments{/number}","comments_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/issues/120/comments","statuses_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/statuses/7cdb3ea7c6f09a3ff3328c9b0d0b9100caf6c11c","head":{"label":"codeforamerica:table_height","ref":"table_height","sha":"7cdb3ea7c6f09a3ff3328c9b0d0b9100caf6c11c","user":{"login":"codeforamerica","id":337792,"avatar_url":"https://avatars.githubusercontent.com/u/337792?v=3","gravatar_id":"","url":"https://api.github.com/users/codeforamerica","html_url":"https://github.com/codeforamerica","followers_url":"https://api.github.com/users/codeforamerica/followers","following_url":"https://api.github.com/users/codeforamerica/following{/other_user}","gists_url":"https://api.github.com/users/codeforamerica/gists{/gist_id}","starred_url":"https://api.github.com/users/codeforamerica/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/codeforamerica/subscriptions","organizations_url":"https://api.github.com/users/codeforamerica/orgs","repos_url":"https://api.github.com/users/codeforamerica/repos","events_url":"https://api.github.com/users/codeforamerica/events{/privacy}","received_events_url":"https://api.github.com/users/codeforamerica/received_events","type":"Organization","site_admin":false},"repo":{"id":72130706,"name":"jail-dashboard","full_name":"codeforamerica/jail-dashboard","owner":{"login":"codeforamerica","id":337792,"avatar_url":"https://avatars.githubusercontent.com/u/337792?v=3","gravatar_id":"","url":"https://api.github.com/users/codeforamerica","html_url":"https://github.com/codeforamerica","followers_url":"https://api.github.com/users/codeforamerica/followers","following_url":"https://api.github.com/users/codeforamerica/following{/other_user}","gists_url":"https://api.github.com/users/codeforamerica/gists{/gist_id}","starred_url":"https://api.github.com/users/codeforamerica/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/codeforamerica/subscriptions","organizations_url":"https://api.github.com/users/codeforamerica/orgs","repos_url":"https://api.github.com/users/codeforamerica/repos","events_url":"https://api.github.com/users/codeforamerica/events{/privacy}","received_events_url":"https://api.github.com/users/codeforamerica/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/codeforamerica/jail-dashboard","description":"A tool to help jail administrators, judges, and other stakeholders understand the conditions in metro jails, and use this data to visualize how their decisions–at both the individual and policy level–affect program, facility, and inmate outcomes.","fork":false,"url":"https://api.github.com/repos/codeforamerica/jail-dashboard","forks_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/forks","keys_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/keys{/key_id}","collaborators_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/teams","hooks_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/hooks","issue_events_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/issues/events{/number}","events_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/events","assignees_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/assignees{/user}","branches_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/branches{/branch}","tags_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/tags","blobs_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/git/refs{/sha}","trees_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/git/trees{/sha}","statuses_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/statuses/{sha}","languages_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/languages","stargazers_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/stargazers","contributors_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/contributors","subscribers_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/subscribers","subscription_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/subscription","commits_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/commits{/sha}","git_commits_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/git/commits{/sha}","comments_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/comments{/number}","issue_comment_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/issues/comments{/number}","contents_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/contents/{+path}","compare_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/compare/{base}...{head}","merges_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/merges","archive_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/downloads","issues_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/issues{/number}","pulls_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/pulls{/number}","milestones_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/milestones{/number}","notifications_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/labels{/name}","releases_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/releases{/id}","deployments_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/deployments","created_at":"2016-10-27T17:14:21Z","updated_at":"2016-12-22T12:24:35Z","pushed_at":"2017-01-04T19:16:42Z","git_url":"git://github.com/codeforamerica/jail-dashboard.git","ssh_url":"git@github.com:codeforamerica/jail-dashboard.git","clone_url":"https://github.com/codeforamerica/jail-dashboard.git","svn_url":"https://github.com/codeforamerica/jail-dashboard","homepage":null,"size":25221,"stargazers_count":2,"watchers_count":2,"language":"JavaScript","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":16,"forks":0,"open_issues":16,"watchers":2,"default_branch":"master"}},"base":{"label":"codeforamerica:master","ref":"master","sha":"70831ebf6d963f2a63bb9692c372921387503c29","user":{"login":"codeforamerica","id":337792,"avatar_url":"https://avatars.githubusercontent.com/u/337792?v=3","gravatar_id":"","url":"https://api.github.com/users/codeforamerica","html_url":"https://github.com/codeforamerica","followers_url":"https://api.github.com/users/codeforamerica/followers","following_url":"https://api.github.com/users/codeforamerica/following{/other_user}","gists_url":"https://api.github.com/users/codeforamerica/gists{/gist_id}","starred_url":"https://api.github.com/users/codeforamerica/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/codeforamerica/subscriptions","organizations_url":"https://api.github.com/users/codeforamerica/orgs","repos_url":"https://api.github.com/users/codeforamerica/repos","events_url":"https://api.github.com/users/codeforamerica/events{/privacy}","received_events_url":"https://api.github.com/users/codeforamerica/received_events","type":"Organization","site_admin":false},"repo":{"id":72130706,"name":"jail-dashboard","full_name":"codeforamerica/jail-dashboard","owner":{"login":"codeforamerica","id":337792,"avatar_url":"https://avatars.githubusercontent.com/u/337792?v=3","gravatar_id":"","url":"https://api.github.com/users/codeforamerica","html_url":"https://github.com/codeforamerica","followers_url":"https://api.github.com/users/codeforamerica/followers","following_url":"https://api.github.com/users/codeforamerica/following{/other_user}","gists_url":"https://api.github.com/users/codeforamerica/gists{/gist_id}","starred_url":"https://api.github.com/users/codeforamerica/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/codeforamerica/subscriptions","organizations_url":"https://api.github.com/users/codeforamerica/orgs","repos_url":"https://api.github.com/users/codeforamerica/repos","events_url":"https://api.github.com/users/codeforamerica/events{/privacy}","received_events_url":"https://api.github.com/users/codeforamerica/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/codeforamerica/jail-dashboard","description":"A tool to help jail administrators, judges, and other stakeholders understand the conditions in metro jails, and use this data to visualize how their decisions–at both the individual and policy level–affect program, facility, and inmate outcomes.","fork":false,"url":"https://api.github.com/repos/codeforamerica/jail-dashboard","forks_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/forks","keys_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/keys{/key_id}","collaborators_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/teams","hooks_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/hooks","issue_events_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/issues/events{/number}","events_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/events","assignees_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/assignees{/user}","branches_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/branches{/branch}","tags_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/tags","blobs_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/git/refs{/sha}","trees_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/git/trees{/sha}","statuses_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/statuses/{sha}","languages_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/languages","stargazers_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/stargazers","contributors_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/contributors","subscribers_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/subscribers","subscription_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/subscription","commits_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/commits{/sha}","git_commits_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/git/commits{/sha}","comments_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/comments{/number}","issue_comment_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/issues/comments{/number}","contents_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/contents/{+path}","compare_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/compare/{base}...{head}","merges_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/merges","archive_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/downloads","issues_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/issues{/number}","pulls_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/pulls{/number}","milestones_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/milestones{/number}","notifications_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/labels{/name}","releases_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/releases{/id}","deployments_url":"https://api.github.com/repos/codeforamerica/jail-dashboard/deployments","created_at":"2016-10-27T17:14:21Z","updated_at":"2016-12-22T12:24:35Z","pushed_at":"2017-01-04T19:16:42Z","git_url":"git://github.com/codeforamerica/jail-dashboard.git","ssh_url":"git@github.com:codeforamerica/jail-dashboard.git","clone_url":"https://github.com/codeforamerica/jail-dashboard.git","svn_url":"https://github.com/codeforamerica/jail-dashboard","homepage":null,"size":25221,"stargazers_count":2,"watchers_count":2,"language":"JavaScript","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":16,"forks":0,"open_issues":16,"watchers":2,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/codeforamerica/jail-dashboard/pulls/120"},"html":{"href":"https://github.com/codeforamerica/jail-dashboard/pull/120"},"issue":{"href":"https://api.github.com/repos/codeforamerica/jail-dashboard/issues/120"},"comments":{"href":"https://api.github.com/repos/codeforamerica/jail-dashboard/issues/120/comments"},"review_comments":{"href":"https://api.github.com/repos/codeforamerica/jail-dashboard/pulls/120/comments"},"review_comment":{"href":"https://api.github.com/repos/codeforamerica/jail-dashboard/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/codeforamerica/jail-dashboard/pulls/120/commits"},"statuses":{"href":"https://api.github.com/repos/codeforamerica/jail-dashboard/statuses/7cdb3ea7c6f09a3ff3328c9b0d0b9100caf6c11c"}}}}
{"actor":{"display_login":"tmaybe"}}