From c00342df6f6fae83bfedd406d6008913ba7a2da8 Mon Sep 17 00:00:00 2001 From: David Sparer Date: Tue, 24 May 2016 16:53:47 -0600 Subject: [PATCH] Refactoring: Extracted a method in frmOptions that sets the initially selected page --- mRemoteV1/UI/Forms/frmOptions.cs | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/mRemoteV1/UI/Forms/frmOptions.cs b/mRemoteV1/UI/Forms/frmOptions.cs index 7c5144521..2eacb53eb 100644 --- a/mRemoteV1/UI/Forms/frmOptions.cs +++ b/mRemoteV1/UI/Forms/frmOptions.cs @@ -26,6 +26,7 @@ namespace mRemoteNG.UI.Forms Runtime.FontOverride(this); SetImageListForListView(); AddOptionsPagesToListView(); + SetInitiallyActivatedPage(); } private void AddOptionsPagesToListView() @@ -38,11 +39,6 @@ namespace mRemoteNG.UI.Forms item.Tag = page.GetType().Name; lstOptionPages.Items.Add(item); } - - // First one to be selected - pnlMain.Controls.Add(_pages.FirstOrDefault().Value); - pnlMain.Focus(); - lstOptionPages.Items[0].Selected = true; } private void SetImageListForListView() @@ -65,6 +61,11 @@ namespace mRemoteNG.UI.Forms _pages.Add(typeof(AdvancedPage).Name, new AdvancedPage()); } + private void SetInitiallyActivatedPage() + { + lstOptionPages.Items[0].Selected = true; + } + private void btnOK_Click(object sender, EventArgs e) { foreach (var page in _pages.Values)