From 1cde5584b148b4bbc399e37a02849b11690f344e Mon Sep 17 00:00:00 2001
From: hegyak <hegyak@gmail.com>
Date: Thu, 16 Oct 2014 23:19:25 +0000
Subject: [PATCH] Changed the Status Bar Profile Icon to use the same logic as
 the Profile Menu Item.  This is done to ensure the user has selected a
 profile when they click on the Profile Icon before we hide it.

---
 BizHawk.Client.EmuHawk/MainForm.Events.cs | 21 +++++++++++++++++----
 1 file changed, 17 insertions(+), 4 deletions(-)

diff --git a/BizHawk.Client.EmuHawk/MainForm.Events.cs b/BizHawk.Client.EmuHawk/MainForm.Events.cs
index 7abbf8ec08..05114e26c3 100644
--- a/BizHawk.Client.EmuHawk/MainForm.Events.cs
+++ b/BizHawk.Client.EmuHawk/MainForm.Events.cs
@@ -2236,10 +2236,23 @@ namespace BizHawk.Client.EmuHawk
 
 		private void ProfileFirstBootLabel_Click(object sender, EventArgs e)
 		{
-			var profileForm = new ProfileConfig();
-			profileForm.ShowDialog();
-			Global.Config.FirstBoot = false;
-			ProfileFirstBootLabel.Visible = false;
+			//We should make sure the user actually picked a Profile.
+			if (new ProfileConfig().ShowDialog() == DialogResult.OK)
+			{
+				GlobalWin.OSD.AddMessage("Profile settings saved");
+				//We hide the FirstBoot items since the user setup a Profile
+				//Is it a bad thing to do this constantly?
+				Global.Config.FirstBoot = false;
+				ProfileFirstBootLabel.Visible = false;
+			}
+			else
+			{
+				GlobalWin.OSD.AddMessage("Profile config aborted");
+			}
+			//var profileForm = new ProfileConfig();
+			//profileForm.ShowDialog();
+			//Global.Config.FirstBoot = false;
+			//ProfileFirstBootLabel.Visible = false;
 		}
 
 		#endregion