Skip to content
This repository has been archived by the owner on Jul 9, 2023. It is now read-only.

Issue minimizing to tray (Ubuntu) #35

Closed
DeflateAwning opened this issue Sep 28, 2020 · 5 comments
Closed

Issue minimizing to tray (Ubuntu) #35

DeflateAwning opened this issue Sep 28, 2020 · 5 comments
Assignees
Labels
bug Something isn't working electron when something is an electron problem
Milestone

Comments

@DeflateAwning
Copy link

I'm running this on the default Ubuntu 20.04.1 installation, and no tray icon appears when I close out. However, htop confirms that it is running in the background still. I'm wondering if we can add a configuration option to disable Minimize to Tray, and also potentially investigate why it is not minimizing. Thanks!

@n8maninger
Copy link
Collaborator

n8maninger commented Sep 28, 2020

I will definitely investigate why it is not minimizing. Are you using the default desktop, Gnome, KDE? In the mean time you should just be able to start it again and it will reopen the existing UI.

What would be the alternative to minimizing to tray? Just shutting down? For most users if you close Host Manager you shutdown your Sia Host, to prevent that I think minimize to tray is pretty important.

@n8maninger n8maninger self-assigned this Sep 28, 2020
@n8maninger n8maninger added the bug Something isn't working label Sep 28, 2020
@n8maninger n8maninger added this to the v1.0.24 milestone Sep 28, 2020
@DeflateAwning
Copy link
Author

I'm using the default install (Gnome).

I figured shutting down the Host Manager would be reasonable, but that makes sense that it wouldn't work for most people.

I thought that every time I opened a new Host Manager window, it was creating a new process while old ones continued to run. I just confirmed that this is likely not the case, as ps aux | grep -i "host manager" | wc -l counts 9 processes every time, even after closing (while it's closed), and after reopening.

I suppose this issue is pretty low priority. You can ignore the request to disable "Minimize to Tray" functionality.

@n8maninger
Copy link
Collaborator

n8maninger commented Sep 28, 2020

Thanks for confirming that it is not starting up a new service every time but showing the old one.

The platform Host Manager is built-on (Electron) starts a lot of child processes and is not very efficient in general.. It's probably one of my biggest annoyances and something I look forward to dropping entirely..

@n8maninger
Copy link
Collaborator

n8maninger commented Oct 6, 2020

Just an update, looks like this is a known Electron issue. I will try some of the solutions.

electron/electron#21445

@n8maninger n8maninger added the electron when something is an electron problem label Feb 7, 2021
@n8maninger
Copy link
Collaborator

Updated to electron 20, appears to be fixed

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working electron when something is an electron problem
Projects
None yet
Development

No branches or pull requests

2 participants