Do not exit app when responding to identity request, just hide.

Temporary fix for issue where identity selector exits without
sending identity.

diff --git a/src/moonshot-identity-management-view.vala b/src/moonshot-identity-management-view.vala
index 9fe081f..40114f3 100644 (file)
--- a/src/moonshot-identity-management-view.vala
+++ b/src/moonshot-identity-management-view.vala
@@ -471,8 +471,13 @@ public class IdentityManagerView : Window {
         if (this.request_queue.is_empty())
         {
             candidates = null;
-            if (!parent_app.explicitly_launched)
-                Gtk.main_quit ();
+            if (!parent_app.explicitly_launched) {
+// The following occasionally causes the app to exit without sending the dbus
+// reply, so for now we just don't exit
+//                Gtk.main_quit ();
+// just hide instead
+                this.hide();
+            }
         } else {
             candidates = this.request_queue.peek_head().candidates;
         }

diff --git a/src/moonshot-identity-manager-app.vala b/src/moonshot-identity-manager-app.vala
index cff2824..7b4389d 100644 (file)
--- a/src/moonshot-identity-manager-app.vala
+++ b/src/moonshot-identity-manager-app.vala
@@ -183,9 +183,11 @@ public class IdentityManagerApp {
         // callback because we may be being called from a 'yield')
         Idle.add(
             () => {
-                request.return_identity (identity); 
-                if (!explicitly_launched)
-                    Idle.add( () => { Gtk.main_quit(); return false; } );
+                request.return_identity (identity);
+// The following occasionally causes the app to exit without sending the dbus
+// reply, so for now we just don't exit
+//                if (!explicitly_launched)
+//                    Idle.add( () => { Gtk.main_quit(); return false; } );
                 return false;
             }
         );