[gnome-shell/wip/fmuellner/notification-redux+sass: 110/141] notificationDaemon: Do the same for GTK+ notifications
- From: Florian Müllner <fmuellner src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gnome-shell/wip/fmuellner/notification-redux+sass: 110/141] notificationDaemon: Do the same for GTK+ notifications
- Date: Fri, 20 Feb 2015 13:06:12 +0000 (UTC)
commit d2540faf3ae5b49e4310c90986c2200a0f792ee9
Author: Florian Müllner <fmuellner gnome org>
Date: Sat Feb 14 06:01:20 2015 +0100
notificationDaemon: Do the same for GTK+ notifications
There is not good reason why activating a GTK+ notification should
behave fundamentally different from fd.o notifications - we don't
raise the app because we expect it to perform an appropriate action,
but that does not include closing overview or calendar for us ...
https://bugzilla.gnome.org/show_bug.cgi?id=744817
js/ui/notificationDaemon.js | 6 ++++++
1 files changed, 6 insertions(+), 0 deletions(-)
---
diff --git a/js/ui/notificationDaemon.js b/js/ui/notificationDaemon.js
index feb9185..aedda63 100644
--- a/js/ui/notificationDaemon.js
+++ b/js/ui/notificationDaemon.js
@@ -716,11 +716,17 @@ const GtkNotificationDaemonAppSource = new Lang.Class({
activateAction: function(actionId, target) {
let app = this._createApp();
app.ActivateActionRemote(actionId, target ? [target] : [], getPlatformData());
+
+ Main.overview.hide();
+ Main.panel.closeCalendar();
},
open: function() {
let app = this._createApp();
app.ActivateRemote(getPlatformData());
+
+ Main.overview.hide();
+ Main.panel.closeCalendar();
},
addNotification: function(notificationId, notificationParams, showBanner) {
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]