daisy / pipeline-ui

A user interface for the DAISY Pipeline 2
MIT License
5 stars 2 forks source link

Separator line in tray menu between engine status and the other items #131

Closed bertfrees closed 1 year ago

bertfrees commented 1 year ago

The engine status is always gray. It would make it stand out as a special item even more if there would be a separator line beneath it.

rdeltour commented 1 year ago

Yeah, I agree that the UX could be improved here. the current solution uses a "disabled" menu item, which is (by design) very low contrast (at least on macOS), and not very readable.

Another possibility would be to integrate the engine state (running/stopped) to the tray icon (e.g. use an icon with an oblique stroke when the engine is not running…) and move the port number detail to the about box.

marisademeglio commented 1 year ago

The engine state and address are available in the about box.

Worth noting: the "engine state" menu item in the tray turns into "start the engine" if the engine stops while the application is still open (though the engine stopping suddenly is not common behavior so you may have not seen this yet).

If we remove the engine state menu item, then we would (?) be removing this function too.

bertfrees commented 1 year ago

Another possibility would be to integrate the engine state (running/stopped) to the tray icon

Yes, thought about this too.

and move the port number detail to the about box.

Indeed I would remove the "on port xxx". I wouldn't remove the whole menu item.

marisademeglio commented 1 year ago

This is what I can do now:

Screenshot 2023-03-16 at 16 39 49

Alt: tray menu says "engine status, separator, open window, new job, settings, about, separator, quit"

?

And then we can make a new issue for using the icon to show status.

bertfrees commented 1 year ago

Perfect