[geary/wip/714104-refine-account-dialog] Don't re-use account editor panes when creating new accounts



commit 117c580fcdf901ae8016079f0e6962dcf25125c6
Author: Michael Gratton <mike vee net>
Date:   Fri Dec 21 11:30:43 2018 +1100

    Don't re-use account editor panes when creating new accounts

 src/client/accounts/accounts-editor-list-pane.vala | 15 +++------------
 1 file changed, 3 insertions(+), 12 deletions(-)
---
diff --git a/src/client/accounts/accounts-editor-list-pane.vala 
b/src/client/accounts/accounts-editor-list-pane.vala
index ae2ad87c..b5e56a5c 100644
--- a/src/client/accounts/accounts-editor-list-pane.vala
+++ b/src/client/accounts/accounts-editor-list-pane.vala
@@ -78,9 +78,6 @@ internal class Accounts.EditorListPane : Gtk.Grid, EditorPane, CommandPane {
     [GtkChild]
     private Gtk.ListBox service_list;
 
-    private Gee.Map<Geary.ServiceProvider,EditorAddPane> add_pane_cache =
-        new Gee.HashMap<Geary.ServiceProvider,EditorAddPane>();
-
     private Gee.Map<Geary.AccountInformation,EditorEditPane> edit_pane_cache =
         new Gee.HashMap<Geary.AccountInformation,EditorEditPane>();
 
@@ -126,18 +123,12 @@ internal class Accounts.EditorListPane : Gtk.Grid, EditorPane, CommandPane {
         this.accounts.account_status_changed.disconnect(on_account_status_changed);
         this.accounts.account_removed.disconnect(on_account_removed);
 
-        this.add_pane_cache.clear();
         this.edit_pane_cache.clear();
         base.destroy();
     }
 
-    internal void show_add_account(Geary.ServiceProvider provider) {
-        EditorAddPane? add_pane = this.add_pane_cache.get(provider);
-        if (add_pane == null) {
-            add_pane = new EditorAddPane(this.editor, provider);
-            this.add_pane_cache.set(provider, add_pane);
-        }
-        this.editor.push(add_pane);
+    internal void show_new_account(Geary.ServiceProvider provider) {
+        this.editor.push(new EditorAddPane(this.editor, provider));
     }
 
     internal void show_existing_account(Geary.AccountInformation account) {
@@ -490,7 +481,7 @@ private class Accounts.AddServiceProviderRow : EditorRow<EditorListPane> {
                 }
 
                 if (add_local) {
-                    pane.show_add_account(this.provider);
+                    pane.show_new_account(this.provider);
                 }
             });
     }


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]