| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
| |
|
| |
|
| |
|
|\
| |
| |
| | |
https://gitlab.gnome.org/GNOME/network-manager-applet/merge_requests/12
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Now that GtkApplication makes good decisions about when to quit (when no
windows are present), the magic that deferred creation of vpn type
selector is useless. Worse even, it makes GtkApplication believe that
we're done and shuts it down before an editor window is added to it.
https://gitlab.gnome.org/GNOME/network-manager-applet/issues/9
Reported-by: Jan Tojnar
|
|/
|
|
|
|
|
| |
It's solely used for parsing --keep-above and doesn't know anything
about actual options and thus prints irrelevant help. Skip that and let
the GtkApplication::command-line signal handler parse it and print help
instead.
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
|
| |
The current AppStream specification prefers /usr/share/metainfo eveyone
who relies on this (that's distros, not the end users) already supports
the new location.
https://gitlab.gnome.org/GNOME/network-manager-applet/issues/2
|
| |
|
|
|
|
| |
https://bugzilla.redhat.com/show_bug.cgi?id=1390638
|
|\
| |
| |
| | |
https://gitlab.gnome.org/GNOME/network-manager-applet/merge_requests/5
|
| |
| |
| |
| |
| |
| | |
The other options should be ignored when we look for --keep-above.
g_option_context_parse() removes it from argv[], thus we can also
revert 88e4dcab27e6.
|
| |
| |
| |
| | |
It's of no use anymore now that GtkApplication decides when to quit.
|
| |
| |
| |
| |
| |
| | |
Tracking all windows as application windows makes it possible for the
GtkApplication to terminate at the correct point -- when no windows are
present.
|
| |
| |
| |
| |
| | |
This indicates that a new editor window has been opened, for the
purposes of tracking the windows as GtkApplication windows.
|
| | |
|
| |
| |
| |
| | |
They're offensive.
|
| |
| |
| |
| | |
Brings joy to Thomas while grepping through code.
|
| |
| |
| |
| |
| |
| | |
This communicates that the page has spawned a new editor (e.g. for editing
slave connections). This will allow the GtkApplication to track the editor
windows.
|
| |
| |
| |
| | |
This is not a public API and we aren't using them anywhere.
|
| |
| |
| |
| |
| | |
Doing this for consistency with code where Thomas insists this is a good
idea. Easier than fixing Thomas.
|
| |
| |
| |
| |
| |
| | |
The error argument is never used, don't pretend we handle it and avoid
leaking it. Just dumping the error on the console is still not optimal,
but unconfusing things is a good start.
|
| |
| |
| |
| |
| | |
We use a signal of the same name, and an extra property just messes up
things.
|
| |
| |
| |
| | |
Fixes: 137ec0477ec30345d5f4762cc9329297e7da43f4
|
| |
| |
| |
| | |
None of them actially used.
|
| |
| |
| |
| | |
It's not used anywhere.
|
|/ |
|
|\
| |
| |
| | |
https://gitlab.gnome.org/GNOME/network-manager-applet/merge_requests/6
|
|/
|
|
| |
The entry never contains a value in such cases. Don't fail the validation.
|
|\
| |
| |
| | |
https://gitlab.gnome.org/GNOME/network-manager-applet/merge_requests/9
|
| |
| |
| |
| |
| |
| |
| | |
This is essentially an usability thing that allows system-owned secrets to
be used even when GNOME keyring is not installed.
https://gitlab.gnome.org/GNOME/network-manager-applet/issues/3
|
|/
|
|
|
| |
If getting old secrets fails for whichever reason, get_existing_secrets_cb()
is called with error set and NULL settings (by keyring_find_secrets_cb()).
|
|
|
|
| |
https://gitlab.gnome.org/GNOME/network-manager-applet/merge_requests/4
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Handling of nm-connection-editor command line arguments was changed in
f3782da2. Since that commit, most args are handled by `editor_command_line`,
but one (`--keep-above`) is handled by `main`. Unfortunately, if
`--keep-above` appears with other arguments, `main` does not strip it from
argv and parsing in `editor_command_line` fails with "Failed to parse
options".
To handle this, let's just have `editor_command_line` recognize but ignore
`--keep-above` if it encounters it.
https://gitlab.gnome.org/GNOME/network-manager-applet/issues/1
Signed-off-by: Adam Williamson <awilliam@redhat.com>
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
| |
"with_appindicator" is the global variable set when parsing command
line arguments. It later results in the initialization of applet's
app-indicator instance. We always should check for INDICATOR_ENABLED(),
not "with_appindicator".
|
|\
| |
| |
| | |
https://bugzilla.gnome.org/show_bug.cgi?id=796258
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
We must use premade icons for the indicator VPN case.
The icons are copies of the original with the padlock pre-composited, because
indicators can't use pixbufs; we use the suffix -secure.
This change got introduced with Ubuntu 16.04. Several icon themes have
upstreamed those -secure icons meanwhile (Numix, MATE Faenza,
elementaryXubuntu-dark from murrine-themes).
This patch requires additional icons in nm-applet:
nm-signal-<xx>.png -> nm-signal-<xx>-secure.png
nm-device-wired.png -> nm-device-wired-secure.png
Patch author: Mathieu Trudel-Lapierre <mathieu@canonical.com>
https://bugzilla.gnome.org/show_bug.cgi?id=796258
|
|/
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Icons were created for Ubuntu and have been obtained from the
network-manager-applet src:package in Ubuntu. The package maintainer
applied no specific copyrights to those files. In debian/copyright of
the Ubuntu package they are covered by this copyright info block:
Files: *
Copyright: 2004 - 2015 Red Hat, Inc.
2005 - 2008 Novell, Inc.
2001, 2002 Free Software Foundation
License: GPL-2+
https://bugzilla.gnome.org/show_bug.cgi?id=796258
|
|
|
|
|
|
|
|
| |
https://bugs.archlinux.org/task/58767
https://gitlab.gnome.org/GNOME/network-manager-applet/merge_requests/2
Fixes: 15ac92e6a42317e86ffb6d52a9ce5cacbf9e8c87
|
|\
| |
| |
| | |
https://gitlab.gnome.org/GNOME/network-manager-applet/merge_requests/1
|