diff options
author | Mario Sanchez Prada <mario@endlessm.com> | 2018-05-03 19:08:10 +0100 |
---|---|---|
committer | Mario Sanchez Prada <mario@endlessm.com> | 2018-05-03 19:08:10 +0100 |
commit | df8eccc2649098f0fc23310d9e1b33e9c3daad10 (patch) | |
tree | 133847b5e419e4a1545be2b50c885c2340fb008f | |
parent | af0186aaf99cb6add6b5a3c85828e8a4d6e49fd5 (diff) | |
download | gnome-control-center-startup-notify.tar.gz |
Don't declare StartupNotify=true in panel-specific desktop filesstartup-notify
As per the Startup Notification Spec, when an application declares this
in its desktop file, it the launcher (e.g. the shell) will initiate the
startup process, expecting the launchee to finish it with a "remove" X
message once it's ready, unless there's a failure launching it.
The problem here is that this panel-specific desktop files declaring that
key confuses the shell, since it will make it initiate the startup process
but then no "remove" message is ever received if g-c-c was already running,
since the newly launched process will realize it's not the main instance and
will exit gracefully after handing over control of the situation to the
original process, with exit code 0, leaving the startup process unfinished.
Due to this, the shell will keep waiting for a supposedly new application
(as per the the panel-specific desktop file used to launch) to finish, which
will result in the cursor to keep spinning after the app is discarded by the
shell on a timeout.
We should fix this in a more general way (see [2]), since apps exiting due
to handing control over to a previous instance should still send the "remove"
message, but in the meantime let's make our user's lives better by not
declaring this key in any of the panel-specific desktop files.
[1] https://www.freedesktop.org/wiki/Specifications/startup-notification-spec
[2] https://bugzilla.gnome.org/show_bug.cgi?id=795779
25 files changed, 0 insertions, 25 deletions
diff --git a/panels/background/gnome-background-panel.desktop.in.in b/panels/background/gnome-background-panel.desktop.in.in index 76534b028..08b38f62c 100644 --- a/panels/background/gnome-background-panel.desktop.in.in +++ b/panels/background/gnome-background-panel.desktop.in.in @@ -7,7 +7,6 @@ Icon=preferences-desktop-wallpaper Terminal=false Type=Application NoDisplay=true -StartupNotify=true Categories=GNOME;GTK;Settings;DesktopSettings;X-GNOME-Settings-Panel;X-GNOME-PersonalizationSettings; OnlyShowIn=GNOME; # Translators: Search terms to find the Background panel. Do NOT translate or localize the semicolons! The list MUST also end with a semicolon! diff --git a/panels/bluetooth/gnome-bluetooth-panel.desktop.in.in b/panels/bluetooth/gnome-bluetooth-panel.desktop.in.in index d460700d0..0fdadd451 100644 --- a/panels/bluetooth/gnome-bluetooth-panel.desktop.in.in +++ b/panels/bluetooth/gnome-bluetooth-panel.desktop.in.in @@ -9,7 +9,6 @@ Type=Application NoDisplay=true Categories=GTK;GNOME;Settings;X-GNOME-NetworkSettings;HardwareSettings;X-GNOME-Settings-Panel;X-GNOME-ConnectivitySettings; OnlyShowIn=GNOME;Unity; -StartupNotify=true X-GNOME-Bugzilla-Bugzilla=GNOME X-GNOME-Bugzilla-Product=gnome-bluetooth X-GNOME-Bugzilla-Component=properties diff --git a/panels/color/gnome-color-panel.desktop.in.in b/panels/color/gnome-color-panel.desktop.in.in index 30656828a..70e92b8a4 100644 --- a/panels/color/gnome-color-panel.desktop.in.in +++ b/panels/color/gnome-color-panel.desktop.in.in @@ -7,7 +7,6 @@ Icon=preferences-color Terminal=false Type=Application NoDisplay=true -StartupNotify=true Categories=GNOME;GTK;Settings;X-GNOME-Settings-Panel;HardwareSettings;X-GNOME-DevicesSettings; OnlyShowIn=GNOME;Unity; X-GNOME-Bugzilla-Bugzilla=GNOME diff --git a/panels/datetime/gnome-datetime-panel.desktop.in.in b/panels/datetime/gnome-datetime-panel.desktop.in.in index f7242f1ae..7943d873a 100644 --- a/panels/datetime/gnome-datetime-panel.desktop.in.in +++ b/panels/datetime/gnome-datetime-panel.desktop.in.in @@ -7,7 +7,6 @@ Icon=preferences-system-time Terminal=false Type=Application NoDisplay=true -StartupNotify=true Categories=GNOME;GTK;Settings;X-GNOME-Settings-Panel;X-GNOME-DetailsSettings; OnlyShowIn=GNOME; # Translators: Search terms to find the Date and Time panel. Do NOT translate or localize the semicolons! The list MUST also end with a semicolon! diff --git a/panels/display/gnome-display-panel.desktop.in.in b/panels/display/gnome-display-panel.desktop.in.in index 9c4744d9d..061ffc4c6 100644 --- a/panels/display/gnome-display-panel.desktop.in.in +++ b/panels/display/gnome-display-panel.desktop.in.in @@ -7,7 +7,6 @@ Icon=preferences-desktop-display Terminal=false Type=Application NoDisplay=true -StartupNotify=true Categories=GNOME;GTK;Settings;HardwareSettings;X-GNOME-Settings-Panel;X-GNOME-DevicesSettings; OnlyShowIn=GNOME;Unity; X-GNOME-Bugzilla-Bugzilla=GNOME diff --git a/panels/info/gnome-default-apps-panel.desktop.in.in b/panels/info/gnome-default-apps-panel.desktop.in.in index 328d6a611..b032d985b 100644 --- a/panels/info/gnome-default-apps-panel.desktop.in.in +++ b/panels/info/gnome-default-apps-panel.desktop.in.in @@ -7,7 +7,6 @@ Icon=starred Terminal=false Type=Application NoDisplay=true -StartupNotify=true Categories=GNOME;GTK;Settings;X-GNOME-Settings-Panel;X-GNOME-DetailsSettings; OnlyShowIn=GNOME;Unity; X-GNOME-Bugzilla-Bugzilla=GNOME diff --git a/panels/info/gnome-info-overview-panel.desktop.in.in b/panels/info/gnome-info-overview-panel.desktop.in.in index 8b939b6e3..3e3e92e75 100644 --- a/panels/info/gnome-info-overview-panel.desktop.in.in +++ b/panels/info/gnome-info-overview-panel.desktop.in.in @@ -7,7 +7,6 @@ Icon=help-about Terminal=false Type=Application NoDisplay=true -StartupNotify=true Categories=GNOME;GTK;Settings;X-GNOME-Settings-Panel;X-GNOME-DetailsSettings; OnlyShowIn=GNOME;Unity; X-GNOME-Bugzilla-Bugzilla=GNOME diff --git a/panels/info/gnome-removable-media-panel.desktop.in.in b/panels/info/gnome-removable-media-panel.desktop.in.in index 91fdc407e..61142ec58 100644 --- a/panels/info/gnome-removable-media-panel.desktop.in.in +++ b/panels/info/gnome-removable-media-panel.desktop.in.in @@ -7,7 +7,6 @@ Icon=media-removable Terminal=false Type=Application NoDisplay=true -StartupNotify=true Categories=GNOME;GTK;Settings;X-GNOME-Settings-Panel;X-GNOME-DevicesSettings; OnlyShowIn=GNOME;Unity; X-GNOME-Bugzilla-Bugzilla=GNOME diff --git a/panels/keyboard/gnome-keyboard-panel.desktop.in.in b/panels/keyboard/gnome-keyboard-panel.desktop.in.in index b7b562716..25ea3f327 100644 --- a/panels/keyboard/gnome-keyboard-panel.desktop.in.in +++ b/panels/keyboard/gnome-keyboard-panel.desktop.in.in @@ -7,7 +7,6 @@ Icon=input-keyboard Terminal=false Type=Application NoDisplay=true -StartupNotify=true Categories=GNOME;GTK;Settings;HardwareSettings;X-GNOME-Settings-Panel;X-GNOME-DevicesSettings; OnlyShowIn=GNOME;Unity; X-GNOME-Bugzilla-Bugzilla=GNOME diff --git a/panels/mouse/gnome-mouse-panel.desktop.in.in b/panels/mouse/gnome-mouse-panel.desktop.in.in index 53d23ecdf..b3ccbf6bb 100644 --- a/panels/mouse/gnome-mouse-panel.desktop.in.in +++ b/panels/mouse/gnome-mouse-panel.desktop.in.in @@ -7,7 +7,6 @@ Icon=input-mouse Terminal=false Type=Application NoDisplay=true -StartupNotify=true Categories=GNOME;GTK;Settings;HardwareSettings;X-GNOME-Settings-Panel;X-GNOME-DevicesSettings; OnlyShowIn=GNOME;Unity; X-GNOME-Bugzilla-Bugzilla=GNOME diff --git a/panels/network/gnome-network-panel.desktop.in.in b/panels/network/gnome-network-panel.desktop.in.in index 5f6734f80..fc07f15a9 100644 --- a/panels/network/gnome-network-panel.desktop.in.in +++ b/panels/network/gnome-network-panel.desktop.in.in @@ -7,7 +7,6 @@ Icon=network-workgroup Terminal=false Type=Application NoDisplay=true -StartupNotify=true Categories=GNOME;GTK;Settings;HardwareSettings;X-GNOME-Settings-Panel; OnlyShowIn=GNOME;Unity; X-GNOME-Bugzilla-Bugzilla=GNOME diff --git a/panels/network/gnome-wifi-panel.desktop.in.in b/panels/network/gnome-wifi-panel.desktop.in.in index eca03c35e..ba9936c48 100644 --- a/panels/network/gnome-wifi-panel.desktop.in.in +++ b/panels/network/gnome-wifi-panel.desktop.in.in @@ -7,7 +7,6 @@ Icon=network-wireless Terminal=false Type=Application NoDisplay=true -StartupNotify=true Categories=GNOME;GTK;Settings;HardwareSettings;X-GNOME-Settings-Panel;X-GNOME-ConnectivitySettings; OnlyShowIn=GNOME;Unity; X-GNOME-Bugzilla-Bugzilla=GNOME diff --git a/panels/notifications/gnome-notifications-panel.desktop.in.in b/panels/notifications/gnome-notifications-panel.desktop.in.in index 4b27b3fc3..cf7224d2f 100644 --- a/panels/notifications/gnome-notifications-panel.desktop.in.in +++ b/panels/notifications/gnome-notifications-panel.desktop.in.in @@ -7,7 +7,6 @@ Icon=preferences-system-notifications Terminal=false Type=Application NoDisplay=true -StartupNotify=true Categories=GNOME;GTK;Settings;DesktopSettings;X-GNOME-Settings-Panel;X-GNOME-PersonalizationSettings; OnlyShowIn=GNOME;Unity; X-GNOME-Bugzilla-Bugzilla=GNOME diff --git a/panels/online-accounts/gnome-online-accounts-panel.desktop.in.in b/panels/online-accounts/gnome-online-accounts-panel.desktop.in.in index f30aff524..13086907b 100644 --- a/panels/online-accounts/gnome-online-accounts-panel.desktop.in.in +++ b/panels/online-accounts/gnome-online-accounts-panel.desktop.in.in @@ -7,7 +7,6 @@ Icon=goa-panel Terminal=false Type=Application NoDisplay=true -StartupNotify=true Categories=GNOME;GTK;Settings;DesktopSettings;X-GNOME-Settings-Panel;X-GNOME-AccountSettings; OnlyShowIn=GNOME;Unity; X-GNOME-Bugzilla-Bugzilla=GNOME diff --git a/panels/power/gnome-power-panel.desktop.in.in b/panels/power/gnome-power-panel.desktop.in.in index 59ff8ab94..fe0d2984d 100644 --- a/panels/power/gnome-power-panel.desktop.in.in +++ b/panels/power/gnome-power-panel.desktop.in.in @@ -7,7 +7,6 @@ Icon=gnome-power-manager Terminal=false Type=Application NoDisplay=true -StartupNotify=true Categories=GNOME;GTK;Settings;DesktopSettings;X-GNOME-Settings-Panel;HardwareSettings; OnlyShowIn=GNOME;Unity; X-GNOME-Bugzilla-Bugzilla=GNOME diff --git a/panels/printers/gnome-printers-panel.desktop.in.in b/panels/printers/gnome-printers-panel.desktop.in.in index ac93dcae9..a543aafa3 100644 --- a/panels/printers/gnome-printers-panel.desktop.in.in +++ b/panels/printers/gnome-printers-panel.desktop.in.in @@ -7,7 +7,6 @@ Icon=printer Terminal=false Type=Application NoDisplay=true -StartupNotify=true # The X-GNOME-Settings-Panel is necessary to show in the main shell UI Categories=GNOME;GTK;Settings;HardwareSettings;X-GNOME-Settings-Panel;X-GNOME-DevicesSettings; OnlyShowIn=GNOME;Unity; diff --git a/panels/privacy/gnome-privacy-panel.desktop.in.in b/panels/privacy/gnome-privacy-panel.desktop.in.in index 41ad29ce7..d1aa1f38f 100644 --- a/panels/privacy/gnome-privacy-panel.desktop.in.in +++ b/panels/privacy/gnome-privacy-panel.desktop.in.in @@ -8,7 +8,6 @@ Icon=preferences-system-privacy Terminal=false Type=Application NoDisplay=true -StartupNotify=true Categories=GNOME;GTK;Settings;DesktopSettings;X-GNOME-Settings-Panel;X-GNOME-AccountSettings; OnlyShowIn=GNOME;Unity; X-GNOME-Bugzilla-Bugzilla=GNOME diff --git a/panels/region/gnome-region-panel.desktop.in.in b/panels/region/gnome-region-panel.desktop.in.in index 16c29bc25..628d83734 100644 --- a/panels/region/gnome-region-panel.desktop.in.in +++ b/panels/region/gnome-region-panel.desktop.in.in @@ -7,7 +7,6 @@ Icon=preferences-desktop-locale Terminal=false Type=Application NoDisplay=true -StartupNotify=true Categories=GNOME;GTK;Settings;DesktopSettings;X-GNOME-Settings-Panel;X-GNOME-PersonalizationSettings; OnlyShowIn=GNOME;Unity; X-GNOME-Bugzilla-Bugzilla=GNOME diff --git a/panels/search/gnome-search-panel.desktop.in.in b/panels/search/gnome-search-panel.desktop.in.in index c4ea2d067..b76dd4494 100644 --- a/panels/search/gnome-search-panel.desktop.in.in +++ b/panels/search/gnome-search-panel.desktop.in.in @@ -7,7 +7,6 @@ Icon=preferences-system-search Terminal=false Type=Application NoDisplay=true -StartupNotify=true Categories=GNOME;GTK;Settings;DesktopSettings;X-GNOME-Settings-Panel;X-GNOME-PersonalizationSettings; OnlyShowIn=GNOME;Unity; X-GNOME-Bugzilla-Bugzilla=GNOME diff --git a/panels/sharing/gnome-sharing-panel.desktop.in.in b/panels/sharing/gnome-sharing-panel.desktop.in.in index 32d0b0a09..1b6e20843 100644 --- a/panels/sharing/gnome-sharing-panel.desktop.in.in +++ b/panels/sharing/gnome-sharing-panel.desktop.in.in @@ -7,7 +7,6 @@ Icon=preferences-system-sharing Terminal=false Type=Application NoDisplay=true -StartupNotify=true Categories=GNOME;GTK;Settings;DesktopSettings;X-GNOME-Settings-Panel;X-GNOME-AccountSettings; OnlyShowIn=GNOME;Unity; X-GNOME-Settings-Panel=sharing diff --git a/panels/sound/data/gnome-sound-panel.desktop.in.in b/panels/sound/data/gnome-sound-panel.desktop.in.in index 1d782a171..ced9c0ed7 100644 --- a/panels/sound/data/gnome-sound-panel.desktop.in.in +++ b/panels/sound/data/gnome-sound-panel.desktop.in.in @@ -7,7 +7,6 @@ Icon=multimedia-volume-control Terminal=false Type=Application NoDisplay=true -StartupNotify=true Categories=GNOME;GTK;Settings;HardwareSettings;X-GNOME-Settings-Panel; OnlyShowIn=GNOME;Unity; X-GNOME-Bugzilla-Bugzilla=GNOME diff --git a/panels/thunderbolt/gnome-thunderbolt-panel.desktop.in.in b/panels/thunderbolt/gnome-thunderbolt-panel.desktop.in.in index 12ba0330c..387e08cd5 100644 --- a/panels/thunderbolt/gnome-thunderbolt-panel.desktop.in.in +++ b/panels/thunderbolt/gnome-thunderbolt-panel.desktop.in.in @@ -7,7 +7,6 @@ Icon=thunderbolt Terminal=false Type=Application NoDisplay=true -StartupNotify=true Categories=GNOME;GTK;Settings;X-GNOME-Settings-Panel;HardwareSettings;X-GNOME-DevicesSettings; OnlyShowIn=GNOME;Unity; X-GNOME-Bugzilla-Bugzilla=GNOME diff --git a/panels/universal-access/gnome-universal-access-panel.desktop.in.in b/panels/universal-access/gnome-universal-access-panel.desktop.in.in index 4b1c3ee0a..135e44a1e 100644 --- a/panels/universal-access/gnome-universal-access-panel.desktop.in.in +++ b/panels/universal-access/gnome-universal-access-panel.desktop.in.in @@ -7,7 +7,6 @@ Icon=preferences-desktop-accessibility Terminal=false Type=Application NoDisplay=true -StartupNotify=true Categories=GNOME;GTK;Settings;X-GNOME-Settings-Panel;X-GNOME-PersonalizationSettings; OnlyShowIn=GNOME;Unity; X-GNOME-Bugzilla-Bugzilla=GNOME diff --git a/panels/user-accounts/data/gnome-user-accounts-panel.desktop.in.in b/panels/user-accounts/data/gnome-user-accounts-panel.desktop.in.in index 219921ad5..e8dc26dbe 100644 --- a/panels/user-accounts/data/gnome-user-accounts-panel.desktop.in.in +++ b/panels/user-accounts/data/gnome-user-accounts-panel.desktop.in.in @@ -7,7 +7,6 @@ Icon=system-users Terminal=false Type=Application NoDisplay=true -StartupNotify=true Categories=System;Settings;X-GNOME-Settings-Panel;X-GNOME-DetailsSettings; OnlyShowIn=GNOME;Unity; X-GNOME-Bugzilla-Bugzilla=GNOME diff --git a/panels/wacom/gnome-wacom-panel.desktop.in.in b/panels/wacom/gnome-wacom-panel.desktop.in.in index c8358ee9e..f2a52edb1 100644 --- a/panels/wacom/gnome-wacom-panel.desktop.in.in +++ b/panels/wacom/gnome-wacom-panel.desktop.in.in @@ -7,7 +7,6 @@ Icon=input-tablet Terminal=false Type=Application NoDisplay=true -StartupNotify=true Categories=GNOME;GTK;Settings;HardwareSettings;X-GNOME-Settings-Panel;X-GNOME-DevicesSettings; OnlyShowIn=GNOME;Unity; X-GNOME-Bugzilla-Bugzilla=GNOME |