Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added option to minimize to tray #1788

Closed
wants to merge 1 commit into from
Closed

Conversation

vectorlit
Copy link

First time contributor checklist

Contributor checklist

  • My contribution is fully baked and ready to be merged as is
  • My changes are rebased on the latest master branch
  • My commits are in nice logical chunks
  • I have followed the best practices in my commit messages
  • I have tested my contribution on these platforms:
  • OSX Sierra
  • Windows 10
  • Xubuntu 17.04
  • Ubuntu 17.10
  • My changes pass all the local tests 100%
  • I have considered whether my changes need additional tests, and in the case they do, I have written them

Description

This commit proposes to address #1480

Added an option to minimize the application to tray. The option defaults to false on first run, but is saved according to current practices and loaded on subsequent runs.

This commit needed 4 new localization strings for the menu items.

System tray icon maintains a menu and click-behavior consistent with modern communication applications.

System tray icon has been tested on Windows, OSX, and Linux (XFCE and older versions of gnome). Newer versions of Gnome (including newest non-LTS versions of Ubuntu) do not support a system tray without modification. If this is an issue in the future, suggest adding module "getos" and maintaining a supported release list (beyond the scope of this commit).

@scottnonnenberg
Copy link
Contributor

Thanks for jumping in and doing this.

First, we have another active pull request implementing a try for the app. Instead of adding a user-visible option, it starts out with just command-line arguments, which I like because it will allow us to test it before rolling out to a large set of users: #1676

Maybe we could put this one on hold, focus on finalizing and testing that one, then come back to this when we need a user-visible option?

@vectorlit
Copy link
Author

Yeah I just saw that after I submitted the request. Typical programmer failure on my part. But actually the other pull request's code is great. Only immediate suggestion for that request to merge from mine would be to set the darwin platform icon as icon_16, as otherwise it'll be too large for the top bar on OSX.

Otherwise the options logic from this pull request should be easy to integrate (if it helps) following UAT of the other pull.

@m-pilia
Copy link
Contributor

m-pilia commented Nov 19, 2017

Hello @vectorlit . Thanks, I followed your suggestion to use the icon_16 on Darwin. Good to know for me since I cannot test this on macOS.

@bungabunga
Copy link

@vectorlit just a correction: the new Ubuntu 17.10 which is based on Gnome actually does support system tray. It comes with a Ubuntu AppIndicators extension preinstalled, that is a customized fork of KStatusNotifierItem/AppIndicator and enables app indicators in the top panel.

@scottnonnenberg
Copy link
Contributor

I'm going to close this for now, since we won't be merging these changes anywhere close to their current form.

@ropimasi
Copy link

ropimasi commented May 5, 2021

Excuseme, hello guys,
This feature (minimize to tray) is still working on Signal Desktop v5.0.0 ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

5 participants