@openui5/sap.m 1.100.0 → 1.102.1

package/src/sap/m/InputBase.js

@@ -64,7 +64,7 @@ function(
 	 * @implements sap.ui.core.IFormContent
 	 *
 	 * @author SAP SE
-	 * @version 1.100.0
+	 * @version 1.102.1
 	 *
 	 * @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.100.0
+	 * @version 1.102.1
 	 *
 	 * @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.100.0
+	 * @version 1.102.1
 	 *
 	 * @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.100.0
+	 * @version 1.102.1
 	 *
 	 * @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.100.0
+	 * @version 1.102.1
 	 *
 	 * @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.100.0
+	 * @version 1.102.1
 	 *
 	 * @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.100.0
+	 * @version 1.102.1
 	 *
 	 * @constructor
 	 * @public

package/src/sap/m/ListBase.js

@@ -90,7 +90,7 @@ function(
 	 * @extends sap.ui.core.Control
 	 *
 	 * @author SAP SE
-	 * @version 1.100.0
+	 * @version 1.102.1
 	 *
 	 * @constructor
 	 * @public
@@ -149,7 +149,9 @@ function(
 			showUnread : {type : "boolean", group : "Misc", defaultValue : false},
 
 			/**
-			 * This text is displayed when the control contains no items.
+			 * This text is displayed if the control contains no items.
+			 * <b>Note:</b> If both a <code>noDataText</code> property and a <code>noData</code> aggregation are provided, the <code>noData</code> aggregation takes priority.
+			 * If the <code>noData</code> aggregation is undefined or set to null, the <code>noDataText</code> property is used instead.
 			 */
 			noDataText : {type : "string", group : "Misc", defaultValue : null},
 
@@ -307,7 +309,15 @@ function(
 			 * Defines the message strip to display binding-related messages.
 			 * @since 1.73
 			 */
-			_messageStrip: {type : "sap.m.MessageStrip", multiple : false, visibility : "hidden"}
+			_messageStrip: {type : "sap.m.MessageStrip", multiple : false, visibility : "hidden"},
+
+			/**
+			 * Defines the custom visualization if there is no data available.
+			 * <b>Note:</b> If both a <code>noDataText</code> property and a <code>noData</code> aggregation are provided, the <code>noData</code> aggregation takes priority.
+			 * If the <code>noData</code> aggregation is undefined or set to null, the <code>noDataText</code> property is used instead.
+			 * @since 1.101
+			 */
+			noData: {type: "sap.ui.core.Control", multiple: false, altTypes: ["string"]}
 		},
 		associations: {
 
@@ -820,9 +830,25 @@ function(
 		return this;
 	};
 
+	ListBase.prototype.setNoData = function (vNoData) {
+		this.setAggregation("noData", vNoData, true);
+
+		if (typeof vNoData === "string") {
+			this.$("nodata-text").text(vNoData);
+		} else if (vNoData) {
+			this.invalidate();
+		} else if (!vNoData) {
+			this.$("nodata-text").text(this.getNoDataText());
+		}
+		return this;
+	};
+
 	ListBase.prototype.setNoDataText = function(sNoDataText) {
 		this.setProperty("noDataText", sNoDataText, true);
-		this.$("nodata-text").text(this.getNoDataText());
+		if (!this.getNoData()) {
+			// only set noDataText, if noData aggregation is not specified
+			this.$("nodata-text").text(this.getNoDataText());
+		}
 		return this;
 	};
 
@@ -1348,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");
 		}
@@ -1364,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());
-			}
 		}
 	};
 
@@ -2403,7 +2417,12 @@ function(
 
 		var oTarget = oEvent.target;
 		if (oTarget.id == this.getId("nodata")) {
-			this.updateInvisibleText(this.getNoDataText(), oTarget);
+			var vNoData = this.getNoData();
+			var sDescription = vNoData || this.getNoDataText();
+			if (vNoData && typeof vNoData !== "string") {
+				sDescription = ListItemBase.getAccessibilityText(vNoData);
+			}
+			this.updateInvisibleText(sDescription, oTarget);
 		}
 
 		// handle only for backward navigation
@@ -2427,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) {
@@ -2444,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();
 	};
@@ -2474,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/ListBaseRenderer.js

@@ -336,13 +336,29 @@ sap.ui.define(["sap/m/library", "sap/ui/Device", "sap/ui/core/InvisibleText", ".
 		ListItemBaseRenderer.addFocusableClasses.call(ListItemBaseRenderer, rm);
 		rm.openEnd();
 
-		rm.openStart("div", oControl.getId("nodata-text")).class("sapMListNoDataText").openEnd();
-		rm.text(oControl.getNoDataText(true));
-		rm.close("div");
+		rm.openStart("div", oControl.getId("nodata-text")).class("sapMListNoDataText");
+
+		var vNoData = oControl.getNoData();
+		if (vNoData && typeof vNoData !== "string") {
+			rm.class("sapMListNoDataContent");
+		}
+		rm.openEnd();
+
+		this.renderNoDataArea(rm, oControl);
 
+		rm.close("div");
 		rm.close("li");
 	};
 
+	ListBaseRenderer.renderNoDataArea = function (rm, oControl) {
+		var vNoData = oControl.getNoData() || oControl.getNoDataText();
+		if (typeof vNoData === "string") {
+			rm.text(vNoData);
+		} else {
+			rm.renderControl(vNoData);
+		}
+	};
+
 	ListBaseRenderer.renderDummyArea = function(rm, oControl, sAreaId, iTabIndex) {
 		rm.openStart("div", oControl.getId(sAreaId)).attr("tabindex", iTabIndex);
 

package/src/sap/m/ListItemBase.js

@@ -76,7 +76,7 @@ function(
 	 * @extends sap.ui.core.Control
 	 *
 	 * @author SAP SE
-	 * @version 1.100.0
+	 * @version 1.102.1
 	 *
 	 * @constructor
 	 * @public
@@ -183,7 +183,8 @@ function(
 			 */
 			detailPress : {}
 		},
-		designtime: "sap/m/designtime/ListItemBase.designtime"
+		designtime: "sap/m/designtime/ListItemBase.designtime",
+		dnd: true
 	}});
 
 	ListItemBase.getAccessibilityText = function(oControl, bDetectEmpty) {
@@ -1209,7 +1210,7 @@ function(
 	};
 
 	ListItemBase.prototype.onsapupmodifiers = function(oEvent) {
-		if (oEvent.isMarked() || oEvent.srcControl !== this) {
+		if (oEvent.isMarked()) {
 			return;
 		}
 
@@ -1217,7 +1218,7 @@ function(
 	};
 
 	ListItemBase.prototype.onsapdownmodifiers = function(oEvent) {
-		if (oEvent.isMarked() || oEvent.srcControl !== this) {
+		if (oEvent.isMarked()) {
 			return;
 		}
 
@@ -1294,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.100.0
+	 * @version 1.102.1
 	 * @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.100.0
+	 * @version 1.102.1
 	 *
 	 * @constructor
 	 * @public

package/src/sap/m/MaskInputRule.js

@@ -20,7 +20,7 @@ sap.ui.define(['sap/ui/core/Element', "sap/base/Log"], function(Element, Log) {
 	 *
 	 * @author SAP SE
 	 * @extends sap.ui.core.Element
-	 * @version 1.100.0
+	 * @version 1.102.1
 	 * @public
 	 * @constructor
 	 * @since 1.34.0

package/src/sap/m/Menu.js

@@ -68,7 +68,7 @@ sap.ui.define([
 		 * @implements sap.ui.core.IContextMenu
 		 *
 		 * @author SAP SE
-		 * @version 1.100.0
+		 * @version 1.102.1
 		 *
 		 * @constructor
 		 * @public

package/src/sap/m/MenuButton.js

@@ -58,7 +58,7 @@ sap.ui.define([
 		 * @extends sap.ui.core.Control
 		 *
 		 * @author SAP SE
-		 * @version 1.100.0
+		 * @version 1.102.1
 		 *
 		 * @constructor
 		 * @public
@@ -100,6 +100,10 @@ sap.ui.define([
 				/**
 				 * Defines the icon to be displayed as a graphical element within the button.
 				 * It can be an image or an icon from the icon font.
+				 *
+				 * Note: If only an icon (without text) is provided when <code>buttonMode</code> is set to <code>Split</code>,
+				 * please provide icons for all menu items. Otherwise the action button will be displayed with no icon or text
+				 * after item selection since there is not enough space for a text.
 				 */
 				icon : {type : "sap.ui.core.URI", group : "Appearance", defaultValue : null},
 

package/src/sap/m/MenuItem.js

@@ -23,7 +23,7 @@ sap.ui.define(['./library', 'sap/ui/core/Item', 'sap/ui/base/ManagedObjectObserv
 		 * @extends sap.ui.core.Item
 		 *
 		 * @author SAP SE
-		 * @version 1.100.0
+		 * @version 1.102.1
 		 *
 		 * @constructor
 		 * @public

package/src/sap/m/MenuListItem.js

@@ -32,7 +32,7 @@ sap.ui.define([
 		 * @extends sap.m.ListItemBase
 		 *
 		 * @author SAP SE
-		 * @version 1.100.0
+		 * @version 1.102.1
 		 *
 		 * @constructor
 		 * @private

package/src/sap/m/MessageBox.js

@@ -82,7 +82,7 @@ sap.ui.define([
 
 			 * </pre>
 			 *
-			 * When using the <code>sap.m.MessageBox</code> in SAP Quartz themes, the breakpoints and layout paddings could be determined by the MessageBox' width. To enable this concept and add responsive paddings to an element of the MessageBox control, you have to add the following classes depending on your use case: <code>sapUiResponsivePadding--header</code>, <code>sapUiResponsivePadding--content</code>, <code>sapUiResponsivePadding--footer</code>.
+			 * When using the <code>sap.m.MessageBox</code> in SAP Quartz and Horizon themes, the breakpoints and layout paddings could be determined by the MessageBox' width. To enable this concept and add responsive paddings to an element of the MessageBox control, you have to add the following classes depending on your use case: <code>sapUiResponsivePadding--header</code>, <code>sapUiResponsivePadding--content</code>, <code>sapUiResponsivePadding--footer</code>.
 			 *
 			 * @namespace
 			 * @alias sap.m.MessageBox

package/src/sap/m/MessageItem.js

@@ -37,7 +37,7 @@ sap.ui.define([
 		 *
 		 * @extends sap.ui.core.Item
 		 * @author SAP SE
-		 * @version 1.100.0
+		 * @version 1.102.1
 		 *
 		 * @constructor
 		 * @public

package/src/sap/m/MessageListItem.js

@@ -30,7 +30,7 @@ sap.ui.define([
 		 * @extends sap.m.StandardListItem
 		 *
 		 * @author SAP SE
-		 * @version 1.100.0
+		 * @version 1.102.1
 		 *
 		 * @constructor
 		 * @private

package/src/sap/m/MessagePage.js

@@ -72,7 +72,7 @@ sap.ui.define([
 		 * @see {@link fiori:https://experience.sap.com/fiori-design-web/message-page/ Message Page}
 		 *
 		 * @extends sap.ui.core.Control
-		 * @version 1.100.0
+		 * @version 1.102.1
 		 *
 		 * @constructor
 		 * @public

package/src/sap/m/MessagePopover.js

@@ -91,7 +91,7 @@ function(
 		 * @extends sap.ui.core.Control
 		 *
 		 * @author SAP SE
-		 * @version 1.100.0
+		 * @version 1.102.1
 		 *
 		 * @constructor
 		 * @public
@@ -152,7 +152,16 @@ function(
 					/**
 					 * A list with message items.
 					 */
-					items: {type: "sap.m.MessageItem", altTypes: ["sap.m.MessagePopoverItem"], multiple: true, singularName: "item"},
+					items: {
+						type: "sap.m.MessageItem",
+						altTypes: ["sap.m.MessagePopoverItem"],
+						multiple: true,
+						singularName: "item",
+						forwarding: {
+							getter: "_getMessageView",
+							aggregation: "items"
+						}
+					},
 
 					/**
 					 * Sets a custom header button.
@@ -415,6 +424,7 @@ function(
 				var sMutation = oChange.mutation;
 				var oItem = oChange.child;
 
+
 				switch (sMutation) {
 
 					case "insert":
@@ -429,7 +439,7 @@ function(
 				}
 			}.bind(this));
 
-			oItemsObserver.observe(this, {
+			oItemsObserver.observe(this._oMessageView, {
 				aggregations: ["items"]
 			});
 		};
@@ -447,39 +457,6 @@ function(
 		 * @private
 		 */
 		MessagePopover.prototype.onBeforeRenderingPopover = function () {
-			// If there is no item's binding given - it should happen automatically in the MessageView
-			// However for backwards compatibility we need to have the same binding on the MessagePopover
-			// TODO: Decide what to do in this case
-			/*if (!this.getBinding("items") && this._oMessageView.getBinding("items")) {
-				this.bindAggregation("items", this._oMessageView.getBindingInfo("items"));
-			}*/
-
-			// Update MV only if 'items' aggregation is changed
-			if (this._oMessageView && this._bItemsChanged) {
-				var items = this.getItems();
-				var sModelName = this.getBindingInfo("items") && this.getBindingInfo("items").model;
-				var that = this;
-
-				this._oMessageView.destroyItems();
-
-				items.forEach(function (item) {
-					// we need to know if the MessagePopover's item was changed so to
-					// update the MessageView's items as well
-					item._updateProperties(function () {
-						that._bItemsChanged = true;
-					});
-
-					// we need to clone the item along with its bindings and aggregations
-					this._oMessageView.addItem(item.clone("", "", {
-						cloneChildren: true,
-						cloneBindings: true
-					}));
-				}, this);
-
-				this._oMessageView.setModel(this.getModel(sModelName), sModelName);
-				this._bItemsChanged = false;
-			}
-
 			this._setInitialFocus();
 
 			// If for some reason the control that opened the popover
@@ -758,6 +735,11 @@ function(
 			}
 		};
 
+		MessagePopover.prototype._getMessageView = function () {
+			return this._oMessageView;
+		};
+
+
 		/*
 		 * =========================================
 		 * MessagePopover async handlers

package/src/sap/m/MessagePopoverItem.js

@@ -19,7 +19,7 @@ sap.ui.define(["./library", "./MessageItem"],
 		 * @extends sap.m.MessageItem
 		 *
 		 * @author SAP SE
-		 * @version 1.100.0
+		 * @version 1.102.1
 		 *
 		 * @constructor
 		 * @public

package/src/sap/m/MessageStrip.js

@@ -83,7 +83,7 @@ sap.ui.define([
 	 * @extends sap.ui.core.Control
 	 *
 	 * @author SAP SE
-	 * @version 1.100.0
+	 * @version 1.102.1
 	 *
 	 * @constructor
 	 * @public

package/src/sap/m/MessageToast.js

@@ -70,7 +70,7 @@ sap.ui.define([
 		 * The message toast has the same behavior on all devices. However, you can adjust the width of the control, for example, for use on a desktop device.
 		 *
 		 * @author SAP SE
-		 * @version 1.100.0
+		 * @version 1.102.1
 		 *
 		 * @namespace
 		 * @public

package/src/sap/m/MessageView.js

@@ -112,7 +112,7 @@ sap.ui.define([
 	 * The responsiveness of the <code>MessageView</code> is determined by the container in which it is embedded. For that reason the control could not be visualized if the
 	 * container’s sizes are not defined.
 	 * @author SAP SE
-	 * @version 1.100.0
+	 * @version 1.102.1
 	 *
 	 * @extends sap.ui.core.Control
 	 * @constructor

package/src/sap/m/MultiComboBox.js

@@ -159,7 +159,7 @@ function(
 	 * </ul>
 	 *
 	 * @author SAP SE
-	 * @version 1.100.0
+	 * @version 1.102.1
 	 *
 	 * @constructor
 	 * @extends sap.m.ComboBoxBase
@@ -3141,7 +3141,8 @@ function(
 			oListItem.addDelegate(this._oListItemEnterEventDelegate, true, this, true);
 
 			// add the mapped item type of sap.m.StandardListItem to the list
-			this._getList().addAggregation("items", oListItem, true);
+			// do not prevent invalidation as invalidations will stack
+			this._getList().addItem(oListItem);
 
 			// add active state to the selected item
 			if (this.isItemSelected(aItems[i])) {

package/src/sap/m/MultiEditField.js

@@ -17,7 +17,7 @@ sap.ui.define(["sap/ui/core/XMLComposite", "./library", "sap/ui/core/Item"], fun
 	 * @implements sap.ui.core.IFormContent
 	 *
 	 * @author SAP SE
-	 * @version 1.100.0
+	 * @version 1.102.1
 	 * @experimental since 1.52
 	 * @since 1.52.0
 	 * @alias sap.m.MultiEditField

package/src/sap/m/MultiInput.js

@@ -121,7 +121,7 @@ function(
 	* @implements sap.ui.core.ISemanticFormContent
 	*
 	* @author SAP SE
-	* @version 1.100.0
+	* @version 1.102.1
 	*
 	* @constructor
 	* @public
@@ -281,7 +281,7 @@ function(
 			.attachAfterClose(this._onAfterCloseTokensPicker.bind(this))
 
 			/* Prevent closing of n more popover when input is clicked */
-			._getPopup().setAutoCloseAreas([oTokenizer, this]);
+			._getPopup().setExtraContent([oTokenizer, this]);
 
 		this.setAggregation("tokenizer", oTokenizer);
 
@@ -855,7 +855,12 @@ function(
 		// for the purpose to copy from column in excel and paste in MultiInput/MultiComboBox
 		sOriginalText = oEvent.originalEvent.clipboardData.getData('text/plain');
 
-		aSeparatedText = sOriginalText.split(/\r\n|\r|\n/g);
+		// Pasting from Excel on Windows always adds "\r\n" at the end, even if a single cell is selected
+		if (sOriginalText.length && sOriginalText.endsWith("\r\n")) {
+			sOriginalText = sOriginalText.substring(0, sOriginalText.lastIndexOf("\r\n"));
+		}
+
+		aSeparatedText = sOriginalText.split(/\r\n|\r|\n|\t/g);
 
 		// if only one piece of text was pasted, we can assume that the user wants to alter it before it is converted into a token
 		// in this case we leave it as plain text input