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

Allow starting in background when tray not available #975

Merged
merged 2 commits into from
Jun 24, 2023

Conversation

WojtekWidomski
Copy link
Contributor

This PR adds --background command line argument that starts GUI in background even when tray not available and uses QLocalSocket and QLocalServer instead of lockfile (now it is possible to open window without starting new instance, using launcher in programs menu).

Please review this code, because I never used Qt.

This will fix issue #940.

use QLocalSocket and QLocalServer instead of lockfile
This allows to open window of running instance
when user is trying to start new instance.
@gustavo-iniguez-goya
Copy link
Collaborator

hey @WojtekWidomski , thank you for looking into this.

What is the purpose of the change, launch the GUI when the systray is not available?

@WojtekWidomski
Copy link
Contributor Author

@gustavo-iniguez-goya I don't want tray icon, but when OpenSnitch is added to startup applications and gnome-shell-extension-appindicator is disabled, OpenSnitch window shows on every login.
After this change it is possible to add opensnitch-ui --background to startup applications.

@gustavo-iniguez-goya gustavo-iniguez-goya merged commit 6d45d9d into evilsocket:master Jun 24, 2023
@gustavo-iniguez-goya
Copy link
Collaborator

Thank you @WojtekWidomski !

I'll rework it a little bit, but it improves what @munix9 solved with the lockfile and at the same time solved the problem of displying the GUI on demand when the systray is not available. Thank you both!

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

Successfully merging this pull request may close these issues.

None yet

2 participants