- rename UpdateState to PlayButtonState for better clarity
- move variables that are only used in constructor to local scope
- make colors and formBG readonly
// Visual studio does it like this for normal winforms projects, so I just used the same format.
// Visual studio does it like this for normal winforms projects, so I just used the same format.
/// <summary>The tray indicator</summary>
/// <summary>The tray indicator</summary>
TrayIndicatortrayIndicator;
privateTrayIndicatortrayIndicator;
/// <summary>The "Show" Button on the tray indicator</summary>
ButtonMenuItemshowButton;
/// <summary><see cref="List{T}"/> of <see cref="ProfileXML"/>s, used for actually working with profile data.</summary>
/// <summary><see cref="List{T}"/> of <see cref="ProfileXML"/>s, used for actually working with profile data.</summary>
//TODO: this should be moved into AM2RLauncher.Core
//TODO: this should be moved into AM2RLauncher.Core
@ -843,31 +830,24 @@ namespace AM2RLauncher
/// <summary><see cref="List{T}"/> of <see cref="ListItem"/>s so that Eto's annoying <see cref="IListItem"/> interface is appeased. Used for profile name display in DropDowns.</summary>
/// <summary><see cref="List{T}"/> of <see cref="ListItem"/>s so that Eto's annoying <see cref="IListItem"/> interface is appeased. Used for profile name display in DropDowns.</summary>
/// <summary><see cref="List{String}"/> of mirror <see cref="string"/>s, used for actually working with mirrors.</summary>
/// <summary><see cref="List{String}"/> of mirror <see cref="string"/>s, used for actually working with mirrors.</summary>
@ -876,7 +856,7 @@ namespace AM2RLauncher
privateList<ListItem>mirrorDescriptionList;
privateList<ListItem>mirrorDescriptionList;
// UI Elements
// UI Elements
/// <summary>The main control of the <see cref="mainPage"/>, used to draw the <see cref="formBG"/> and hold the main interface.</summary>
/// <summary>The main control of the main page, used to draw the <see cref="formBG"/> and hold the main interface.</summary>
privateDrawabledrawable;
privateDrawabledrawable;
/// <summary>A <see cref="ColorButton"/> that acts as the main Button</summary>
/// <summary>A <see cref="ColorButton"/> that acts as the main Button</summary>
@ -894,8 +874,6 @@ namespace AM2RLauncher
/// <summary>A <see cref="ColorButton"/> that is used to delete a mod</summary>
/// <summary>A <see cref="ColorButton"/> that is used to delete a mod</summary>
privateColorButtondeleteModButton;
privateColorButtondeleteModButton;
/// <summary>The <see cref="Label"/> that gives information for <see cref="languageDropDown"/>.</summary>
privateLabellanguageLabel;
/// <summary>The <see cref="Label"/> that entitles <see cref="profileDropDown"/>.</summary>
/// <summary>The <see cref="Label"/> that entitles <see cref="profileDropDown"/>.</summary>
privateLabelprofileLabel;
privateLabelprofileLabel;
/// <summary>The <see cref="Label"/> that gives author information for <see cref="profileDropDown"/>.</summary>
/// <summary>The <see cref="Label"/> that gives author information for <see cref="profileDropDown"/>.</summary>
@ -904,22 +882,12 @@ namespace AM2RLauncher
privateLabelprofileVersionLabel;
privateLabelprofileVersionLabel;
/// <summary>The <see cref="Label"/> that gives information for <see cref="mirrorDropDown"/>.</summary>
/// <summary>The <see cref="Label"/> that gives information for <see cref="mirrorDropDown"/>.</summary>
privateLabelmirrorLabel;
privateLabelmirrorLabel;
/// <summary>The <see cref="Label"/> that displays <see cref="VERSION"/>, aka the current launcher version.</summary>
privateLabelversionLabel;
/// <summary>The <see cref="Label"/> that gives information for <see cref="modSettingsProfileDropDown"/>.</summary>
/// <summary>The <see cref="Label"/> that gives information for <see cref="modSettingsProfileDropDown"/>.</summary>
privateLabelsettingsProfileLabel;
privateLabelsettingsProfileLabel;
/// <summary>The <see cref="Label"/> that compliments <see cref="progressBar"/>.</summary>
/// <summary>The <see cref="Label"/> that compliments <see cref="progressBar"/>.</summary>
privateLabelprogressLabel;
privateLabelprogressLabel;
/// <summary>The <see cref="Label"/> that gives a warning on failure to load the <see cref="newsWebView"/>.</summary>
privateLabelnewsNoConnectionLabel;
/// <summary>The <see cref="Label"/> that gives a warning on failure to load the <see cref="changelogWebView"/>.</summary>
privateLabelchangelogNoConnectionLabel;
/// <summary>The <see cref="Label"/> that gives a warning if the current selected <see cref="ProfileXML"/> shares the same save location has default AM2R.</summary>
/// <summary>The <see cref="Label"/> that gives a warning if the current selected <see cref="ProfileXML"/> shares the same save location has default AM2R.</summary>
privateLabelsaveWarningLabel;
privateLabelsaveWarningLabel;
/// <summary>The <see cref="Label"/> that describes <see cref="customEnvVarTextBox"/>.</summary>
privateLabelcustomEnvVarLabel;
/// <summary>A <see cref="CheckBox"/>, that indicates wether to automatically update AM2R or not.</summary>
/// <summary>A <see cref="CheckBox"/>, that indicates wether to automatically update AM2R or not.</summary>
privateCheckBoxautoUpdateAM2RCheck;
privateCheckBoxautoUpdateAM2RCheck;
@ -955,25 +923,6 @@ namespace AM2RLauncher
/// <summary>A <see cref="ProgressBar"/> that can be used to show progress for a specific task.</summary>
/// <summary>A <see cref="ProgressBar"/> that can be used to show progress for a specific task.</summary>
privateProgressBarprogressBar;
privateProgressBarprogressBar;
/// <summary>The Uri used by <see cref="newsWebView"/>.</summary>
privateUrinewsUri;
/// <summary>The Uri used by <see cref="changelogWebView"/>.</summary>
privateUrichangelogUri;
/// <summary>A <see cref="WebView"/> to display the DistributionCenter news page.</summary>
privateWebViewnewsWebView;
/// <summary>A <see cref="WebView"/> to display the DistributionCenter changelog page.</summary>
privateWebViewchangelogWebView;
/// <summary>A <see cref="TabPage"/> for the Launcher's main interface.</summary>
privateTabPagemainPage;
/// <summary>A <see cref="TabPage"/> for the Launcher's news integration.</summary>
privateTabPagenewsPage;
/// <summary>A <see cref="TabPage"/> for the Launcher's changelog integration.</summary>
privateTabPagechangelogPage;
/// <summary>A <see cref="TabPage"/> for the Launcher's profile settings.</summary>