fit-ui 2.10.13 → 2.11.2

package/dist/Fit.UI.js

@@ -682,7 +682,7 @@ Fit._internal =
 {
 	Core:
 	{
-		VersionInfo: { Major: 2, Minor: 10, Patch: 13 } // 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: 2 } // Do NOT modify format - version numbers are programmatically changed when releasing new versions - MUST be on a separate line!
 	}
 };
 
@@ -2219,17 +2219,29 @@ for (var i in vals = [1,3,6,8])
 Fit.Browser = {};
 Fit._internal.Browser = {};
 
+// Allow Fit.Browser unit tests to override user agent information
+// to test parsing of a range of different user agent strings.
+Fit._internal.Browser.UserAgent = navigator.userAgent;
+
 /// <function container="Fit.Browser" name="GetBrowser" access="public" static="true" returns='"Edge" | "Chrome" | "Safari" | "MSIE" | "Firefox" | "Opera" | "Unknown"'>
 /// 	<description>
-/// 		Returns name of browser. Possible values are: Chrome (which also covers modern versions of Opera and Edge),
-/// 		Safari, Edge (version 12-18), MSIE (version 8-11), Firefox, Opera (version 1-12), Unknown.
+/// 		Returns name of browser useful for adjusting behaviour based on render engine.
+/// 		Possible values are: Chrome (both WebKit and Blink based - also returned for modern versions of
+/// 		Opera and Edge), Safari (also returned for Edge, Chrome, Opera, and Firefox on iOS), Edge (version 12-18),
+/// 		MSIE (version 8-11), Firefox, Opera (version 1-12), and Unknown.
 /// 	</description>
 /// 	<param name="returnAppId" type="false" default="false"> Set True to have app specific identifier returned </param>
 /// </function>
-/// <function container="Fit.Browser" name="GetBrowser" access="public" static="true" returns='"Edge" | "EdgeChromium" | "Chrome" | "Safari" | "MSIE" | "Firefox" | "Opera" | "OperaChromium" | "Unknown"'>
+/// <function container="Fit.Browser" name="GetBrowser" access="public" static="true" returns='"Edge" | "Chrome" | "Safari" | "MSIE" | "Firefox" | "Opera" | "Unknown"'>
 /// 	<description>
-/// 		Returns browser app identifer. Possible values are: Chrome, Safari, Edge (version 12-18), EdgeChromium (version 85+),
-/// 		MSIE (version 8-11), Firefox, Opera (version 1-12), OperaChromium (version 15+), Unknown
+/// 		Returns browser app name useful for adjusting behaviour based on actual
+/// 		application, regardless of render engine and platform. Possible values are:
+/// 		Chrome (both Webkit and Blink based), Safari, Edge (both legacy and modern),
+/// 		MSIE (version 8-11), Firefox, Opera (both legacy and modern), and Unknown.
+/// 		Be careful not to check against browser app name and app version alone.
+/// 		For instance Opera 3 on a touch device is newer than Opera 60 on a Desktop
+/// 		device, as they are two completely different browsers. Check whether the browser runs on
+/// 		a tablet or phone using e.g. Fit.Browser.IsMobile(true) or Fit.Browser.GetInfo(true).IsMobile.
 /// 	</description>
 /// 	<param name="returnAppId" type="true"> Set True to have app specific identifier returned </param>
 /// </function>
@@ -2237,7 +2249,7 @@ Fit.Browser.GetBrowser = function(returnAppId)
 {
 	Fit.Validation.ExpectBoolean(returnAppId, true);
 
-	var agent = navigator.userAgent;
+	var agent = Fit._internal.Browser.UserAgent;
 
 	// IMPORTANT: The order in which browsers are detected matters! For instance, several browsers define both Chrome and Safari as part of their user agent string. Examples:
 	// Firefox 46	"Mozilla/5.0 (Windows NT 5.2; rv:46.0) Gecko/20100101 Firefox/46.0"
@@ -2251,20 +2263,18 @@ Fit.Browser.GetBrowser = function(returnAppId)
 	// Edge 18		"Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/70.0.3538.102 Safari/537.36 Edge/18.18363"
 	// Edge 85		"Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/85.0.4183.83 Safari/537.36 Edg/85.0.564.41"
 
-	if (agent.indexOf("Edge/") > -1)
-		return "Edge";
-	if (returnAppId === true && agent.indexOf("Edg/") > -1)
-		return "EdgeChromium";
+	if (agent.indexOf("Edge/") > -1 || (returnAppId === true && (agent.indexOf("Edg/") > -1 || agent.indexOf("EdgiOS/") > -1 || agent.indexOf("EdgA/") > -1)))
+		return "Edge"; // Legacy Edge is identified by "Edge/", Chromium based Edge is identified by "Edg/", Edge on iOS is identified by "EdgiOS/", and Edge on Android is identified by "EdgA/"
 	if (agent.indexOf("MSIE") > -1 || agent.indexOf("Trident") > -1)
 		return "MSIE";
-	if (agent.indexOf("Firefox") > -1)
-		return "Firefox";
-	if (agent.indexOf("Opera") > -1)
-		return "Opera";
-	if (returnAppId === true && agent.indexOf("OPR/") > -1)
-		return "OperaChromium";
-	if (agent.indexOf("Chrome") > -1)
-		return "Chrome";
+	if (agent.indexOf("Opera") > -1 || (returnAppId === true && (agent.indexOf("OPR/") > -1 || agent.indexOf("OPT/") > -1)))
+		return "Opera"; // Legacy Opera is identified by "Opera", Chromium based Opera is identified by "OPR/", and Opera on iOS is identified by "OPT/"
+	if (agent.indexOf("OPT/") > -1 && (agent.indexOf("iPhone;") > -1 || agent.indexOf("iPad;") > -1))
+		return "Safari"; // Opera on iOS does not contain "Opera", nor "Safari" to identify the engine like other browsers using WebView - we can use "OPT/" plus "iPhone;" or "iPad;" instead
+	if (agent.indexOf("Firefox") > -1 || (returnAppId === true && agent.indexOf("FxiOS/") > -1))
+		return "Firefox"; // Firefox is identified by "Firefox" except on iOS where "FxiOS/" identifies it instead
+	if (agent.indexOf("Chrome") > -1 || (returnAppId === true && agent.indexOf("CriOS/") > -1))
+		return "Chrome"; // Chrome is identified by "Chrome" except on iOS where "CriOS/" identifies it instead
 	if (agent.indexOf("Safari") > -1)
 		return "Safari";
 
@@ -2298,37 +2308,56 @@ Fit.Browser.GetVersion = function(returnAppVersion)
 
 	var start = 0;
 	var end = 0;
-	var agent = navigator.userAgent;
+	var agent = Fit._internal.Browser.UserAgent;
 
 	if (browser === "Edge")
 	{
-		start = agent.indexOf("Edge/");
-		start = (start !== -1 ? start + 5 : 0);
-		end = agent.indexOf(".", start);
-		end = (end !== -1 ? end : 0);
-	}
-	if (browser === "EdgeChromium")
-	{
-		start = agent.indexOf("Edg/");
-		start = (start !== -1 ? start + 4 : 0);
+		var search = agent.indexOf("Edge/") > -1 && "Edge/" || agent.indexOf("Edg/") > -1 && "Edg/" || agent.indexOf("EdgiOS/") > -1 && "EdgiOS/" || agent.indexOf("EdgA/") > -1 && "EdgA/" || null;
+
+		start = search !== null && agent.indexOf(search) || -1;
+		start = (start !== -1 ? start + search.length : 0);
 		end = agent.indexOf(".", start);
 		end = (end !== -1 ? end : 0);
 	}
-	if (browser === "Chrome")
+	else if (browser === "Chrome")
 	{
-		start = agent.indexOf("Chrome/");
-		start = (start !== -1 ? start + 7 : 0);
+		var search = agent.indexOf("Chrome/") > -1 && "Chrome/" || agent.indexOf("CriOS/") > -1 && "CriOS/" || null;
+
+		start = search !== null && agent.indexOf(search) || -1;
+		start = (start !== -1 ? start + search.length : 0);
 		end = agent.indexOf(".", start);
 		end = (end !== -1 ? end : 0);
 	}
-	if (browser === "Safari")
+	else if (browser === "Safari")
 	{
-		start = agent.indexOf("Version/");
-		start = (start !== -1 ? start + 8 : 0);
-		end = agent.indexOf(".", start);
-		end = (end !== -1 ? end : 0);
+		var search = agent.indexOf("CriOS/") > -1 && "CriOS/" || agent.indexOf("FxiOS/") > -1 && "FxiOS/" || agent.indexOf("EdgiOS/") > -1 && "EdgiOS/" || agent.indexOf("OPT/") > -1 && "OPT/" || null;
+
+		if (search !== null) // Browser based on WebView on iOS - Chrome, Firefox, Edge, or Opera
+		{
+			if (returnAppVersion !== true) // Return Safari version parsed from OS version
+			{
+				start = agent.indexOf(" OS ");
+				start = (start !== -1 ? start + 4 : 0);
+				end = agent.indexOf("_", start);
+				end = (end !== -1 ? end : 0);
+			}
+			else // Return application version
+			{
+				start = agent.indexOf(search);
+				start = (start !== -1 ? start + search.length : 0);
+				end = agent.indexOf(".", start);
+				end = (end !== -1 ? end : 0);
+			}
+		}
+		else // Real Safari or Firefox on iPad which does not identify itself as Firefox
+		{
+			start = agent.indexOf("Version/");
+			start = (start !== -1 ? start + 8 : 0);
+			end = agent.indexOf(".", start);
+			end = (end !== -1 ? end : 0);
+		}
 	}
-	if (browser === "MSIE")
+	else if (browser === "MSIE")
 	{
 		if (agent.indexOf("MSIE") > -1)
 		{
@@ -2345,37 +2374,21 @@ Fit.Browser.GetVersion = function(returnAppVersion)
 			end = (end !== -1 ? end : 0);
 		}
 	}
-	if (browser === "Firefox")
+	else if (browser === "Firefox")
 	{
-		start = agent.indexOf("Firefox/");
-		start = (start !== -1 ? start + 8 : 0);
-		end = agent.indexOf(".", start);
-		end = (end !== -1 ? end : 0);
-	}
-	if (browser === "Opera")
-	{
-		start = agent.indexOf("Version/");
-		start = (start !== -1 ? start + 8 : -1);
-
-		if (start === -1)
-		{
-			start = agent.indexOf("Opera/");
-			start = (start !== -1 ? start + 6 : -1);
-		}
-
-		if (start === -1)
-		{
-			start = agent.indexOf("Opera ");
-			start = (start !== -1 ? start + 6 : -1);
-		}
+		var search = agent.indexOf("Firefox/") > -1 && "Firefox/" || agent.indexOf("FxiOS/") > -1 && "FxiOS/" || null;
 
+		start = search !== null && agent.indexOf(search) || -1;
+		start = (start !== -1 ? start + search.length : 0);
 		end = agent.indexOf(".", start);
 		end = (end !== -1 ? end : 0);
 	}
-	if (browser === "OperaChromium")
+	else if (browser === "Opera")
 	{
-		start = agent.indexOf("OPR/");
-		start = (start !== -1 ? start + 4 : 0);
+		var search = agent.indexOf("Version/") > -1 && "Version/" || agent.indexOf("Opera ") > -1 && "Opera " || agent.indexOf("Opera/") > -1 && "Opera/" || agent.indexOf("OPR/") > -1 && "OPR/" || agent.indexOf("OPT/") > -1 && "OPT/" || null;
+
+		start = search !== null && agent.indexOf(search) || -1;
+		start = (start !== -1 ? start + search.length : 0);
 		end = agent.indexOf(".", start);
 		end = (end !== -1 ? end : 0);
 	}
@@ -2909,7 +2922,22 @@ Fit.Browser.GetScreenDimensions = function(onlyAvailable)
 }
 
 /// <function container="Fit.Browser" name="IsMobile" access="public" static="true" returns="boolean">
-/// 	<description> Returns value indicating whether device is a mobile device or not </description>
+/// 	<description>
+/// 		Returns value indicating whether device is a mobile device or not.
+/// 		Notice that some phones and tablets may identify as desktop devices,
+/// 		in which case IsMobile(..) will return False. In this case consider
+/// 		using Fit.Browser.IsTouchEnabled(), e.g. in combination with
+/// 		Fit.Browser.GetBrowser(), or a check against the size of the viewport,
+/// 		which will provide some indication as to whether device should be treated
+/// 		as a mobile device or not. As an example, Safari on iPad identifies as
+/// 		a Mac computer by default (it has &quot;Request Desktop Website&quot; enabled by default).
+/// 		To always detect iPad and iPhone as a mobile device, no matter the configuration
+/// 		of &quot;Request Desktop Website&quot;), simply use:
+/// 		var isMobile = Fit.Browser.IsMobile(true) || (Fit.Browser.GetBrowser() === &quot;Safari&quot; &amp;&amp; Fit.Browser.IsTouchEnabled())
+/// 		We will not be able to distinguish between an iPhone and an iPad though, not even
+/// 		by looking at the viewport size, since this approach is unreliable with high resolution
+/// 		iPhones and support for split screen which affects the viewport size.
+/// 	</description>
 /// 	<param name="includeTablets" type="boolean" default="true"> Value indicating whether tablets are considered mobile devices or not </param>
 /// </function>
 Fit.Browser.IsMobile = function(includeTablets)
@@ -2919,7 +2947,7 @@ Fit.Browser.IsMobile = function(includeTablets)
 	// Based on http://detectmobilebrowsers.com
 	// See About section for Tablet support: http://detectmobilebrowsers.com/about
 
-	var nav = navigator.userAgent || navigator.vendor;
+	var nav = Fit._internal.Browser.UserAgent;
 
 	if (includeTablets !== false && /android|ipad|playbook|silk/i.test(nav))
 		return true;
@@ -2935,7 +2963,6 @@ Fit.Browser.IsTouchEnabled = function()
 	return ("ontouchstart" in window);
 }
 
-
 /// <function container="Fit.Browser" name="Log" access="public" static="true">
 /// 	<description> Log message or object </description>
 /// 	<param name="msg" type="object"> Message or object to log </param>
@@ -8442,7 +8469,9 @@ Fit.Events.AddMutationObserver = function(elm, obs, deep)
 		Fit.Events.AddHandler(document, "touchstart", Fit._internal.Events.CheckMutations);
 		Fit.Events.AddHandler(document, "touchend", Fit._internal.Events.CheckMutations);
 		Fit.Events.AddHandler(document, "touchcancel", Fit._internal.Events.CheckMutations);
-		Fit._internal.Events.MutationObserverIntervalId = setInterval(Fit._internal.Events.CheckMutations, 1000);
+
+		var checkInterval = Fit._internal.Events.Browser.Name !== "MSIE" ? 500 : 1000;
+		Fit._internal.Events.MutationObserverIntervalId = setInterval(Fit._internal.Events.CheckMutations, checkInterval);
 	}
 
 	// Add mutation observer
@@ -22782,12 +22811,25 @@ Fit.Controls.Input = function(ctlId)
 			}
 		}
 
+		// Guard against disposed control in case Focused(false) was called and an OnBlur handler disposed the control.
+		// As the code further up shows, we call me._internal.FireOnBlur() if a dialog is currently open. This results
+		// in OnBlur firing immediately, while normally it happens asynchronously due to how OnFocus and OnBlur is handled
+		// in ControlBase, in which case we do not need to worry that the control might be disposed. It happens "later".
+		// The situation can easily arise if an OnScroll handler is reponsible for removing focus from a control,
+		// and if that control also has an OnBlur handler registered which disposes the control. Scrolling with a
+		// dialog open will then trigger the situation which we guard against here.
+		if (me === null)
+		{
+			return false;
+		}
+
 		if (me.DesignMode() === true)
 		{
 			// If a dialog is open and it belongs to this control instance, and focus is found within dialog, then control is considered having focus.
 			// However, if <body> is focused while dialog is open, control is also considered to have focus, since dialog temporarily assigns focus to
 			// <body> when tabbing between elements within the dialog. This seems safe as no other control can be considered focused if <body> has focus.
-			if (Fit._internal.Controls.Input.ActiveEditorForDialog === me && (Fit.Dom.Contained(Fit._internal.Controls.Input.ActiveDialogForEditor.getElement().$, Fit.Dom.GetFocused()) === true || Fit.Dom.GetFocused() === document.body))
+			// We also consider the control focused if an associated dialog (modal) is currently loading (Fit._internal.Controls.Input.ActiveDialogForEditor is null).
+			if (Fit._internal.Controls.Input.ActiveEditorForDialog === me && (Fit._internal.Controls.Input.ActiveDialogForEditor === null || Fit.Dom.Contained(Fit._internal.Controls.Input.ActiveDialogForEditor.getElement().$, Fit.Dom.GetFocused()) === true || Fit.Dom.GetFocused() === document.body))
 				return true;
 
 			// If a toolbar dialog/callout is open and contains the element currently having focus, then control is considered having focus.
@@ -22834,12 +22876,48 @@ Fit.Controls.Input = function(ctlId)
 				});
 
 				updateDesignEditorPlaceholder();
+				updateDesignEditorSize(); // In case auto grow is enabled, in which case editor must adjust its height to its new content
 			}
 			else
 			{
 				input.value = val;
 			}
 
+			// Notice: Identical logic is NOT found in DesignMode(true, config) as with RevokeExternalBlobUrlsOnDispose below.
+			// When the RevokeUnreferencedBlobUrlsOnValueSet mechanism is in play, the control has already been used as an HTML editor
+			// before, as we are expecting the control to be re(-used) to manipulate different values. In this case we already have
+			// designEditorConfig available, although it could theoretically be changed over time so RevokeUnreferencedBlobUrlsOnValueSet
+			// is sometimes enabled and sometimes not - but we don't care to support poor design like this:
+			// input.DesignMode(true, configWithRevokeUnreferencedBlobUrlsOnValueSetDISBLED);
+			// input.Value("New HTML value");
+			// input.DesignMode(true, configWithRevokeUnreferencedBlobUrlsOnValueSetENABLED); // This will not clean up image blobs no longer referenced
+			if (designEditorConfig !== null && designEditorConfig.Plugins && designEditorConfig.Plugins.Images && designEditorConfig.Plugins.Images.RevokeUnreferencedBlobUrlsOnValueSet === true)
+			{
+				// Remove image blobs from memory when a new value is set, unless some (or all)
+				// of these image blobs are still referenced in the new value, of course.
+				// This is useful if an editor instance is being (re-)used to modify different values.
+				// NOTICE: There is a major memory leak in CKEditor related to bulk pasting images
+				// from the file system, and the last image pasted always remains in memory:
+				// https://github.com/ckeditor/ckeditor4/issues/5124
+
+				var blobUrlsReferenced = Fit.String.ParseImageBlobUrls(val);
+				var newImageBlobUrls = [];
+
+				Fit.Array.ForEach(imageBlobUrls, function(blobUrl)
+				{
+					if (Fit.Array.Contains(blobUrlsReferenced, blobUrl) === true)
+					{
+						newImageBlobUrls.push(blobUrl); // Keep - still referenced in new value
+					}
+					else
+					{
+						URL.revokeObjectURL(blobUrl); // Revoke - no longer referenced in new value
+					}
+				});
+
+				imageBlobUrls = newImageBlobUrls;
+			}
+
 			// Notice: Identical logic found in DesignMode(true, config)!
 			if (designEditorConfig !== null && designEditorConfig.Plugins && designEditorConfig.Plugins.Images && designEditorConfig.Plugins.Images.RevokeExternalBlobUrlsOnDispose === true)
 			{
@@ -22848,8 +22926,11 @@ Fit.Controls.Input = function(ctlId)
 				// is allowed (and expected) to take control over memory management for these blobs
 				// based on the rule set in RevokeBlobUrlsOnDispose.
 				// This code is also found in DesignMode(true, config) since images might be added before
-				// editor is created, in which case we do not yet have the editor configuration used to determine
-				// the desired behaviour.
+				// DesignMode is enabled, in which case we do not yet have the editor configuration needed
+				// to determine the desired behaviour.
+				// NOTICE: There is a major memory leak in CKEditor related to bulk pasting images
+				// from the file system, and the last image pasted always remains in memory:
+				// https://github.com/ckeditor/ckeditor4/issues/5124
 
 				var blobUrls = Fit.String.ParseImageBlobUrls(val);
 
@@ -23598,6 +23679,12 @@ Fit.Controls.Input = function(ctlId)
 	/// 		These images are furthermore subject to the rule set in RevokeBlobUrlsOnDispose.
 	/// 		Defaults to False.
 	/// 	</member>
+	/// 	<member name="RevokeUnreferencedBlobUrlsOnValueSet" type="boolean" default="undefined">
+	/// 		This option is in effect when EmbedType is blob.
+	/// 		Dispose images from blob storage (revoke blob URLs) when value is changed with Value(..),
+	/// 		but keep any images still referenced in new value. This is useful if an editor instance
+	/// 		is being used to modify different HTML values over time.
+	/// 	</member>
 	/// </container>
 
 	/// <container name="Fit.Controls.InputTypeDefs.DesignModeConfigPlugins">
@@ -25453,6 +25540,8 @@ Fit.Controls.Input = function(ctlId)
 	{
 		if (me.DesignMode() === true && designEditorHeightMonitorId === -1)
 		{
+			// Postpone if editor is not ready yet
+
 			if (designEditorUpdateSizeDebouncer !== -1)
 			{
 				clearTimeout(designEditorUpdateSizeDebouncer);
@@ -25468,7 +25557,7 @@ Fit.Controls.Input = function(ctlId)
 				// This is a problem because CKEditor uses setTimeout(..) to for instance
 				// allow early registration of events, and because resources are loaded
 				// in an async. manner.
-				designEditorUpdateSizeDebouncer = setTimeout(function()
+				designEditorUpdateSizeDebouncer = setTimeout(function() // Timer is stopped if control is disposed
 				{
 					designEditorUpdateSizeDebouncer = -1;
 					updateDesignEditorSize();
@@ -25477,19 +25566,8 @@ Fit.Controls.Input = function(ctlId)
 				return;
 			}
 
-			//var w = me.Width();
-			var h = me.Height();
-
-			// Default control width is 200px (defined in Styles.css).
-			// NOTICE: resize does not work reliably when editor is hidden, e.g. behind a tab with display:none.
-			// The height set will not have the height of the toolbar substracted since the height can not be
-			// determined for hidden objects, so the editor will become larger than the value set (height specified + toolbar height).
-			// http://docs.ckeditor.com/#!/api/CKEDITOR.editor-method-resize
-			designEditorSuppressOnResize = true;
-			designEditor.resize("100%", h.Value > -1 ? h.Value + h.Unit : "100%"); // A height of 100% allow editor to automatically adjust the height of the editor's content area to the height of its content (data-autogrow="true" must be set to make control container adjust to its content as well)
-			designEditorSuppressOnResize = false;
-
-			// Set mutation observer responsible for updating editor size once it becomes visible
+			// Postpone update to editor size if control is currently hidden or not
+			// rooted in DOM, in which case designEditor.resize(..) will throw an error.
 
 			if (mutationObserverId !== -1) // Cancel any mutation observer previously registered
 			{
@@ -25497,22 +25575,43 @@ Fit.Controls.Input = function(ctlId)
 				mutationObserverId = -1;
 			}
 
-			var concealer = Fit.Dom.GetConcealer(me.GetDomElement()); // Get element hiding editor
-
-			if (concealer !== null) // Editor is hidden - adjust size when it becomes visible
+			if (Fit.Dom.IsVisible(me.GetDomElement()) === false) // Hidden (e.g. display:none or not rooted in DOM)
 			{
-				mutationObserverId = Fit.Events.AddMutationObserver(concealer, function(elm)
+				// Mutation observer is triggered when element changes, including when rooted, in which case
+				// width and height becomes measurable, and changes to dimensions also trigger mutation observer.
+				mutationObserverId = Fit.Events.AddMutationObserver(me.GetDomElement(), function(elm)
 				{
 					if (Fit.Dom.IsVisible(me.GetDomElement()) === true)
 					{
-						designEditorSuppressOnResize = true;
-						designEditor.resize("100%", h.Value > -1 ? h.Value + h.Unit : "100%"); // A height of 100% allow editor to automatically adjust the height of the editor's content area to the height of its content (data-autogrow="true" must be set to make control container adjust to its content as well)
-						designEditorSuppressOnResize = false;
+						disconnect();
+						mutationObserverId = -1;
 
-						disconnect(); // Observers are expensive - remove when no longer needed
+						updateDesignEditorSize(); // Does nothing if DesignMode is no longer enabled
 					}
 				});
+
+				return;
 			}
+
+			//var w = me.Width();
+			var h = me.Height();
+
+			// If editor is configured with AutoGrow enabled and toolbar is configured with HideWhenInactive,
+			// then editor won't be able to adjust its height when not focused, since a fixed height is applied
+			// to the editable area while the toolbar is hidden. Therefore, temporarily show the toolbar, update
+			// the editor size, and then hide the toolbar again.
+			var showHideToolbar = me.Focused() === false;
+
+			// Default control width is 200px (defined in Styles.css).
+			// NOTICE: resize does not work reliably when editor is hidden, e.g. behind a tab with display:none.
+			// The height set will not have the height of the toolbar substracted since the height can not be
+			// determined for hidden objects, so the editor will become larger than the value set (height specified + toolbar height).
+			// http://docs.ckeditor.com/#!/api/CKEDITOR.editor-method-resize
+			designEditorSuppressOnResize = true;
+			showHideToolbar && restoreHiddenToolbarInDesignEditor(true); // Does nothing unless HideWhenInactive is enabled - true argument prevents call back to updateDesignEditorSize again, hence preventing a "maximum call stack exceeded" error
+			designEditor.resize("100%", h.Value > -1 ? h.Value + h.Unit : "100%"); // A height of 100% allow editor to automatically adjust the height of the editor's content area to the height of its content (data-autogrow="true" must be set to make control container adjust to its content as well)
+			showHideToolbar && hideToolbarInDesignMode(true); // Does nothing unless HideWhenInactive is enabled - true argument prevents call back to updateDesignEditorSize again, hence preventing a "maximum call stack exceeded" error
+			designEditorSuppressOnResize = false;
 		}
 	}
 
@@ -25560,8 +25659,10 @@ Fit.Controls.Input = function(ctlId)
 		return (toolbarContainer !== null && toolbarContainer.style.display === "none");
 	}
 
-	function hideToolbarInDesignMode()
+	function hideToolbarInDesignMode(suppressUpdateEditorSize)
 	{
+		Fit.Validation.ExpectBoolean(suppressUpdateEditorSize, true);
+
 		if (designModeEnabledAndReady() === true && designEditorConfig !== null && designEditorConfig.Toolbar && designEditorConfig.Toolbar.HideWhenInactive === true)
 		{
 			var toolbarContainer = designEditorDom.Top || designEditorDom.Bottom; // Top is null if editor is placed at the bottom
@@ -25593,12 +25694,14 @@ Fit.Controls.Input = function(ctlId)
 			toolbarContainer.style.display = "none";
 
 			// Make editable area adjust to take up space previously consumed by toolbar
-			updateSize === true && updateDesignEditorSize();
+			updateSize === true && suppressUpdateEditorSize !== true && updateDesignEditorSize();
 		}
 	}
 
-	function restoreHiddenToolbarInDesignEditor()
+	function restoreHiddenToolbarInDesignEditor(suppressUpdateEditorSize)
 	{
+		Fit.Validation.ExpectBoolean(suppressUpdateEditorSize, true);
+
 		if (designModeEnabledAndReady() === true && designEditorConfig !== null && designEditorConfig.Toolbar && designEditorConfig.Toolbar.HideWhenInactive === true)
 		{
 			// Toolbar has been initially hidden - make it appear again
@@ -25642,7 +25745,7 @@ Fit.Controls.Input = function(ctlId)
 				// Update size of editable area in case auto grow is not enabled, in which case
 				// toolbar will now have taken up space outside of control's container (overflowing).
 				// Make editable area fit control container again.
-				updateDesignEditorSize();
+				suppressUpdateEditorSize !== true && updateDesignEditorSize();
 			}
 			else
 			{