fit-ui 2.20.0 → 2.20.1

package/dist/Fit.UI.js

@@ -682,7 +682,7 @@ Fit._internal =
 {
 	Core:
 	{
-		VersionInfo: { Major: 2, Minor: 20, Patch: 0 } // Do NOT modify format - version numbers are programmatically changed when releasing new versions - MUST be on a separate line!
+		VersionInfo: { Major: 2, Minor: 20, Patch: 1 } // Do NOT modify format - version numbers are programmatically changed when releasing new versions - MUST be on a separate line!
 	}
 };
 
@@ -20131,9 +20131,9 @@ Fit.Controls.DropDown = function(ctlId)
 				// container starts.
 				// https://github.com/Jemt/Fit.UI/issues/51
 
-				var scrollParent = Fit.Dom.GetScrollParent(me.GetDomElement());
+				var scrollParent = Fit.Dom.GetScrollParent(me.GetDomElement()); // Null if DropDown or a parent has position:fixed
 
-				if (scrollParent !== Fit.Dom.GetScrollDocument())
+				if (scrollParent !== null && scrollParent !== Fit.Dom.GetScrollDocument())
 				{
 					// Control is positioned within a container with scroll.
 					// Calculate position relative to viewport to determine
@@ -20162,10 +20162,10 @@ Fit.Controls.DropDown = function(ctlId)
 				// container ends.
 				// https://github.com/Jemt/Fit.UI/issues/51 (see re-open comment oct. 30, 2020)
 
-				var scrollParent = Fit.Dom.GetScrollParent(me.GetDomElement());
+				var scrollParent = Fit.Dom.GetScrollParent(me.GetDomElement()); // Null if DropDown or a parent has position:fixed
 				var alternativeTopPos = -1;
 
-				if (scrollParent !== Fit.Dom.GetScrollDocument())
+				if (scrollParent !== null && scrollParent !== Fit.Dom.GetScrollDocument())
 				{
 					// Control is positioned within a container with scroll.
 					// Calculate position relative to viewport to determine
@@ -22114,7 +22114,7 @@ Fit.Controls.WSDropDown = function(ctlId)
 
 		Fit.Internationalization.RemoveOnLocaleChanged(localize);
 
-		me = list = tree = actionMenu = search = forceNewSearch = hideLinesForFlatData = dataRequested = dataLoading /*= nodesPopulated*/ = requestCount = onDataLoadedCallback = suppressTreeOnOpen = timeOut = currentRequest = classes = autoUpdatedSelections = useActionMenu = useActionMenuForced = useActionMenuAfterLoad = useActionMenuSortingLocale = useActionMenuHtmlAllowed = treeViewEnabled = orgPlaceholder = customPlaceholderSet = translations = onRequestHandlers = onResponseHandlers = null;
+		me = list = tree = actionMenu = search = forceNewSearch = hideLinesForFlatData = dataRequested = dataLoading /*= nodesPopulated*/ = requestCount = onDataLoadedCallback = suppressTreeOnOpen = timeOut = currentRequest = classes = autoUpdatedSelections = useActionMenu = useActionMenuForced = useActionMenuAfterLoad = useActionMenuSortingLocale = useActionMenuHtmlAllowed = treeViewEnabled = orgPlaceholder = customPlaceholderSet = translations = onRequestHandlers = onResponseHandlers = onAbortHandlers /*= onPopulatedHandlers*/ = null;
 
 		base();
 	});
@@ -32832,9 +32832,9 @@ Fit.Controls.WSTreeView = function(ctlId)
 
 			// Preserve scroll position in case user scrolled view while data was loading. Calling node.Focused(true)
 			// or SetActiveNode(..) further down will not only focus or highlight node, but also scroll it into view.
-			var scrollbars = Fit.Dom.GetScrollBars(me.GetDomElement());
-			var scrollParent = scrollbars.Vertical.Enabled === true || scrollbars.Horizontal.Enabled === true ? me.GetDomElement() : Fit.Dom.GetScrollParent(me.GetDomElement());
-			var scrollPositionToRestore = keepState === true ? { Top: scrollParent.scrollTop, Left: scrollParent.scrollLeft } : null;
+			var scrollbars = Fit.Dom.GetScrollBars(me.GetDomElement()); // Scrollbars not enabled when no data is loaded to overflow own container
+			var scrollParent = scrollbars.Vertical.Enabled === true || scrollbars.Horizontal.Enabled === true ? me.GetDomElement() : Fit.Dom.GetScrollParent(me.GetDomElement()); // Null if data is not causing local scroll and if WSTreeView is not mounted in another scrollable container (likely within a parent with position:fixed)
+			var scrollPositionToRestore = scrollParent !== null && keepState === true ? { Top: scrollParent.scrollTop, Left: scrollParent.scrollLeft } : null;
 
 			// Remove nodes
 			me._internal.ExecuteWithNoOnChange(function()