From fa5f7adad85d129ee3d067e031eb1d02e9c28816 Mon Sep 17 00:00:00 2001
From: wzxjohn <wzxjohn@users.noreply.github.com>
Date: Sun, 9 Nov 2014 06:06:26 +0800
Subject: [PATCH] small bug

Fix a small bug
---
 shadowsocks-csharp/View/ConfigForm.cs | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/shadowsocks-csharp/View/ConfigForm.cs b/shadowsocks-csharp/View/ConfigForm.cs
index 894b7f27..eb2a3d8f 100755
--- a/shadowsocks-csharp/View/ConfigForm.cs
+++ b/shadowsocks-csharp/View/ConfigForm.cs
@@ -155,9 +155,9 @@ namespace Shadowsocks.View
             ServersListBox.SelectedIndex = _modifiedConfiguration.index;
             LoadSelectedServer();
 
-            UpdateServersMenu();
+            UpdateServersMenu();
             enableItem.Checked = _modifiedConfiguration.enabled;
-            ShareOverLANItem.Checked = _modifiedConfiguration.shareOverLan;
+            ShareOverLANItem.Checked = _modifiedConfiguration.shareOverLan;
         }
 
         private void UpdateServersMenu()
@@ -279,8 +279,8 @@ namespace Shadowsocks.View
             {
                 MessageBox.Show("Please add at least one server");
                 return;
-            }
-            controller.SaveServers(_modifiedConfiguration.configs);
+            }
+            controller.SaveServers(_modifiedConfiguration.configs);
             this.Hide();
             ShowFirstTimeBalloon();
         }
@@ -352,7 +352,10 @@ namespace Shadowsocks.View
 
         private void enableLogBox_CheckedChanged(object sender, EventArgs e)
         {
-            MessageBox.Show("This option only works on next startup.");
+            if (this.Visible)
+            {
+                MessageBox.Show("This option only works on next startup.");
+            }
         }
     }
 }