Compacted logic that toggles control visibility.

Signed-off-by: Konstantina Chremmou <[email protected]>
xenserver · Nov 1, 2023 · b9c5678 · b9c5678
1 parent df69ec3
commit b9c5678
Showing 1 changed file with 7 additions and 26 deletions.
diff --git a/XenAdmin/Controls/MultipleDvdIsoList.cs b/XenAdmin/Controls/MultipleDvdIsoList.cs
@@ -169,33 +169,14 @@ private void RefreshDrives()
                 }
             }
 
-            if (comboBoxDrive.Items.Count == 0)
-            {
-                comboBoxDrive.Visible = false;
-                cdChanger1.Visible = false;
-                labelSingleDvd.Visible = false;
-                linkLabelEject.Visible = false;
-                newCDLabel.Visible = VM != null && !VM.is_control_domain;
-
-            }
-            else if (comboBoxDrive.Items.Count == 1)
-            {
-                comboBoxDrive.Visible = false;
-                cdChanger1.Visible = true;
+            labelSingleDvd.Visible = comboBoxDrive.Items.Count == 1;
+            if (labelSingleDvd.Visible)
                 labelSingleDvd.Text = comboBoxDrive.Items[0].ToString();
-                labelSingleDvd.Visible = true;
-                tableLayoutPanel1.ColumnStyles[0].Width = labelSingleDvd.Width;
-                newCDLabel.Visible = false;
-                linkLabelEject.Visible = true;
-            }
-            else
-            {
-                comboBoxDrive.Visible = true;
-                cdChanger1.Visible = true;
-                labelSingleDvd.Visible = false;
-                newCDLabel.Visible = false;
-                linkLabelEject.Visible = true;
-            }
+
+            comboBoxDrive.Visible = comboBoxDrive.Items.Count > 1;
+            cdChanger1.Visible = comboBoxDrive.Items.Count > 0;
+            linkLabelEject.Visible = comboBoxDrive.Items.Count > 0;
+            newCDLabel.Visible = comboBoxDrive.Items.Count == 0 && VM != null && !VM.is_control_domain;
 
             _inRefresh = false;