From 8fdb38b99be9c588148d576c23212af765ff7302 Mon Sep 17 00:00:00 2001 From: fiaxh Date: Tue, 19 Sep 2017 22:41:33 +0200 Subject: Use GNotification instead of libnotify --- main/src/ui/application.vala | 1 - main/src/ui/notifications.vala | 116 ++++++++++++++++++++++--------------- main/src/ui/util/label_hybrid.vala | 20 ++++--- 3 files changed, 80 insertions(+), 57 deletions(-) (limited to 'main/src/ui') diff --git a/main/src/ui/application.vala b/main/src/ui/application.vala index 65bb8743..5ce5e063 100644 --- a/main/src/ui/application.vala +++ b/main/src/ui/application.vala @@ -16,7 +16,6 @@ public class Dino.Ui.Application : Gtk.Application, Dino.Application { public Application() throws Error { Object(application_id: "im.dino", flags: ApplicationFlags.HANDLES_OPEN); init(); - Notify.init("dino"); Environment.set_application_name("Dino"); Window.set_default_icon_name("dino"); diff --git a/main/src/ui/notifications.vala b/main/src/ui/notifications.vala index 470f9d6d..313d0854 100644 --- a/main/src/ui/notifications.vala +++ b/main/src/ui/notifications.vala @@ -11,7 +11,8 @@ public class Notifications : Object { private StreamInteractor stream_interactor; private Gtk.Window window; - private HashMap notifications = new HashMap(Conversation.hash_func, Conversation.equals_func); + private HashMap notifications = new HashMap(Conversation.hash_func, Conversation.equals_func); + private Set? active_notification_ids = null; private enum ClosedReason { // org.freedesktop.Notifications.NotificationClosed EXPIRED = 1, @@ -23,6 +24,51 @@ public class Notifications : Object { public Notifications(StreamInteractor stream_interactor, Gtk.Window window) { this.stream_interactor = stream_interactor; this.window = window; + + stream_interactor.get_module(ChatInteraction.IDENTITY).focused_in.connect(() => { + if (active_notification_ids == null) { + Gee.List conversations = stream_interactor.get_module(ConversationManager.IDENTITY).get_active_conversations(); + foreach (Conversation conversation in conversations) { + GLib.Application.get_default().withdraw_notification(conversation.id.to_string()); + } + active_notification_ids = new HashSet(); + } else { + foreach (string id in active_notification_ids) { + GLib.Application.get_default().withdraw_notification(id); + } + active_notification_ids.clear(); + } + }); + + SimpleAction open_conversation_action = new SimpleAction("open-conversation", VariantType.INT32); + open_conversation_action.activate.connect((variant) => { + Conversation? conversation = stream_interactor.get_module(ConversationManager.IDENTITY).get_conversation_by_id(variant.get_int32()); + if (conversation != null) conversation_selected(conversation); + window.present(); + }); + GLib.Application.get_default().add_action(open_conversation_action); + + SimpleAction accept_subscription_action = new SimpleAction("accept-subscription", VariantType.INT32); + accept_subscription_action.activate.connect((variant) => { + Conversation? conversation = stream_interactor.get_module(ConversationManager.IDENTITY).get_conversation_by_id(variant.get_int32()); + if (conversation == null) return; + stream_interactor.get_module(PresenceManager.IDENTITY).approve_subscription(conversation.account, conversation.counterpart); + if (stream_interactor.get_module(RosterManager.IDENTITY).get_roster_item(conversation.account, conversation.counterpart) == null) { + AddConversation.Chat.AddContactDialog dialog = new AddConversation.Chat.AddContactDialog(stream_interactor); + dialog.jid = conversation.counterpart.bare_jid.to_string(); + dialog.account = conversation.account; + dialog.present(); + } + }); + GLib.Application.get_default().add_action(accept_subscription_action); + + SimpleAction deny_subscription_action = new SimpleAction("deny-subscription", VariantType.INT32); + deny_subscription_action.activate.connect((variant) => { + Conversation? conversation = stream_interactor.get_module(ConversationManager.IDENTITY).get_conversation_by_id(variant.get_int32()); + if (conversation == null) return; + stream_interactor.get_module(PresenceManager.IDENTITY).deny_subscription(conversation.account, conversation.counterpart); + }); + GLib.Application.get_default().add_action(deny_subscription_action); } public void start() { @@ -34,65 +80,33 @@ public class Notifications : Object { if (!should_notify_message(message, conversation)) return; if (!notifications.has_key(conversation)) { - notifications[conversation] = new Notify.Notification("", null, null); - notifications[conversation].set_hint("persistent", true); - notifications[conversation].add_action("default", "Open", () => { - conversation_selected(conversation); -#if GDK3_WITH_X11 - Gdk.X11.Window x11window = window.get_window() as Gdk.X11.Window; - if (x11window != null) { - window.present_with_time(Gdk.X11.get_server_time(x11window)); - } else { - window.present(); - } -#else - window.present(); -#endif - }); + notifications[conversation] = new Notification(""); + notifications[conversation].set_default_action_and_target_value("app.open-conversation", new Variant.int32(conversation.id)); } if (!stream_interactor.get_module(ChatInteraction.IDENTITY).is_active_focus()) { string display_name = Util.get_conversation_display_name(stream_interactor, conversation); string text = message.body; if (stream_interactor.get_module(MucManager.IDENTITY).is_groupchat(conversation.counterpart, conversation.account)) { string muc_occupant = Util.get_display_name(stream_interactor, message.from, conversation.account); - text = @"$muc_occupant $text"; + text = @"$muc_occupant: $text"; } - notifications[conversation].update(display_name, text, null); - notifications[conversation].set_image_from_pixbuf((new AvatarGenerator(40, 40)).draw_conversation(stream_interactor, conversation)); - notifications[conversation].set_timeout(3); - try { - notifications[conversation].show(); - } catch (Error error) { } + notifications[conversation].set_title(display_name); + notifications[conversation].set_body(text); + notifications[conversation].set_icon(get_pixbuf_icon((new AvatarGenerator(40, 40)).draw_conversation(stream_interactor, conversation))); + window.get_application().send_notification(conversation.id.to_string(), notifications[conversation]); + active_notification_ids.add(conversation.id.to_string()); window.urgency_hint = true; } } private void on_received_subscription_request(Jid jid, Account account) { - Notify.Notification notification = new Notify.Notification(_("Subscription request"), jid.bare_jid.to_string(), null); - notification.set_image_from_pixbuf((new AvatarGenerator(40, 40)).draw_jid(stream_interactor, jid, account)); - notification.set_hint("persistent", true); - notification.add_action("accept", _("Accept"), () => { - stream_interactor.get_module(PresenceManager.IDENTITY).approve_subscription(account, jid); - - if (stream_interactor.get_module(RosterManager.IDENTITY).get_roster_item(account, jid) == null) { - AddConversation.Chat.AddContactDialog dialog = new AddConversation.Chat.AddContactDialog(stream_interactor); - dialog.jid = jid.bare_jid.to_string(); - dialog.account = account; - dialog.present(); - } - try { - notification.close(); - } catch (Error error) { } - }); - notification.add_action("deny", _("Deny"), () => { - stream_interactor.get_module(PresenceManager.IDENTITY).deny_subscription(account, jid); - try { - notification.close(); - } catch (Error error) { } - }); - try { - notification.show(); - } catch (Error error) { } + Notification notification = new Notification(_("Subscription request")); + notification.set_body(jid.bare_jid.to_string()); + notification.set_icon(get_pixbuf_icon((new AvatarGenerator(40, 40)).draw_jid(stream_interactor, jid, account))); + Conversation conversation = stream_interactor.get_module(ConversationManager.IDENTITY).create_conversation(jid, account, Conversation.Type.CHAT); + notification.add_button_with_target_value(_("Accept"), "app.accept-subscription", conversation.id); + notification.add_button_with_target_value(_("Deny"), "app.deny-subscription", conversation.id); + window.get_application().send_notification(null, notification); } private bool should_notify_message(Entities.Message message, Conversation conversation) { @@ -102,6 +116,12 @@ public class Notifications : Object { if (notify == Conversation.NotifySetting.HIGHLIGHT && nick != null && !message.body.contains(nick.resourcepart)) return false; return true; } + + private Icon get_pixbuf_icon(Gdk.Pixbuf avatar) { + uint8[] buffer; + avatar.save_to_buffer(out buffer, "png"); + return new BytesIcon(new Bytes(buffer)); + } } } diff --git a/main/src/ui/util/label_hybrid.vala b/main/src/ui/util/label_hybrid.vala index ab765dd1..8ca56316 100644 --- a/main/src/ui/util/label_hybrid.vala +++ b/main/src/ui/util/label_hybrid.vala @@ -34,13 +34,7 @@ public class EntryLabelHybrid : LabelHybrid { get { return entry.text; } set { entry.text = value; - if (visibility) { - label.label = value; - } else { - string filler = ""; - for (int i = 0; i < value.length; i++) filler += entry.get_invisible_char().to_string(); - label.label = filler; - } + set_label_label(value); } } @@ -83,7 +77,7 @@ public class EntryLabelHybrid : LabelHybrid { if (event.keyval == Gdk.Key.Return) { show_label(); } else { - label.label = entry.text; + set_label_label(entry.text); } return false; }); @@ -93,6 +87,16 @@ public class EntryLabelHybrid : LabelHybrid { }); } + private void set_label_label(string value) { + if (visibility) { + label.label = value; + } else { + string filler = ""; + for (int i = 0; i < value.length; i++) filler += entry.get_invisible_char().to_string(); + label.label = filler; + } + } + private void update_label() { text = text; } -- cgit v1.2.3-70-g09d2