{"action":"created","issue":{"url":"https://api.github.com/repos/apache/accumulo/issues/745","repository_url":"https://api.github.com/repos/apache/accumulo","labels_url":"https://api.github.com/repos/apache/accumulo/issues/745/labels{/name}","comments_url":"https://api.github.com/repos/apache/accumulo/issues/745/comments","events_url":"https://api.github.com/repos/apache/accumulo/issues/745/events","html_url":"https://github.com/apache/accumulo/pull/745","id":377121401,"node_id":"MDExOlB1bGxSZXF1ZXN0MjI4MTU3ODYw","number":745,"title":"Monitor 2.0: Minor bug fixes, chart updates","user":{"login":"cleroux","id":1497949,"node_id":"MDQ6VXNlcjE0OTc5NDk=","avatar_url":"https://avatars3.githubusercontent.com/u/1497949?v=4","gravatar_id":"","url":"https://api.github.com/users/cleroux","html_url":"https://github.com/cleroux","followers_url":"https://api.github.com/users/cleroux/followers","following_url":"https://api.github.com/users/cleroux/following{/other_user}","gists_url":"https://api.github.com/users/cleroux/gists{/gist_id}","starred_url":"https://api.github.com/users/cleroux/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cleroux/subscriptions","organizations_url":"https://api.github.com/users/cleroux/orgs","repos_url":"https://api.github.com/users/cleroux/repos","events_url":"https://api.github.com/users/cleroux/events{/privacy}","received_events_url":"https://api.github.com/users/cleroux/received_events","type":"User","site_admin":false},"labels":[{"id":869765279,"node_id":"MDU6TGFiZWw4Njk3NjUyNzk=","url":"https://api.github.com/repos/apache/accumulo/labels/v2.0.0","name":"v2.0.0","color":"1d76db","default":false}],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2018-11-04T04:59:09Z","updated_at":"2018-11-30T04:07:31Z","closed_at":"2018-11-30T00:33:45Z","author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/apache/accumulo/pulls/745","html_url":"https://github.com/apache/accumulo/pull/745","diff_url":"https://github.com/apache/accumulo/pull/745.diff","patch_url":"https://github.com/apache/accumulo/pull/745.patch"},"body":"Fixed avatar jumping left/right between small/medium layouts.\r\nWhen resizing the browser window between medium and small bootstrap layouts, the accumulo avatar (or \"brand image\" in bootstrap terms) was shifting position on the navbar.\r\nAccording to the [Bootstrap Navbar example](https://getbootstrap.com/docs/3.3/components/#navbar-brand-image), the image element should be inside the navbar-brand element.\r\n\r\nFixed overview plots timeformat option not working because of a typo.\r\nCode was using \"timeFormat\" instead of \"timeformat\".\r\nIt was supposed to display like '00:00<br>EDT' but it was defaulting to '00:00'\r\nI fixed the bug but I think I liked it better without the time zone.  It looks really repetitive to show the time zone with every tick on ten charts.  To me, it seems obvious that it's local time.\r\nIf you feel it's appropriate I would like to remove the time zone from chart ticks.\r\n\r\nI simplified the creation of the charts by generalizing the chart options into one global style and one sub-style for the cache hit rate charts.\r\nThe makePlot() function in functions.js became less useful to the point that I decided to eliminate it and consolidate all of that functionality in overview.js.\r\n\r\nI added a very slight background gradient to the charts.  I felt they needed a bit of style.\r\n\r\nI also added a .plot CSS class to replace the inline styles on the chart divs.  In addition to conforming to best practices, this change will also allow me to make the charts hoverable with tooltips in an upcoming pull request."},"comment":{"url":"https://api.github.com/repos/apache/accumulo/issues/comments/443084314","html_url":"https://github.com/apache/accumulo/pull/745#issuecomment-443084314","issue_url":"https://api.github.com/repos/apache/accumulo/issues/745","id":443084314,"node_id":"MDEyOklzc3VlQ29tbWVudDQ0MzA4NDMxNA==","user":{"login":"cleroux","id":1497949,"node_id":"MDQ6VXNlcjE0OTc5NDk=","avatar_url":"https://avatars3.githubusercontent.com/u/1497949?v=4","gravatar_id":"","url":"https://api.github.com/users/cleroux","html_url":"https://github.com/cleroux","followers_url":"https://api.github.com/users/cleroux/followers","following_url":"https://api.github.com/users/cleroux/following{/other_user}","gists_url":"https://api.github.com/users/cleroux/gists{/gist_id}","starred_url":"https://api.github.com/users/cleroux/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cleroux/subscriptions","organizations_url":"https://api.github.com/users/cleroux/orgs","repos_url":"https://api.github.com/users/cleroux/repos","events_url":"https://api.github.com/users/cleroux/events{/privacy}","received_events_url":"https://api.github.com/users/cleroux/received_events","type":"User","site_admin":false},"created_at":"2018-11-30T04:07:31Z","updated_at":"2018-11-30T04:07:31Z","author_association":"CONTRIBUTOR","body":"Thanks for merging this PR.  I had hoped to find a good solution for the timezone but I haven't had much time and what I tried so far wasn't very good.  I'll submit another PR when I can."}}
{"actor":{"display_login":"cleroux"}}