fit-ui 2.11.17 → 2.11.18

package/dist/Fit.UI.js

@@ -682,7 +682,7 @@ Fit._internal =
 {
 	Core:
 	{
-		VersionInfo: { Major: 2, Minor: 11, Patch: 17 } // Do NOT modify format - version numbers are programmatically changed when releasing new versions - MUST be on a separate line!
+		VersionInfo: { Major: 2, Minor: 11, Patch: 18 } // Do NOT modify format - version numbers are programmatically changed when releasing new versions - MUST be on a separate line!
 	}
 };
 
@@ -17044,67 +17044,70 @@ Fit.Controls.DropDown = function(ctlId)
 
 		// Make drop down close when user clicks outside of control
 
-		if (isMobile === false)
+		postpone(function() // Postpone to prevent drop down from closing if created and auto opened during OnClick/OnTouchStart (https://github.com/Jemt/Fit.UI/issues/170)
 		{
-			var eventId = Fit.Events.AddHandler(document, "click", function(e)
+			if (isMobile === false)
 			{
-				var target = Fit.Events.GetTarget(e);
-
-				if (target !== document.documentElement && target !== document.body && Fit.Dom.IsRooted(target) === false)
+				var eventId = Fit.Events.AddHandler(document, "click", function(e)
 				{
-					return; // Do not close DropDown if target no longer exists - this may happen if something is removed within DropDown (e.g. an item in the WSDropDown's action menu)
-				}
+					var target = Fit.Events.GetTarget(e);
 
-				if (me.IsDropDownOpen() === true && target !== me.GetDomElement() && Fit.Dom.Contained(me.GetDomElement(), target) === false)
-				{
-					me.CloseDropDown();
-				}
-			});
-			Fit.Array.Add(closeHandlers, eventId);
-		}
-		else
-		{
-			// OnClick does not work reliably on mobile (at least not on iOS 9 and 10), so using touch events instead
-
-			var coords = null;
-			var eventId = -1;
+					if (target !== document.documentElement && target !== document.body && Fit.Dom.IsRooted(target) === false)
+					{
+						return; // Do not close DropDown if target no longer exists - this may happen if something is removed within DropDown (e.g. an item in the WSDropDown's action menu)
+					}
 
-			eventId = Fit.Events.AddHandler(document, "touchstart", function(e)
+					if (me.IsDropDownOpen() === true && target !== me.GetDomElement() && Fit.Dom.Contained(me.GetDomElement(), target) === false)
+					{
+						me.CloseDropDown();
+					}
+				});
+				Fit.Array.Add(closeHandlers, eventId);
+			}
+			else
 			{
-				var target = Fit.Events.GetTarget(e);
+				// OnClick does not work reliably on mobile (at least not on iOS 9 and 10), so using touch events instead
 
-				coords = null;
+				var coords = null;
+				var eventId = -1;
 
-				if (target !== document.documentElement && target !== document.body && Fit.Dom.IsRooted(target) === false)
+				eventId = Fit.Events.AddHandler(document, "touchstart", function(e)
 				{
-					return; // Do not close DropDown if target no longer exists - this may happen if something is removed within DropDown (e.g. an item in the WSDropDown's action menu)
-				}
+					var target = Fit.Events.GetTarget(e);
 
-				if (me.IsDropDownOpen() === true && target !== me.GetDomElement() && Fit.Dom.Contained(me.GetDomElement(), target) === false)
-				{
-					coords = Fit.Events.GetPointerState().Coordinates.Document;
-				}
-			});
-			Fit.Array.Add(closeHandlers, eventId);
+					coords = null;
 
-			eventId = Fit.Events.AddHandler(document, "touchend", function(e)
-			{
-				if (coords === null)
-					return;
-
-				// Determine whether user moved finger (e.g. to scroll page) in which case we do not want to close the menu
+					if (target !== document.documentElement && target !== document.body && Fit.Dom.IsRooted(target) === false)
+					{
+						return; // Do not close DropDown if target no longer exists - this may happen if something is removed within DropDown (e.g. an item in the WSDropDown's action menu)
+					}
 
-				var curCoords = Fit.Events.GetPointerState().Coordinates.Document;
-				var moved = (Math.abs(coords.X - curCoords.X) > 10 || Math.abs(coords.Y - curCoords.Y) > 10); // Must be moved at least 10px left/right or up/down
+					if (me.IsDropDownOpen() === true && target !== me.GetDomElement() && Fit.Dom.Contained(me.GetDomElement(), target) === false)
+					{
+						coords = Fit.Events.GetPointerState().Coordinates.Document;
+					}
+				});
+				Fit.Array.Add(closeHandlers, eventId);
 
-				if (moved === false)
+				eventId = Fit.Events.AddHandler(document, "touchend", function(e)
 				{
-					me.CloseDropDown();
-					me.Focused(false);
-				}
-			});
-			Fit.Array.Add(closeHandlers, eventId);
-		}
+					if (coords === null)
+						return;
+
+					// Determine whether user moved finger (e.g. to scroll page) in which case we do not want to close the menu
+
+					var curCoords = Fit.Events.GetPointerState().Coordinates.Document;
+					var moved = (Math.abs(coords.X - curCoords.X) > 10 || Math.abs(coords.Y - curCoords.Y) > 10); // Must be moved at least 10px left/right or up/down
+
+					if (moved === false)
+					{
+						me.CloseDropDown();
+						me.Focused(false);
+					}
+				});
+				Fit.Array.Add(closeHandlers, eventId);
+			}
+		});
 
 		// Make drop down close if focus is lost on mobile
 		// which happens when onscreen keyboard is closed.
@@ -17136,7 +17139,7 @@ Fit.Controls.DropDown = function(ctlId)
 				if (Fit.Browser.GetBrowser() === "MSIE" || Fit.Browser.GetBrowser() === "Edge")
 				{
 					txtPrimary.readOnly = false; // ReadOnly set to true to make text-overflow:ellipsis work
-					setTimeout(function() { txtPrimary.blur(); txtPrimary.focus(); }, 0); // Change to ReadOnly is not applied immediately unless blurred and re-focused
+					postpone(function() { txtPrimary.blur(); txtPrimary.focus(); }); // Change to ReadOnly is not applied immediately unless blurred and re-focused
 				}
 
 				clearTextSelectionOnInputChange = true;
@@ -18741,7 +18744,7 @@ Fit.Controls.DropDown = function(ctlId)
 		{
 			var orgValue = txt.value;
 
-			setTimeout(function() // Timeout to queue event to have pasted value available
+			postpone(function() // Timeout to queue event to have pasted value available
 			{
 				/*if (me.TextSelectionMode() === true && txt.value === prevTextSelection)
 				{
@@ -18784,7 +18787,7 @@ Fit.Controls.DropDown = function(ctlId)
 						prevValue = txt.value;
 					}
 				}
-			}, 0);
+			});
 
 			clearTextSelectionOnInputChange = false;
 		}
@@ -19879,6 +19882,20 @@ Fit.Controls.DropDown = function(ctlId)
 			picker._internal.FireOnHide();
 	}
 
+	function postpone(cb, timeout)
+	{
+		Fit.Validation.ExpectFunction(cb);
+		Fit.Validation.ExpectInteger(timeout, true)
+
+		setTimeout(function()
+		{
+			if (me !== null) // Make sure control has not been disposed
+			{
+				cb();
+			}
+		}, timeout || 0);
+	}
+
 	init();
 }