Skip to content
This repository was archived by the owner on Dec 21, 2023. It is now read-only.

Make minimize-to-tray optional #287

Open
mvastarelli opened this issue Apr 25, 2022 · 1 comment
Open

Make minimize-to-tray optional #287

mvastarelli opened this issue Apr 25, 2022 · 1 comment
Labels
enhancement New feature or request

Comments

@mvastarelli
Copy link
Contributor

Currently the application will minimize to the tray by default. We should turn this into an optional feature since the icon for MM is not widely known and it could confuse the user to see the application "disappear" as soon as it's been minimized.

@mvastarelli mvastarelli added the enhancement New feature or request label Apr 25, 2022
@mvastarelli mvastarelli added this to the May Release B milestone Apr 25, 2022
@mvastarelli
Copy link
Contributor Author

To fix the following should be done.

  1. Disable this functionality by default in the main process.
  2. Add a new app setting to toggle this.
  3. Update the Settings screen to display this setting.
  4. Add a hook to the main process to react when this setting is changed.

Optionally we could update the handler in the main process to check the app setting whenever the user clicks the minimize button, but I'm uncomfortable with the main window taking a direct dependency on a module.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant