← Back to team overview

deja-dup-team team mailing list archive

[Merge] lp:~andrewfister/deja-dup/close-button-fix into lp:deja-dup

 

Andrew Fister has proposed merging lp:~andrewfister/deja-dup/close-button-fix into lp:deja-dup.

Requested reviews:
  Déjà Dup Maintainers (deja-dup-team)
Related bugs:
  #581232 closing main window should not stop backup
  https://bugs.launchpad.net/bugs/581232


Instead of stopping the backup and closing when the window manager close button is pressed, Deja-Dup should "minimize to tray", or in other words just hide the window. This is the functionality implemented in this branch.
-- 
https://code.launchpad.net/~andrewfister/deja-dup/close-button-fix/+merge/29686
Your team Déjà Dup Maintainers is requested to review the proposed merge of lp:~andrewfister/deja-dup/close-button-fix into lp:deja-dup.
=== modified file 'AUTHORS'
--- AUTHORS	2010-05-29 17:44:34 +0000
+++ AUTHORS	2010-07-12 10:46:40 +0000
@@ -5,7 +5,7 @@
 
 Files: *
 Copyright: 2008–2010 Michael Terry <mike@xxxxxxxxxxx>
-           2009 Andrew Fister <temposs@xxxxxxxxx>
+           2009,2010 Andrew Fister <temposs@xxxxxxxxx>
            2010 Michael Vogt <michael.vogt@xxxxxxxxxx>
 License: GPL-3+
 

=== modified file 'deja-dup/Assistant.vala'
--- deja-dup/Assistant.vala	2010-06-14 01:06:33 +0000
+++ deja-dup/Assistant.vala	2010-07-12 10:46:40 +0000
@@ -2,6 +2,7 @@
 /*
     This file is part of Déjà Dup.
     © 2009 Michael Terry <mike@xxxxxxxxxxx>
+    © 2010 Andrew Fister <temposs@xxxxxxxxx>
 
     Déjà Dup is free software: you can redistribute it and/or modify
     it under the terms of the GNU General Public License as published by
@@ -128,6 +129,7 @@
   void handle_response(int resp)
   {
     switch (resp) {
+    case Gtk.ResponseType.DELETE_EVENT: break;
     case BACK: go_back(); break;
     case APPLY:
     case FORWARD: go_forward(); break;

=== modified file 'deja-dup/AssistantOperation.vala'
--- deja-dup/AssistantOperation.vala	2010-06-14 01:06:33 +0000
+++ deja-dup/AssistantOperation.vala	2010-07-12 10:46:40 +0000
@@ -2,6 +2,7 @@
 /*
     This file is part of Déjà Dup.
     © 2008,2009 Michael Terry <mike@xxxxxxxxxxx>
+    © 2010 Andrew Fister <temposs@xxxxxxxxx>
 
     Déjà Dup is free software: you can redistribute it and/or modify
     it under the terms of the GNU General Public License as published by
@@ -78,6 +79,7 @@
     canceled.connect(do_cancel);
     closed.connect(do_close);
     prepare.connect(do_prepare);
+    delete_event.connect(do_minimize_to_tray);
   }
   
   protected abstract Gtk.Widget? make_confirm_page();
@@ -472,6 +474,16 @@
     else
       do_close();
   }
+
+  bool do_minimize_to_tray(Gdk.Event event)
+  {
+    if (op != null)
+      hide_on_delete(); // minimize to tray when operation is in progress
+    else
+      do_close(); // otherwise, do the normal close operation
+
+    return true;
+  }
   
   protected virtual void do_close()
   {


Follow ups