@openui5/sap.m 1.101.0 → 1.102.0

package/src/sap/m/GrowingEnablement.js

@@ -65,7 +65,6 @@ sap.ui.define([
 			this._sGroupingPath = "";
 			this._bDataRequested = false;
 			this._bSkippedItemsUpdateUntilDataReceived = false;
-			this._oContainerDomRef = null;
 			this._iLastItemsCount = 0;
 			this._iTriggerTimer = 0;
 			this._aChunk = [];
@@ -100,7 +99,6 @@ sap.ui.define([
 			this._oControl.$("triggerList").remove();
 			this._oControl.bUseExtendedChangeDetection = false;
 			this._oControl.removeDelegate(this);
-			this._oContainerDomRef = null;
 			this._oControl = null;
 		},
 
@@ -210,6 +208,10 @@ sap.ui.define([
 
 		// called after new page loaded
 		_onAfterPageLoaded : function(sChangeReason) {
+			if (!this._oControl) {
+				return;
+			}
+
 			this._bLoading = false;
 			this._updateTriggerDelayed(false);
 			this._oControl.onAfterPageLoaded(this.getInfo(), sChangeReason);
@@ -417,11 +419,16 @@ sap.ui.define([
 
 		// render all the collected items in the chunk and flush them into the DOM
 		// vInsert whether to append (true) or replace (falsy) or to insert at a certain position (int)
-		applyChunk : function(vInsert, oDomRef) {
+		applyChunk : function(vInsert) {
+			if (!this._oControl) {
+				return;
+			}
+
 			this.applyPendingGroupItem();
 
 			var iLength = this._aChunk.length;
-			if (!iLength || !this._oControl.shouldRenderItems()) {
+			var oDomRef = this._oControl.getItemsContainerDomRef();
+			if (!iLength || !oDomRef || !this._oControl.shouldRenderItems()) {
 				return;
 			}
 
@@ -434,17 +441,26 @@ sap.ui.define([
 				}
 			}
 
-			oDomRef = oDomRef || this._oContainerDomRef;
 			this._oRM = this._oRM || sap.ui.getCore().createRenderManager();
-
 			for (var i = 0; i < iLength; i++) {
 				this._oRM.renderControl(this._aChunk[i]);
 			}
 
+			var bHasFocus = (vInsert == false) && oDomRef.contains(document.activeElement);
 			this._oRM.flush(oDomRef, false, this._getDomIndex(vInsert));
+			bHasFocus && this._oControl.focus();
 			this._aChunk = [];
 		},
 
+		// async version of applyChunk
+		applyChunkAsync : function(vInsert) {
+			if (this._bNonClientModel) {
+				setTimeout(this.applyChunk.bind(this, vInsert));
+			} else {
+				this.applyChunk(vInsert);
+			}
+		},
+
 		// add multiple items to the list via BindingContext
 		addListItems : function(aContexts, oBindingInfo, bSuppressInvalidate) {
 			for (var i = 0; i < aContexts.length; i++) {
@@ -457,9 +473,7 @@ sap.ui.define([
 			this.destroyListItems(bSuppressInvalidate);
 			this.addListItems(aContexts, oBindingInfo, bSuppressInvalidate);
 			if (bSuppressInvalidate) {
-				var bHasFocus = this._oContainerDomRef.contains(document.activeElement);
-				this.applyChunk(false);
-				bHasFocus && this._oControl.focus();
+				this.applyChunkAsync(false);
 			} else {
 				this.applyPendingGroupItem();
 			}
@@ -482,6 +496,7 @@ sap.ui.define([
 		 * refresh items only for OData model.
 		 */
 		refreshItems : function(sChangeReason) {
+			this._bNonClientModel = true;
 			if (!this._bDataRequested) {
 				this._bDataRequested = true;
 				this._onBeforePageLoaded(sChangeReason);
@@ -497,9 +512,13 @@ sap.ui.define([
 			var oBinding = this._oControl.getBinding("items");
 
 			if (this._aItemsPool) {
-				oBinding.attachEventOnce("dataRequested", function() {
+				if (this._oControl._bBusy) {
 					setTimeout(this.fillItemsPool.bind(this));
-				}, this);
+				} else {
+					oBinding.attachEventOnce("dataRequested", function() {
+						setTimeout(this.fillItemsPool.bind(this));
+					}, this);
+				}
 			}
 
 			oBinding.getContexts(0, this._iLimit);
@@ -513,7 +532,8 @@ sap.ui.define([
 			var oControl = this._oControl,
 				oBinding = oControl.getBinding("items"),
 				oBindingInfo = oControl.getBindingInfo("items"),
-				aItems = oControl.getItems(true);
+				aItems = oControl.getItems(true),
+				sGroupingPath = this._sGroupingPath;
 
 			// set limit to initial value if not set yet or no items at the control yet
 			if (!this._iLimit || this.shouldReset(sChangeReason) || !aItems.length) {
@@ -545,102 +565,101 @@ sap.ui.define([
 				}
 			}
 
-			// cache dom ref for internal functions not to lookup again and again
-			this._oContainerDomRef = oControl.getItemsContainerDomRef();
+			// let the grouping path to be stored
+			this._sGroupingPath = this._getGroupingPath(oBinding);
 
 			// aContexts.diff ==> undefined : New data we should build from scratch
 			// aContexts.diff ==> [] : There is no diff, means data did not changed at all
 			// aContexts.diff ==> [{index: 0, type: "delete"}, {index: 1, type: "insert"},...] : Run the diff logic
-			var aDiff = aContexts.diff,
-				bFromScratch = false,
-				vInsertIndex;
+			var aDiff = aContexts.diff;
 
 			// process the diff
 			if (!aContexts.length) {
 				// no context, destroy list items
 				this.destroyListItems();
-			} else if (!this._oContainerDomRef) {
+			} else if (!oControl.getItemsContainerDomRef()) {
 				// no dom ref for compatibility reason start from scratch
 				this.rebuildListItems(aContexts, oBindingInfo);
 			} else if (!aDiff || !aItems.length && aDiff.length) {
 				// new records need to be applied from scratch
 				this.rebuildListItems(aContexts, oBindingInfo, oControl.shouldGrowingSuppressInvalidation());
-			} else if (oBinding.isGrouped() || oControl.checkGrowingFromScratch()) {
+			} else {
+				// diff handling case for grouping and merging
+				var bFromScratch = false, vInsertIndex = true;
+				if (oBinding.isGrouped() || oControl.checkGrowingFromScratch()) {
 
-				if (this._sGroupingPath != this._getGroupingPath(oBinding)) {
-					// grouping is changed so we need to rebuild the list for the group headers
-					bFromScratch = true;
-				} else {
-					// append items if possible
-					for (var i = 0; i < aDiff.length; i++) {
-						var oDiff = aDiff[i],
-							oContext = aContexts[oDiff.index];
-
-						if (oDiff.type == "delete" || oDiff.type == "replace") {
-							// group header may need to be deleted as well
-							bFromScratch = true;
-							break;
-						} else if (oDiff.index != this._iRenderedDataItems) {
-							// this item is not appended
-							bFromScratch = true;
-							break;
-						} else {
-							this.addListItem(oContext, oBindingInfo, true);
-							vInsertIndex = true;
+					if (sGroupingPath != this._sGroupingPath) {
+						// grouping is changed so we need to rebuild the list for the group headers
+						bFromScratch = true;
+					} else {
+						// append items if possible
+						for (var i = 0; i < aDiff.length; i++) {
+							var oDiff = aDiff[i],
+								oContext = aContexts[oDiff.index];
+
+							if (oDiff.type == "delete" || oDiff.type == "replace") {
+								// group header may need to be deleted as well
+								bFromScratch = true;
+								break;
+							} else if (oDiff.index != this._iRenderedDataItems) {
+								// this item is not appended
+								bFromScratch = true;
+								break;
+							} else {
+								// the item is appended
+								this.addListItem(oContext, oBindingInfo, true);
+							}
 						}
 					}
-				}
-
-			} else {
 
-				if (this._sGroupingPath) {
-					// if it was already grouped then we need to remove group headers first
-					oControl.removeGroupHeaders(true);
-				}
+				} else {
 
-				vInsertIndex = -1;
-				var iLastInsertIndex = -1;
-				for (var i = 0; i < aDiff.length; i++) {
-					var oDiff = aDiff[i],
-						iDiffIndex = oDiff.index,
-						oContext = aContexts[iDiffIndex];
-
-					if (oDiff.type == "delete") {
-						if (vInsertIndex != -1) {
-							// this record is deleted while the chunk is getting build
-							this.applyChunk(vInsertIndex);
-							iLastInsertIndex = -1;
-							vInsertIndex = -1;
-						}
+					if (sGroupingPath && !this._sGroupingPath) {
+						// if it was already grouped then we need to remove group headers first
+						oControl.removeGroupHeaders(true);
+					}
 
-						this.deleteListItem(iDiffIndex);
-					} else if (oDiff.type == "insert") {
-						if (vInsertIndex == -1) {
-							// the subsequent of items needs to be inserted at this position
-							vInsertIndex = iDiffIndex;
-						} else if (iLastInsertIndex > -1 && iDiffIndex != iLastInsertIndex + 1) {
-							// this item is not simply appended to the last one but has been inserted
-							this.applyChunk(vInsertIndex);
-							vInsertIndex = iDiffIndex;
+					vInsertIndex = -1;
+					var iLastInsertIndex = -1;
+					for (var i = 0; i < aDiff.length; i++) {
+						var oDiff = aDiff[i],
+							iDiffIndex = oDiff.index,
+							oContext = aContexts[iDiffIndex];
+
+						if (oDiff.type == "delete") {
+							if (vInsertIndex != -1) {
+								// this record is deleted while the chunk is getting build
+								this.applyChunk(vInsertIndex);
+								iLastInsertIndex = -1;
+								vInsertIndex = -1;
+							}
+
+							this.deleteListItem(iDiffIndex);
+						} else if (oDiff.type == "insert") {
+							if (vInsertIndex == -1) {
+								// the subsequent of items needs to be inserted at this position
+								vInsertIndex = iDiffIndex;
+							} else if (iLastInsertIndex > -1 && iDiffIndex != iLastInsertIndex + 1) {
+								// this item is not simply appended to the last one but has been inserted
+								this.applyChunk(vInsertIndex);
+								vInsertIndex = iDiffIndex;
+							}
+
+							this.insertListItem(oContext, oBindingInfo, iDiffIndex);
+							iLastInsertIndex = iDiffIndex;
 						}
-
-						this.insertListItem(oContext, oBindingInfo, iDiffIndex);
-						iLastInsertIndex = iDiffIndex;
 					}
 				}
-			}
 
-			if (bFromScratch) {
-				this.rebuildListItems(aContexts, oBindingInfo, true);
-			} else if (this._oContainerDomRef && aDiff) {
-				// set the binding context of items inserting/deleting entries shifts the index of all following items
-				this.updateItemsBindingContext(aContexts, oBindingInfo.model);
-				this.applyChunk(vInsertIndex);
+				if (bFromScratch) {
+					this.rebuildListItems(aContexts, oBindingInfo, true);
+				} else {
+					// set the binding context of items inserting/deleting entries shifts the index of all following items
+					this.updateItemsBindingContext(aContexts, oBindingInfo.model);
+					this.applyChunkAsync(vInsertIndex);
+				}
 			}
 
-			this._oContainerDomRef = null;
-			this._sGroupingPath = this._getGroupingPath(oBinding);
-
 			if (!this._bDataRequested) {
 				this._onAfterPageLoaded(sChangeReason);
 			}

package/src/sap/m/GrowingList.js

@@ -22,7 +22,7 @@ sap.ui.define(['./List', './library', './GrowingListRenderer'],
 	 * @extends sap.m.List
 	 *
 	 * @author SAP SE
-	 * @version 1.101.0
+	 * @version 1.102.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/m/HBox.js

@@ -25,7 +25,7 @@ sap.ui.define(['./FlexBox', './library', './HBoxRenderer'],
 	 * @extends sap.m.FlexBox
 	 *
 	 * @author SAP SE
-	 * @version 1.101.0
+	 * @version 1.102.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/m/HeaderContainer.js

@@ -125,7 +125,7 @@ sap.ui.define([
 		 * @since 1.44.0
 		 *
 		 * @author SAP SE
-		 * @version 1.101.0
+		 * @version 1.102.0
 		 *
 		 * @public
 		 * @alias sap.m.HeaderContainer

package/src/sap/m/HeaderContainerItemNavigator.js

@@ -19,7 +19,7 @@ sap.ui.define([
 	 * @since 1.67.0
 	 *
 	 * @author SAP SE
-	 * @version 1.101.0
+	 * @version 1.102.0
 	 *
 	 * @constructor
 	 * @alias sap.m.HeaderContainerItemNavigator

package/src/sap/m/IconTabBar.js

@@ -94,7 +94,7 @@ sap.ui.define([
 	 * @implements sap.m.ObjectHeaderContainer, sap.f.IDynamicPageStickyContent
 	 *
 	 * @author SAP SE
-	 * @version 1.101.0
+	 * @version 1.102.0
 	 *
 	 * @public
 	 * @alias sap.m.IconTabBar
@@ -361,7 +361,7 @@ sap.ui.define([
 
 		// toggle animation if control is already rendered
 		if (this.$().length) {
-			this._toggleExpandCollapse(bExpanded);
+			this._toggleExpandCollapse(this.getProperty("expanded"));
 		}
 		return this;
 	};

package/src/sap/m/IconTabBarSelectList.js

@@ -38,7 +38,7 @@ sap.ui.define([
 	 * @extends sap.ui.core.Control
 	 *
 	 * @author SAP SE
-	 * @version 1.101.0
+	 * @version 1.102.0
 	 *
 	 * @constructor
 	 * @private

package/src/sap/m/IconTabFilter.js

@@ -94,7 +94,7 @@ sap.ui.define([
 	 * @implements sap.m.IconTab
 	 *
 	 * @author SAP SE
-	 * @version 1.101.0
+	 * @version 1.102.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/m/IconTabFilterExpandButtonBadge.js

@@ -31,7 +31,7 @@ sap.ui.define([
 	 * @extends sap.ui.core.Control
 	 *
 	 * @author SAP SE
-	 * @version 1.101.0
+	 * @version 1.102.0
 	 *
 	 * @constructor
 	 * @private

package/src/sap/m/IconTabHeader.js

@@ -77,7 +77,7 @@ sap.ui.define([
 	 * @extends sap.ui.core.Control
 	 *
 	 * @author SAP SE
-	 * @version 1.101.0
+	 * @version 1.102.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/m/IconTabSeparator.js

@@ -36,7 +36,7 @@ sap.ui.define([
 	 * @implements sap.m.IconTab
 	 *
 	 * @author SAP SE
-	 * @version 1.101.0
+	 * @version 1.102.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/m/IllustratedMessage.js

@@ -82,7 +82,7 @@ sap.ui.define([
 	 * @extends sap.ui.core.Control
 	 *
 	 * @author SAP SE
-	 * @version 1.101.0
+	 * @version 1.102.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/m/Illustration.js

@@ -32,7 +32,7 @@ sap.ui.define([
 	 * @extends sap.ui.core.Control
 	 *
 	 * @author SAP SE
-	 * @version 1.101.0
+	 * @version 1.102.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/m/Image.js

@@ -57,7 +57,7 @@ sap.ui.define([
 	 * @implements sap.ui.core.IFormContent
 	 *
 	 * @author SAP SE
-	 * @version 1.101.0
+	 * @version 1.102.0
 	 *
 	 * @public
 	 * @alias sap.m.Image

package/src/sap/m/ImageContent.js

@@ -33,7 +33,7 @@ sap.ui.define([
 	 * @extends sap.ui.core.Control
 	 *
 	 * @author SAP SE
-	 * @version 1.101.0
+	 * @version 1.102.0
 	 * @since 1.38
 	 *
 	 * @public

package/src/sap/m/Input.js

@@ -150,7 +150,7 @@ function(
 	 *
 	 * @extends sap.m.InputBase
 	 * @author SAP SE
-	 * @version 1.101.0
+	 * @version 1.102.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/m/InputBase.js

@@ -64,7 +64,7 @@ function(
 	 * @implements sap.ui.core.IFormContent
 	 *
 	 * @author SAP SE
-	 * @version 1.101.0
+	 * @version 1.102.0
 	 *
 	 * @constructor
 	 * @public
@@ -438,9 +438,9 @@ function(
 			this._oValueStateMessage.destroy();
 		}
 
-		if (this.oInvisibleMessage) {
-			this.oInvisibleMessage.destroy();
-			this.oInvisibleMessage = null;
+		if (this._oInvisibleMessage) {
+			this._oInvisibleMessage.destroy();
+			this._oInvisibleMessage = null;
 		}
 
 		this._oValueStateMessage = null;
@@ -927,7 +927,7 @@ function(
 	 * @param {boolean} bPrefer True, if the user interaction is prefered
 	 *
 	 * @private
-	 * @restricted sap.ui.mdc
+	 * @ui5-restricted sap.ui.mdc, sap.ui.comp.smartfield.SmartField
 	 */
 	InputBase.prototype._setPreferUserInteraction = function(bPrefer) {
 		this._bPreferUserInteraction = bPrefer;

package/src/sap/m/InputListItem.js

@@ -30,7 +30,7 @@ sap.ui.define([
 	 * @extends sap.m.ListItemBase
 	 *
 	 * @author SAP SE
-	 * @version 1.101.0
+	 * @version 1.102.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/m/Label.js

@@ -76,7 +76,7 @@ function(
 	 * @implements sap.ui.core.Label, sap.ui.core.IShrinkable
 	 *
 	 * @author SAP SE
-	 * @version 1.101.0
+	 * @version 1.102.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/m/LightBox.js

@@ -99,7 +99,7 @@ sap.ui.define([
 	 * @extends sap.ui.core.Control
 	 *
 	 * @author SAP SE
-	 * @version 1.101.0
+	 * @version 1.102.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/m/LightBoxItem.js

@@ -43,7 +43,7 @@ sap.ui.define([
 	 * @extends sap.ui.core.Element
 	 *
 	 * @author SAP SE
-	 * @version 1.101.0
+	 * @version 1.102.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/m/Link.js

@@ -81,7 +81,7 @@ function(
 	 * @implements sap.ui.core.IShrinkable, sap.ui.core.IFormContent, sap.ui.core.ITitleContent
 	 *
 	 * @author SAP SE
-	 * @version 1.101.0
+	 * @version 1.102.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/m/List.js

@@ -30,7 +30,7 @@ sap.ui.define(["./library", "./ListBase", "./ListRenderer"],
 	 * @extends sap.m.ListBase
 	 *
 	 * @author SAP SE
-	 * @version 1.101.0
+	 * @version 1.102.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/m/ListBase.js

@@ -90,7 +90,7 @@ function(
 	 * @extends sap.ui.core.Control
 	 *
 	 * @author SAP SE
-	 * @version 1.101.0
+	 * @version 1.102.0
 	 *
 	 * @constructor
 	 * @public
@@ -1374,12 +1374,6 @@ function(
 			// set the busy state
 			this._bBusy = true;
 
-			// TODO: would be great to have an event when busy indicator visually seen
-			this._sBusyTimer = setTimeout(function() {
-				// clean no data text
-				this.$("nodata-text").text("");
-			}.bind(this), this.getBusyIndicatorDelay());
-
 			// set busy property
 			this.setBusy(true, "listUl");
 		}
@@ -1390,12 +1384,6 @@ function(
 			// revert busy state
 			this._bBusy = false;
 			this.setBusy(false, "listUl");
-			clearTimeout(this._sBusyTimer);
-
-			// revert no data texts when necessary
-			if (!this.getItems(true).length) {
-				this.$("nodata-text").text(this.getNoDataText());
-			}
 		}
 	};
 
@@ -2458,12 +2446,17 @@ function(
 
 	// this gets called when items up arrow key is pressed for the edit keyboard mode
 	ListBase.prototype.onItemArrowUpDown = function(oListItem, oEvent) {
-		var aItems = this.getItems(true),
-			iIndex = aItems.indexOf(oListItem) + (oEvent.type == "sapup" ? -1 : 1),
+		if (this.getKeyboardMode() !== ListKeyboardMode.Edit || oEvent.target instanceof HTMLTextAreaElement) {
+			return;
+		}
+
+		var aItems = this.getVisibleItems(true),
+			iDirection = (oEvent.type == "sapup" || oEvent.type == "sapupmodifiers" ? -1 : 1),
+			iIndex = aItems.indexOf(oListItem) + iDirection,
 			oItem = aItems[iIndex];
 
 		if (oItem && oItem.isGroupHeader()) {
-			oItem = aItems[iIndex + (oEvent.type == "sapup" ? -1 : 1)];
+			oItem = aItems[iIndex + iDirection];
 		}
 
 		if (!oItem) {
@@ -2475,6 +2468,7 @@ function(
 			$Element = $Tabbables.eq($Tabbables[iFocusPos] ? iFocusPos : -1);
 
 		$Element[0] ? $Element.trigger("focus") : oItem.focus();
+		$Element[0] && $Element[0].select && $Element[0].select();
 		oEvent.preventDefault();
 		oEvent.setMarked();
 	};
@@ -2505,6 +2499,13 @@ function(
 	};
 
 	ListBase.prototype.onItemUpDownModifiers = function(oItem, oEvent, iDirection) {
+		if (oEvent.srcControl != oItem) {
+			if (!oEvent.shiftKey && (oEvent.metaKey || oEvent.ctrlKey)) {
+				this.onItemArrowUpDown(oItem, oEvent);
+			}
+			return;
+		}
+
 		if (!this._mRangeSelection) {
 			return;
 		}

package/src/sap/m/ListItemBase.js

@@ -76,7 +76,7 @@ function(
 	 * @extends sap.ui.core.Control
 	 *
 	 * @author SAP SE
-	 * @version 1.101.0
+	 * @version 1.102.0
 	 *
 	 * @constructor
 	 * @public
@@ -1210,7 +1210,7 @@ function(
 	};
 
 	ListItemBase.prototype.onsapupmodifiers = function(oEvent) {
-		if (oEvent.isMarked() || oEvent.srcControl !== this) {
+		if (oEvent.isMarked()) {
 			return;
 		}
 
@@ -1218,7 +1218,7 @@ function(
 	};
 
 	ListItemBase.prototype.onsapdownmodifiers = function(oEvent) {
-		if (oEvent.isMarked() || oEvent.srcControl !== this) {
+		if (oEvent.isMarked()) {
 			return;
 		}
 
@@ -1295,7 +1295,8 @@ function(
 	ListItemBase.prototype.onsapup = function(oEvent) {
 		if (oEvent.isMarked() ||
 			oEvent.srcControl === this ||
-			this.getListProperty("keyboardMode") === ListKeyboardMode.Navigation) {
+			oEvent.target instanceof HTMLInputElement ||
+			oEvent.target instanceof HTMLTextAreaElement) {
 			return;
 		}
 

package/src/sap/m/MaskEnabler.js

@@ -26,7 +26,7 @@ sap.ui.define([
 	 * Applies mask support for input controls.
 	 * It should should be applied to the prototype of a <code>sap.m.InputBase</code>.
 	 *
-	 * @version 1.101.0
+	 * @version 1.102.0
 	 * @private
 	 * @mixin
 	 * @alias sap.m.MaskEnabler

package/src/sap/m/MaskInput.js

@@ -29,7 +29,7 @@ sap.ui.define(['./InputBase', './MaskEnabler', './MaskInputRenderer'], function(
 	 *
 	 * @author SAP SE
 	 * @extends sap.m.InputBase
-	 * @version 1.101.0
+	 * @version 1.102.0
 	 *
 	 * @constructor
 	 * @public