cylc / cylc-ui

Web app for monitoring and controlling Cylc workflows
https://cylc.github.io
GNU General Public License v3.0
37 stars 27 forks source link

Add copy button for ID in command menu and log file path in Log view #1966

Closed MetRonnie closed 2 weeks ago

MetRonnie commented 1 month ago

Brushed off this branch which had been languishing for a while.

Also removed username from ID displayed in the menu, to make it shorter

image

image

For the log view, I made the path truncate but leave the filename visible if the viewport is narrow:

image

Check List

MetRonnie commented 1 month ago

Will also add to the log view path by popular request...

oliver-sanders commented 2 weeks ago

Also removed username from ID displayed in the menu, to make it shorter

The user name was intended to be displayed as the UI was written in a way that allowed it to display multiple user's workflows (see BoM requirement). This is also the reason that the cylcTree has a user-name level at the top of the tree.

Given lack of Cylc resourcing and the BoM development of a top-level multi-user interface, I think it is ok to drop this, but we should leave the door open to multi-user functionality in the future (we didn't decide to drop this, we just didn't get around to it).

We could add Electron to the GH Actions testing matrix if desired

This is what we're testing with locally, makes sense to test in CI.