{"action":"created","comment":{"url":"https://api.github.com/repos/influxdata/chronograf/pulls/comments/118800025","pull_request_review_id":40591506,"id":118800025,"diff_hunk":"@@ -1,11 +1,14 @@\n import React, {PropTypes, Component} from 'react'\n+import classnames from 'classnames'\n \n import OnClickOutside from 'shared/components/OnClickOutside'\n import ConfirmButtons from 'shared/components/ConfirmButtons'\n \n-const DeleteButton = ({onClickDelete}) => (\n+const DeleteButton = ({onClickDelete, buttonSize}) => (\n   <button\n-    className=\"btn btn-xs btn-danger admin-table--hidden\"\n+    className={classnames('btn btn-danger table--show-on-row-hover', {\n+      [buttonSize]: buttonSize,","path":"ui/src/shared/components/DeleteConfirmButtons.js","position":12,"original_position":12,"commit_id":"c20a82912bdd0359bca5657ef3275ee2b84264bb","original_commit_id":"c20a82912bdd0359bca5657ef3275ee2b84264bb","user":{"login":"lukevmorris","id":1438089,"avatar_url":"https://avatars2.githubusercontent.com/u/1438089?v=3","gravatar_id":"","url":"https://api.github.com/users/lukevmorris","html_url":"https://github.com/lukevmorris","followers_url":"https://api.github.com/users/lukevmorris/followers","following_url":"https://api.github.com/users/lukevmorris/following{/other_user}","gists_url":"https://api.github.com/users/lukevmorris/gists{/gist_id}","starred_url":"https://api.github.com/users/lukevmorris/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lukevmorris/subscriptions","organizations_url":"https://api.github.com/users/lukevmorris/orgs","repos_url":"https://api.github.com/users/lukevmorris/repos","events_url":"https://api.github.com/users/lukevmorris/events{/privacy}","received_events_url":"https://api.github.com/users/lukevmorris/received_events","type":"User","site_admin":false},"body":"Same thing here","created_at":"2017-05-26T22:29:38Z","updated_at":"2017-05-26T22:49:47Z","html_url":"https://github.com/influxdata/chronograf/pull/1544#discussion_r118800025","pull_request_url":"https://api.github.com/repos/influxdata/chronograf/pulls/1544","_links":{"self":{"href":"https://api.github.com/repos/influxdata/chronograf/pulls/comments/118800025"},"html":{"href":"https://github.com/influxdata/chronograf/pull/1544#discussion_r118800025"},"pull_request":{"href":"https://api.github.com/repos/influxdata/chronograf/pulls/1544"}}},"pull_request":{"url":"https://api.github.com/repos/influxdata/chronograf/pulls/1544","id":122511601,"html_url":"https://github.com/influxdata/chronograf/pull/1544","diff_url":"https://github.com/influxdata/chronograf/pull/1544.diff","patch_url":"https://github.com/influxdata/chronograf/pull/1544.patch","issue_url":"https://api.github.com/repos/influxdata/chronograf/issues/1544","number":1544,"state":"open","locked":false,"title":"Theme refactor","user":{"login":"alexpaxton","id":2433762,"avatar_url":"https://avatars2.githubusercontent.com/u/2433762?v=3","gravatar_id":"","url":"https://api.github.com/users/alexpaxton","html_url":"https://github.com/alexpaxton","followers_url":"https://api.github.com/users/alexpaxton/followers","following_url":"https://api.github.com/users/alexpaxton/following{/other_user}","gists_url":"https://api.github.com/users/alexpaxton/gists{/gist_id}","starred_url":"https://api.github.com/users/alexpaxton/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/alexpaxton/subscriptions","organizations_url":"https://api.github.com/users/alexpaxton/orgs","repos_url":"https://api.github.com/users/alexpaxton/repos","events_url":"https://api.github.com/users/alexpaxton/events{/privacy}","received_events_url":"https://api.github.com/users/alexpaxton/received_events","type":"User","site_admin":false},"body":"  - [ ] CHANGELOG.md updated with a link to the PR (not the Issue)\r\n  - [x] Rebased/mergable\r\n  - [x] Tests pass\r\n\r\n### The problem\r\n- Find myself having to use `!important` to override the bootstrap theme in order to make a \"dark theme\", and in turn using more `!important` to override my overrides.\r\n- Lots of the CSS in chronograf is actually just overriding aspects of the theme. The UI has evolved a lot and the theme has not, which results in a lot of redundancy\r\n- The bootstrap theme looks very different than the current UI of chronograf\r\n- There are many repetitive solutions in the CSS, for example there are 3 different sets of styles for tab selectors that all look the same\r\n\r\n### The Solution\r\n- Make the bootstrap theme dark\r\n  - Refactor the theme to not use `!important` so gratuitously\r\n- Take every bit of CSS that overrides the theme and incorporate it into the theme\r\n  - Dropdowns\r\n  - Multi-Select Dropdowns\r\n  - Radio Tabs (ex: visualization type in cell editor)\r\n  - Dark Form & Input styles\r\n  - Dark Panels\r\n  - Dark Tables\r\n  - Alerts (Flash Messages)\r\n- Remove unused and redundant styles\r\n- Comb through the UI and use styles from the theme instead of one-off solutions\r\n  - Focus on generic, re-usable styles instead of high specificity\r\n- Unify discordant styles from around the entire UI\r\n- Remove modules from the theme that are never used in the UI\r\n  - `Jumbotron`\r\n  - `Bootstrap Tooltips`\r\n  - `Bootstrap Popovers`\r\n  - `Cloud Pricing Slider`\r\n  - `Hero Images for the Cloud App`\r\n  - `List Groups`\r\n- Remove unused stylesheets\r\n  - `theme/theme-dark.scss`\r\n  - `components/dropdown.scss`\r\n  - `components/multi-select-dropdowns.scss`\r\n- Visual polish on the entire Admin UI page\r\n  - Managing all table sizing via a constants file\r\n- Refactor styles for the Kapacitor rule builder page\r\n  - Using about half as many styles to accomplish the same look & feel\r\n\r\n### Preview\r\n\r\n![scale-single-stat-type](https://cloud.githubusercontent.com/assets/2433762/26467927/6ce47e80-4149-11e7-88e6-1cb158c28a51.gif)\r\nScale font size in relation to `cell.h`\r\n\r\n\r\n![users-table-polish](https://cloud.githubusercontent.com/assets/2433762/26467945/7c38dc3c-4149-11e7-916b-316a7cd9f28f.gif)\r\n![new-user](https://cloud.githubusercontent.com/assets/2433762/26467950/7f7a5d9e-4149-11e7-9828-b87dba2cb95b.gif)\r\nSome admin UI polish (no more table column jitter)\r\n\r\n\r\n<img width=\"920\" alt=\"screen shot 2017-05-23 at 6 07 43 pm\" src=\"https://cloud.githubusercontent.com/assets/2433762/26468002/a93fe0a4-4149-11e7-9d5b-ba516d224860.png\">\r\nStyled redacted input\r\n\r\n\r\n![screen shot 2017-05-16 at 12 05 45 pm](https://cloud.githubusercontent.com/assets/2433762/26468010/b57485d2-4149-11e7-8e79-d62a88454b27.png)\r\nUpdated button styles (Yellow buttons are newest)\r\n\r\n\r\n![screen shot 2017-05-15 at 3 15 06 pm](https://cloud.githubusercontent.com/assets/2433762/26468025/c2b5d822-4149-11e7-8e8d-e76d1a997547.png)\r\nUpdated alert styles (boy do I love gradients)\r\n\r\n\r\n<img width=\"215\" alt=\"screen shot 2017-05-25 at 1 00 20 pm\" src=\"https://cloud.githubusercontent.com/assets/2433762/26468139/374b2c00-414a-11e7-9ea2-9b48396ed82a.png\">\r\n<img width=\"219\" alt=\"screen shot 2017-05-25 at 1 00 14 pm\" src=\"https://cloud.githubusercontent.com/assets/2433762/26468144/396bda02-414a-11e7-9a85-59e10bea2838.png\">\r\nStandardized alternate themes for dropdowns\r\n\r\n<img width=\"194\" alt=\"screen shot 2017-05-25 at 2 32 18 pm\" src=\"https://cloud.githubusercontent.com/assets/2433762/26471800/0bb12d84-4158-11e7-9948-3b54fd10cc82.png\">\r\nDistinct style for \"Add\" items in a dropdown","created_at":"2017-05-25T19:48:27Z","updated_at":"2017-05-26T22:49:47Z","closed_at":null,"merged_at":null,"merge_commit_sha":"6fa236c09026fe5821d779982bfb3f354ad80e1d","assignee":{"login":"lukevmorris","id":1438089,"avatar_url":"https://avatars2.githubusercontent.com/u/1438089?v=3","gravatar_id":"","url":"https://api.github.com/users/lukevmorris","html_url":"https://github.com/lukevmorris","followers_url":"https://api.github.com/users/lukevmorris/followers","following_url":"https://api.github.com/users/lukevmorris/following{/other_user}","gists_url":"https://api.github.com/users/lukevmorris/gists{/gist_id}","starred_url":"https://api.github.com/users/lukevmorris/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lukevmorris/subscriptions","organizations_url":"https://api.github.com/users/lukevmorris/orgs","repos_url":"https://api.github.com/users/lukevmorris/repos","events_url":"https://api.github.com/users/lukevmorris/events{/privacy}","received_events_url":"https://api.github.com/users/lukevmorris/received_events","type":"User","site_admin":false},"assignees":[{"login":"lukevmorris","id":1438089,"avatar_url":"https://avatars2.githubusercontent.com/u/1438089?v=3","gravatar_id":"","url":"https://api.github.com/users/lukevmorris","html_url":"https://github.com/lukevmorris","followers_url":"https://api.github.com/users/lukevmorris/followers","following_url":"https://api.github.com/users/lukevmorris/following{/other_user}","gists_url":"https://api.github.com/users/lukevmorris/gists{/gist_id}","starred_url":"https://api.github.com/users/lukevmorris/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lukevmorris/subscriptions","organizations_url":"https://api.github.com/users/lukevmorris/orgs","repos_url":"https://api.github.com/users/lukevmorris/repos","events_url":"https://api.github.com/users/lukevmorris/events{/privacy}","received_events_url":"https://api.github.com/users/lukevmorris/received_events","type":"User","site_admin":false}],"requested_reviewers":[],"milestone":null,"commits_url":"https://api.github.com/repos/influxdata/chronograf/pulls/1544/commits","review_comments_url":"https://api.github.com/repos/influxdata/chronograf/pulls/1544/comments","review_comment_url":"https://api.github.com/repos/influxdata/chronograf/pulls/comments{/number}","comments_url":"https://api.github.com/repos/influxdata/chronograf/issues/1544/comments","statuses_url":"https://api.github.com/repos/influxdata/chronograf/statuses/c20a82912bdd0359bca5657ef3275ee2b84264bb","head":{"label":"influxdata:theme-refactor","ref":"theme-refactor","sha":"c20a82912bdd0359bca5657ef3275ee2b84264bb","user":{"login":"influxdata","id":5713248,"avatar_url":"https://avatars1.githubusercontent.com/u/5713248?v=3","gravatar_id":"","url":"https://api.github.com/users/influxdata","html_url":"https://github.com/influxdata","followers_url":"https://api.github.com/users/influxdata/followers","following_url":"https://api.github.com/users/influxdata/following{/other_user}","gists_url":"https://api.github.com/users/influxdata/gists{/gist_id}","starred_url":"https://api.github.com/users/influxdata/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/influxdata/subscriptions","organizations_url":"https://api.github.com/users/influxdata/orgs","repos_url":"https://api.github.com/users/influxdata/repos","events_url":"https://api.github.com/users/influxdata/events{/privacy}","received_events_url":"https://api.github.com/users/influxdata/received_events","type":"Organization","site_admin":false},"repo":{"id":66506962,"name":"chronograf","full_name":"influxdata/chronograf","owner":{"login":"influxdata","id":5713248,"avatar_url":"https://avatars1.githubusercontent.com/u/5713248?v=3","gravatar_id":"","url":"https://api.github.com/users/influxdata","html_url":"https://github.com/influxdata","followers_url":"https://api.github.com/users/influxdata/followers","following_url":"https://api.github.com/users/influxdata/following{/other_user}","gists_url":"https://api.github.com/users/influxdata/gists{/gist_id}","starred_url":"https://api.github.com/users/influxdata/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/influxdata/subscriptions","organizations_url":"https://api.github.com/users/influxdata/orgs","repos_url":"https://api.github.com/users/influxdata/repos","events_url":"https://api.github.com/users/influxdata/events{/privacy}","received_events_url":"https://api.github.com/users/influxdata/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/influxdata/chronograf","description":"Open source monitoring and visualization UI for the TICK stack","fork":false,"url":"https://api.github.com/repos/influxdata/chronograf","forks_url":"https://api.github.com/repos/influxdata/chronograf/forks","keys_url":"https://api.github.com/repos/influxdata/chronograf/keys{/key_id}","collaborators_url":"https://api.github.com/repos/influxdata/chronograf/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/influxdata/chronograf/teams","hooks_url":"https://api.github.com/repos/influxdata/chronograf/hooks","issue_events_url":"https://api.github.com/repos/influxdata/chronograf/issues/events{/number}","events_url":"https://api.github.com/repos/influxdata/chronograf/events","assignees_url":"https://api.github.com/repos/influxdata/chronograf/assignees{/user}","branches_url":"https://api.github.com/repos/influxdata/chronograf/branches{/branch}","tags_url":"https://api.github.com/repos/influxdata/chronograf/tags","blobs_url":"https://api.github.com/repos/influxdata/chronograf/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/influxdata/chronograf/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/influxdata/chronograf/git/refs{/sha}","trees_url":"https://api.github.com/repos/influxdata/chronograf/git/trees{/sha}","statuses_url":"https://api.github.com/repos/influxdata/chronograf/statuses/{sha}","languages_url":"https://api.github.com/repos/influxdata/chronograf/languages","stargazers_url":"https://api.github.com/repos/influxdata/chronograf/stargazers","contributors_url":"https://api.github.com/repos/influxdata/chronograf/contributors","subscribers_url":"https://api.github.com/repos/influxdata/chronograf/subscribers","subscription_url":"https://api.github.com/repos/influxdata/chronograf/subscription","commits_url":"https://api.github.com/repos/influxdata/chronograf/commits{/sha}","git_commits_url":"https://api.github.com/repos/influxdata/chronograf/git/commits{/sha}","comments_url":"https://api.github.com/repos/influxdata/chronograf/comments{/number}","issue_comment_url":"https://api.github.com/repos/influxdata/chronograf/issues/comments{/number}","contents_url":"https://api.github.com/repos/influxdata/chronograf/contents/{+path}","compare_url":"https://api.github.com/repos/influxdata/chronograf/compare/{base}...{head}","merges_url":"https://api.github.com/repos/influxdata/chronograf/merges","archive_url":"https://api.github.com/repos/influxdata/chronograf/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/influxdata/chronograf/downloads","issues_url":"https://api.github.com/repos/influxdata/chronograf/issues{/number}","pulls_url":"https://api.github.com/repos/influxdata/chronograf/pulls{/number}","milestones_url":"https://api.github.com/repos/influxdata/chronograf/milestones{/number}","notifications_url":"https://api.github.com/repos/influxdata/chronograf/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/influxdata/chronograf/labels{/name}","releases_url":"https://api.github.com/repos/influxdata/chronograf/releases{/id}","deployments_url":"https://api.github.com/repos/influxdata/chronograf/deployments","created_at":"2016-08-24T23:28:56Z","updated_at":"2017-05-25T09:08:16Z","pushed_at":"2017-05-26T22:44:56Z","git_url":"git://github.com/influxdata/chronograf.git","ssh_url":"git@github.com:influxdata/chronograf.git","clone_url":"https://github.com/influxdata/chronograf.git","svn_url":"https://github.com/influxdata/chronograf","homepage":"https://www.influxdata.com/time-series-platform/chronograf/","size":40434,"stargazers_count":274,"watchers_count":274,"language":"Go","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":50,"mirror_url":null,"open_issues_count":193,"forks":50,"open_issues":193,"watchers":274,"default_branch":"master"}},"base":{"label":"influxdata:master","ref":"master","sha":"c7d74122b69e3cc6d75fe77e9a85f8065ad2be8a","user":{"login":"influxdata","id":5713248,"avatar_url":"https://avatars1.githubusercontent.com/u/5713248?v=3","gravatar_id":"","url":"https://api.github.com/users/influxdata","html_url":"https://github.com/influxdata","followers_url":"https://api.github.com/users/influxdata/followers","following_url":"https://api.github.com/users/influxdata/following{/other_user}","gists_url":"https://api.github.com/users/influxdata/gists{/gist_id}","starred_url":"https://api.github.com/users/influxdata/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/influxdata/subscriptions","organizations_url":"https://api.github.com/users/influxdata/orgs","repos_url":"https://api.github.com/users/influxdata/repos","events_url":"https://api.github.com/users/influxdata/events{/privacy}","received_events_url":"https://api.github.com/users/influxdata/received_events","type":"Organization","site_admin":false},"repo":{"id":66506962,"name":"chronograf","full_name":"influxdata/chronograf","owner":{"login":"influxdata","id":5713248,"avatar_url":"https://avatars1.githubusercontent.com/u/5713248?v=3","gravatar_id":"","url":"https://api.github.com/users/influxdata","html_url":"https://github.com/influxdata","followers_url":"https://api.github.com/users/influxdata/followers","following_url":"https://api.github.com/users/influxdata/following{/other_user}","gists_url":"https://api.github.com/users/influxdata/gists{/gist_id}","starred_url":"https://api.github.com/users/influxdata/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/influxdata/subscriptions","organizations_url":"https://api.github.com/users/influxdata/orgs","repos_url":"https://api.github.com/users/influxdata/repos","events_url":"https://api.github.com/users/influxdata/events{/privacy}","received_events_url":"https://api.github.com/users/influxdata/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/influxdata/chronograf","description":"Open source monitoring and visualization UI for the TICK stack","fork":false,"url":"https://api.github.com/repos/influxdata/chronograf","forks_url":"https://api.github.com/repos/influxdata/chronograf/forks","keys_url":"https://api.github.com/repos/influxdata/chronograf/keys{/key_id}","collaborators_url":"https://api.github.com/repos/influxdata/chronograf/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/influxdata/chronograf/teams","hooks_url":"https://api.github.com/repos/influxdata/chronograf/hooks","issue_events_url":"https://api.github.com/repos/influxdata/chronograf/issues/events{/number}","events_url":"https://api.github.com/repos/influxdata/chronograf/events","assignees_url":"https://api.github.com/repos/influxdata/chronograf/assignees{/user}","branches_url":"https://api.github.com/repos/influxdata/chronograf/branches{/branch}","tags_url":"https://api.github.com/repos/influxdata/chronograf/tags","blobs_url":"https://api.github.com/repos/influxdata/chronograf/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/influxdata/chronograf/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/influxdata/chronograf/git/refs{/sha}","trees_url":"https://api.github.com/repos/influxdata/chronograf/git/trees{/sha}","statuses_url":"https://api.github.com/repos/influxdata/chronograf/statuses/{sha}","languages_url":"https://api.github.com/repos/influxdata/chronograf/languages","stargazers_url":"https://api.github.com/repos/influxdata/chronograf/stargazers","contributors_url":"https://api.github.com/repos/influxdata/chronograf/contributors","subscribers_url":"https://api.github.com/repos/influxdata/chronograf/subscribers","subscription_url":"https://api.github.com/repos/influxdata/chronograf/subscription","commits_url":"https://api.github.com/repos/influxdata/chronograf/commits{/sha}","git_commits_url":"https://api.github.com/repos/influxdata/chronograf/git/commits{/sha}","comments_url":"https://api.github.com/repos/influxdata/chronograf/comments{/number}","issue_comment_url":"https://api.github.com/repos/influxdata/chronograf/issues/comments{/number}","contents_url":"https://api.github.com/repos/influxdata/chronograf/contents/{+path}","compare_url":"https://api.github.com/repos/influxdata/chronograf/compare/{base}...{head}","merges_url":"https://api.github.com/repos/influxdata/chronograf/merges","archive_url":"https://api.github.com/repos/influxdata/chronograf/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/influxdata/chronograf/downloads","issues_url":"https://api.github.com/repos/influxdata/chronograf/issues{/number}","pulls_url":"https://api.github.com/repos/influxdata/chronograf/pulls{/number}","milestones_url":"https://api.github.com/repos/influxdata/chronograf/milestones{/number}","notifications_url":"https://api.github.com/repos/influxdata/chronograf/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/influxdata/chronograf/labels{/name}","releases_url":"https://api.github.com/repos/influxdata/chronograf/releases{/id}","deployments_url":"https://api.github.com/repos/influxdata/chronograf/deployments","created_at":"2016-08-24T23:28:56Z","updated_at":"2017-05-25T09:08:16Z","pushed_at":"2017-05-26T22:44:56Z","git_url":"git://github.com/influxdata/chronograf.git","ssh_url":"git@github.com:influxdata/chronograf.git","clone_url":"https://github.com/influxdata/chronograf.git","svn_url":"https://github.com/influxdata/chronograf","homepage":"https://www.influxdata.com/time-series-platform/chronograf/","size":40434,"stargazers_count":274,"watchers_count":274,"language":"Go","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":50,"mirror_url":null,"open_issues_count":193,"forks":50,"open_issues":193,"watchers":274,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/influxdata/chronograf/pulls/1544"},"html":{"href":"https://github.com/influxdata/chronograf/pull/1544"},"issue":{"href":"https://api.github.com/repos/influxdata/chronograf/issues/1544"},"comments":{"href":"https://api.github.com/repos/influxdata/chronograf/issues/1544/comments"},"review_comments":{"href":"https://api.github.com/repos/influxdata/chronograf/pulls/1544/comments"},"review_comment":{"href":"https://api.github.com/repos/influxdata/chronograf/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/influxdata/chronograf/pulls/1544/commits"},"statuses":{"href":"https://api.github.com/repos/influxdata/chronograf/statuses/c20a82912bdd0359bca5657ef3275ee2b84264bb"}}}}
{"actor":{"display_login":"lukevmorris"}}