Load confirmations only once form is shown

DoubleBullet · Jun 28, 2023 · 4d0603a · 4d0603a
1 parent 6ce732e
commit 4d0603a
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 15 deletions.
diff --git a/Steam Desktop Authenticator/ConfirmationFormWeb.Designer.cs b/Steam Desktop Authenticator/ConfirmationFormWeb.Designer.cs
diff --git a/Steam Desktop Authenticator/ConfirmationFormWeb.cs b/Steam Desktop Authenticator/ConfirmationFormWeb.cs
@@ -16,8 +16,6 @@ public ConfirmationFormWeb(SteamGuardAccount steamAccount)
             InitializeComponent();
             this.steamAccount = steamAccount;
             this.Text = String.Format("Trade Confirmations - {0}", steamAccount.AccountName);
-
-            LoadData();
         }
         private async Task LoadData()
         {
@@ -140,5 +138,16 @@ private async void btnRefresh_Click(object sender, EventArgs e)
             this.btnRefresh.Enabled = true;
             this.btnRefresh.Text = "Refresh";
         }
+
+        private async void ConfirmationFormWeb_Shown(object sender, EventArgs e)
+        {
+            this.btnRefresh.Enabled = false;
+            this.btnRefresh.Text = "Refreshing...";
+
+            await this.LoadData();
+
+            this.btnRefresh.Enabled = true;
+            this.btnRefresh.Text = "Refresh";
+        }
     }
 }
diff --git a/Steam Desktop Authenticator/MainForm.cs b/Steam Desktop Authenticator/MainForm.cs
@@ -133,19 +133,8 @@ private void btnTradeConfirmations_Click(object sender, EventArgs e)
             btnTradeConfirmations.Text = "Loading...";
             btnTradeConfirmations.Text = oText;
 
-            try
-            {
-                ConfirmationFormWeb confirms = new ConfirmationFormWeb(currentAccount);
-                confirms.Show();
-            }
-            catch (Exception)
-            {
-                DialogResult res = MessageBox.Show("You are missing a dependency required to view your trade confirmations.\nWould you like to install it now?", "Trade confirmations failed to open", MessageBoxButtons.YesNo);
-                if (res == DialogResult.Yes)
-                {
-                    new InstallRedistribForm(true).ShowDialog();
-                }
-            }
+            ConfirmationFormWeb confirms = new ConfirmationFormWeb(currentAccount);
+            confirms.Show();
         }
 
         private void btnManageEncryption_Click(object sender, EventArgs e)