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

Help VPN users #2738

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Help VPN users #2738

wants to merge 2 commits into from

Conversation

hboetes
Copy link
Member

@hboetes hboetes commented Nov 5, 2023

  1. Add (For VPN) so VPN users know they should look into this setting.
  2. Make tooltip easier to understand and translate

1. Add (For VPN) so VPN users know they should look into this setting.
2. Make tooltip easier to understand and translate
@hboetes
Copy link
Member Author

hboetes commented Nov 5, 2023

For after the next release.

@@ -327,11 +327,11 @@
<object class="GtkBox">
<property name="homogeneous">True</property>
<property name="spacing">12</property>
<property name="tooltip-text" translatable="yes">Binds connections to a specific network interface, useful for e.g. ensuring a VPN is used at all times. Leave empty to use any available interface. Only change this value if you know what you are doing.</property>
<property name="tooltip-text" translatable="yes">Makes sure a VPN is used by binding connections to a specific network interface. Usually left empty to use all the interfaces available.</property>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The setting is not only unique to VPN applications, there are other purposes for it.

pynicotine/gtkgui/ui/settings/network.ui Outdated Show resolved Hide resolved
Discard unnecessary hunk.
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.

None yet

2 participants