Merge pull request #15 from online-VCDS/sb/refactor-btns

Sb/refactor btns

Author: Stephanos

AutoScanCompare/CodingCompare.Designer.cs

@@ -29,7 +29,7 @@ public CodingCompare(TreeNode node, string parentText, string coding1 = "", stri
             this.Text = parentTitle;
 
 			label15.Text = WinFormStrings.str_lblCodeForum;
-			button1.Text = WinFormStrings.str_copyClipboard;
+			CopyToClipboardBtn.Text = WinFormStrings.str_copyClipboard;
 
             if(node == null)
             {
@@ -196,9 +196,14 @@ private void tbClick(object sender, EventArgs e)
 
         }
 
-		private void button1_Click(object sender, EventArgs e)
+		private void CopyToClipboardBtn_Click(object sender, EventArgs e)
 		{
 			Clipboard.SetText(richTextBox1.Text);
-		}
-	}
+		}
+
+        private void CodingCompare_Load(object sender, EventArgs e)
+        {
+
+        }
+    }
 }

AutoScanCompare/CodingCompare.cs

@@ -49,8 +49,8 @@ public Form1()
 			label4.Text = WinFormStrings.str_CodingQuickCompare;
 			label5.Text = WinFormStrings.str_coding1;
 			label6.Text = WinFormStrings.str_coding2;
-			button2.Text = WinFormStrings.str_btnCompare;
-			button3.Text = WinFormStrings.str_contextMenue;
+			OpenCodingCompareBtn.Text = WinFormStrings.str_btnCompare;
+			EnableContexMenuBtn.Text = WinFormStrings.str_contextMenue;
 
 
 			this.BackColor = ColorTranslator.FromHtml("#CDE1E2");
@@ -125,14 +125,22 @@ private void richTextBox2_TextChanged(object sender, EventArgs e)
             }
         }
 
-        private void button1_Click(object sender, EventArgs e)
+        private void RunAutoscanCompareBtn_Click(object sender, EventArgs e)
         {
             sourceSTGs.Clear();
             destSTGs.Clear();
             mergedSTGs.Clear();
-            treeView1.Nodes.Clear();
+            treeView1.Nodes.Clear();
+
+
+
+            if (richTextBox1.Text == "" || richTextBox2.Text == "")
+            {
+                MessageBox.Show(WinFormStrings.str_autoscanNotSelected, WinFormStrings.str_Error, MessageBoxButtons.OK, MessageBoxIcon.Error);
+                return;
+            }
 
-            if(richTextBox1.BackColor != Color.LightGreen || richTextBox2.BackColor != Color.LightGreen || richTextBox1.Text == "" || richTextBox2.Text == "")
+            if (richTextBox1.BackColor != Color.LightGreen || richTextBox2.BackColor != Color.LightGreen)
             {
                 MessageBox.Show(WinFormStrings.str_autoscanNotValid, WinFormStrings.str_Error, MessageBoxButtons.OK, MessageBoxIcon.Error);
                 return;
@@ -336,7 +344,7 @@ private void treeView1_NodeMouseDoubleClick(object sender, TreeNodeMouseClickEve
             }
         }
 
-        private void button2_Click(object sender, EventArgs e)
+        private void OpenCodingCompareBtn_Click(object sender, EventArgs e)
         {
             if(Regex.IsMatch(textBox1.Text, "^[0-9a-fA-F]+$") && Regex.IsMatch(textBox2.Text, "^[0-9a-fA-F]+$") && textBox1.Text.Length == textBox2.Text.Length)
             {
@@ -426,7 +434,7 @@ protected bool GetFilename(out string filename, DragEventArgs e)
             return ret;
         }
 
-        private void button3_Click(object sender, EventArgs e)
+        private void EnableContexMenuBtn_Click(object sender, EventArgs e)
         {
             Helper.RegisterContextMenu();
         }