Skip to content

Tidy up the shelf monitor web interface

Dylan Oliver Teague requested to merge feature/shelfmonitor-display into develop

Description

This MR is aimed at cleaning up as the ShelfMonitor code as well as making the monitoring suite more visually appealing. While !65 (merged) takes care of the back end, this aims more at the user experience. Changes made include:

  • Moving variables around to appropriate spots
  • Changing the format for some input json files for clearer javascript
  • Creating special css classes for the monitor table to fine tune formatting

Related Issue

Closes #153 (closed).

How Has This Been Tested?

The monitoring suite works as did before on teh coffin setup, but it has some added bells and whistles, a flashy one being info about a monitorable is displayed is the name is hovered over (gives register name and the doc string on the register).

Screenshots:

image

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.
Edited by Dylan Oliver Teague

Merge request reports