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

Allows protonvpn-app to start minimized #58

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

hsiurebgjk
Copy link

Addresses: https://protonmail.uservoice.com/forums/932836-proton-vpn/suggestions/45290605-start-protonvpn-minimized-in-tray-linux

Adds two command line options to protonvpn-app:

  • --start-minimized
  • --version
protonvpn-app --help
Usage:
  protonvpn-app [OPTION…]

Help Options:
  -h, --help                 Show help options
  --help-all                 Show all help options
  --help-gapplication        Show GApplication options
  --help-gtk                 Show GTK+ Options

Application Options:
  --start-minimized          Start minimized in the system tray
  -v, --version              Display the application's version
  --display=DISPLAY          X display to use

Example output with --version

protonvpn-app --version
4.3.2
echo "Exit status: $?"
Exit status: 0
protonvpn-app -v
4.3.2

To start Proton VPN minimized:

protonvpn-app --start-minimized

--version, -v
--start-minimized
@calexandru2018
Copy link
Contributor

Hey @hsiurebgjk , appreciate you making this MR. We'll take a closer look at your MR and comeback later at it. Thank you 👍🏻

@calexandru2018
Copy link
Contributor

Heu @hsiurebgjk sorry for the delay but just wanted to let you know that we've merged this internally and should be released next week to beta, following the week after the next one it should be released to stable and thus on github. I'll close the ticket once the code is published on github, thanks a lot for your contribution! 👍🏻

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.

2 participants