[GitHub] cleroux opened a new pull request #832: Monitor 2.0: HTML, CSS, and plot updates

[GitBox <gi...@apache.org>]

cleroux opened a new pull request #832: Monitor 2.0: HTML, CSS, and plot updates
URL: https://github.com/apache/accumulo/pull/832
 
 
   Fixed a .container div nested in a .container-fluid. Bootstrap [container classes are not nestable](https://getbootstrap.com/docs/3.3/css/#overview-container).
   
   Changed main div from absolute positioning to relative.  This was causing plot tooltips to display offset from the actual location of the data point. (Tooltips to be added as a separate pull request)
   
   Replaced divs using .center-block class with Bootstrap's .row and .col-* to improve layout responsiveness.
   Also moved <h3> titles inside .row and .col-* divs.
   
   Cleaned up unused and redundant CSS rules.
   
   Side-by-side plots on the overview page still look fine at the small layout. Changed .col-md-6 to .col-sm-6
   
   Removed time zone from plots' X axis tick labels. Planning to add other means of displaying and selecting time zone.  Plot option objects can be reused so they were moved outside of makePlots().
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services