@openui5/sap.m 1.138.0 → 1.139.0

package/src/sap/m/FeedInput.js

@@ -7,7 +7,6 @@
 sap.ui.define([
 	"./library",
 	"sap/ui/core/Control",
-	"sap/ui/core/IconPool",
 	"sap/m/TextArea",
 	"sap/m/Button",
 	"./FeedInputRenderer",
@@ -17,10 +16,11 @@ sap.ui.define([
 	"sap/m/Avatar",
 	"sap/m/AvatarShape",
 	"sap/m/AvatarSize",
-	"sap/ui/core/Lib"
+	"sap/ui/core/Lib",
+	"sap/base/Log"
 ],
-	function(library, Control, IconPool, TextArea, Button, FeedInputRenderer, jQuery, URLListValidator, sanitizeHTML0, Avatar,
-		AvatarShape, AvatarSize, CoreLib) {
+	function(library, Control, TextArea, Button, FeedInputRenderer, jQuery, URLListValidator, sanitizeHTML0, Avatar,
+		AvatarShape, AvatarSize, CoreLib, Log) {
 	"use strict";
 
 	// shortcut for sap.m.ButtonType
@@ -41,7 +41,7 @@ sap.ui.define([
 	 * @extends sap.ui.core.Control
 	 *
 	 * @author SAP SE
-	 * @version 1.138.0
+	 * @version 1.139.0
 	 *
 	 * @constructor
 	 * @public
@@ -163,7 +163,15 @@ sap.ui.define([
 				/**
 				 * Defines the inner avatar control.
 				 */
-				_avatar: { type: "sap.m.Avatar", multiple: false, visibility: "hidden" }
+				_avatar: { type: "sap.m.Avatar", multiple: false, visibility: "hidden" },
+				/**
+				 * Defines the actions that are displayed next to the text area. These buttons can be used to trigger actions, such as attaching a file.
+				 * This is a {@link sap.m.Button}
+				 * If the actions are not set, the post button is displayed as the last control in the feed input.
+				 * <b>Note:</b> Only <code>sap.m.Button</code> is supported for this aggregation. If another control is provided, an error is logged and actions are ignored.
+				 * @since 1.139
+				 */
+				actions: {type: "sap.ui.core.Control", multiple: true}
 			},
 			events : {
 
@@ -395,7 +403,7 @@ sap.ui.define([
 	FeedInput.prototype.setValue = function (sValue) {
 		this.setProperty("value", sValue, true);
 		this._getTextArea().setValue(sValue);
-		this._enablePostButton();
+		this.enablePostButton();
 		return this;
 	};
 
@@ -417,7 +425,7 @@ sap.ui.define([
 			}
 		}
 		this._getTextArea().setEnabled(bEnabled);
-		this._enablePostButton();
+		this.enablePostButton();
 		return this;
 	};
 
@@ -448,7 +456,7 @@ sap.ui.define([
 				liveChange : jQuery.proxy(function (oEvt) {
 					var sValue = oEvt.getParameter("value");
 					this.setProperty("value", sValue, true); // update myself without re-rendering
-					this._enablePostButton();
+					this.enablePostButton();
 				}, this)
 			});
 			this._oTextArea.setParent(this);
@@ -487,21 +495,62 @@ sap.ui.define([
 	};
 
 	/**
-	 * Enable post button depending on the current value
+	 * Enables or disables the post button.
+	 * If a boolean parameter is provided, the post button's enabled state is set directly to that value.
+	 * @param {boolean} [bEnabled] - Sets the enabled state of the post button.
 	 */
-	FeedInput.prototype._enablePostButton = function () {
-		var bPostButtonEnabled = this._isControlEnabled();
+	FeedInput.prototype.enablePostButton = function(bEnabled) {
 		var oButton = this._getPostButton();
-		oButton.setEnabled(bPostButtonEnabled);
+		// If bEnabled is a boolean, set the button's enabled state directly
+		// Otherwise, check the control's validity to determine the button's enabled state
+		if (typeof bEnabled === "boolean") {
+			this._bEnablePostButton = bEnabled;
+			oButton.setEnabled(bEnabled);
+			// If the button is disabled and the current value is not empty, enable the button
+			// This ensures that the button is enabled when the value is set but the button is disabled
+			if (bEnabled == false && this.getValue()) {
+				oButton.setEnabled(!bEnabled);
+			}
+		} else {
+			var bPostButtonEnabled = this._isControlEnabled();
+			oButton.setEnabled(bPostButtonEnabled);
+		}
 	};
 
 	/**
-	 * Verifies if the control is enabled or not
-	 * @returns {boolean} True if control is enabled
+	 * Checks if the control is valid based on the enablePostButton flag and the value.
+	 * @param {boolean} bEnablePostButton
+	 * @returns {boolean}
 	 */
 	FeedInput.prototype._isControlEnabled = function() {
 		var sValue = this.getValue();
-		return this.getEnabled() && (typeof sValue === "string" || sValue instanceof String) && sValue.trim().length > 0;
+		if (this._bEnablePostButton) {
+			// If bEnablePostButton is true, always return true regardless of value
+			return true;
+		} else {
+			// If bEnablePostButton is false, only return true if there is a value
+			return (typeof sValue === "string" || sValue instanceof String) && sValue.trim().length > 0;
+		}
+	};
+
+	/**
+	 * Returns the action button if it is a sap.m.Button, otherwise logs an error and returns null.
+	 * @returns {sap.m.Button|null}
+	 */
+	FeedInput.prototype._getActionButtonIfValid = function () {
+		var oActionButton = this.getActions()[0]; // Get the first action button, if any
+		if (oActionButton && oActionButton.isA("sap.m.Button")) {
+			var sText = oActionButton.getText();
+			var sIcon = oActionButton.getIcon();
+			if (sIcon && (!sText || sText.trim() === "")) {
+				return oActionButton;
+			} else {
+				Log.error("FeedInput: The actionButton should only have an icon and no text.");
+			}
+		} else if (oActionButton) {
+			Log.error("FeedInput: The provided actionButton aggregation is not a sap.m.Button instance.");
+		}
+		return;
 	};
 
 	/**

package/src/sap/m/FeedInputRenderer.js

@@ -22,6 +22,7 @@ sap.ui.define(["sap/ui/core/Lib"],
 	 */
 	FeedInputRenderer.render = function (oRm, oControl) {
 		var sMyId = oControl.getId();
+		var oActionButton = oControl._getActionButtonIfValid();
 
 		oRm.openStart("div", oControl);
 		oRm.class("sapMFeedInBase");
@@ -42,9 +43,15 @@ sap.ui.define(["sap/ui/core/Lib"],
 		}
 		oRm.openStart("div", sMyId + "-container");
 		oRm.class("sapMFeedInContainer");
+		if (oActionButton) {
+			oRm.class("sapMFeedInAcitonButtonContainer");
+		}
 		oRm.openEnd();
 		var oTextArea = oControl._getTextArea();
 		oRm.renderControl(oTextArea);
+		if (oActionButton) {
+			oRm.renderControl(oActionButton.addStyleClass("sapMFeedInActionButton"));
+		}
 		oRm.renderControl(oControl._getPostButton());
 		oRm.close("div");
 		oRm.close("div");

package/src/sap/m/FeedListItem.js

@@ -65,7 +65,7 @@ function(
 	 * @extends sap.m.ListItemBase
 	 *
 	 * @author SAP SE
-	 * @version 1.138.0
+	 * @version 1.139.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/m/FeedListItemAction.js

@@ -19,7 +19,7 @@ sap.ui.define(["./ListItemActionBase"],
 	 * @extends sap.m.ListItemActionBase
 	 *
 	 * @author SAP SE
-	 * @version 1.138.0
+	 * @version 1.139.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/m/Fiori20Adapter.js

@@ -28,7 +28,7 @@ sap.ui.define([
 	 *
 	 *
 	 * @class text
-	 * @version 1.138.0
+	 * @version 1.139.0
 	 * @private
 	 * @since 1.38
 	 * @alias HeaderAdapter
@@ -249,7 +249,7 @@ sap.ui.define([
 	 * Constructor for an sap.m.Fiori20Adapter.
 	 *
 	 * @class text
-	 * @version 1.138.0
+	 * @version 1.139.0
 	 * @private
 	 * @since 1.38
 	 * @alias sap.m.Fiori20Adapter

package/src/sap/m/FlexBox.js

@@ -62,7 +62,7 @@ function(
 	 * @extends sap.ui.core.Control
 	 *
 	 * @author SAP SE
-	 * @version 1.138.0
+	 * @version 1.139.0
 	 *
 	 * @public
 	 * @alias sap.m.FlexBox

package/src/sap/m/FlexItemData.js

@@ -24,7 +24,7 @@ sap.ui.define(["./FlexBoxStylingHelper", "./library", "sap/ui/core/LayoutData"],
 	 * @class
 	 * Holds layout data for a FlexBox / HBox / VBox.
 	 * @extends sap.ui.core.LayoutData
-	 * @version 1.138.0
+	 * @version 1.139.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/m/FormattedText.js

@@ -46,7 +46,7 @@ function(
 		 * @class
 		 * The FormattedText control allows the usage of a limited set of tags for inline display of formatted text in HTML format.
 		 * @extends sap.ui.core.Control
-		 * @version 1.138.0
+		 * @version 1.139.0
 		 *
 		 * @constructor
 		 * @public

package/src/sap/m/GenericTag.js

@@ -53,7 +53,7 @@ sap.ui.define([
 	 * @extends sap.ui.core.Control
 	 *
 	 * @author SAP SE
-	 * @version 1.138.0
+	 * @version 1.139.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/m/GenericTile.js

@@ -89,7 +89,7 @@ sap.ui.define([
 	 * @extends sap.ui.core.Control
 	 *
 	 * @author SAP SE
-	 * @version 1.138.0
+	 * @version 1.139.0
 	 * @since 1.34.0
 	 *
 	 * @public

package/src/sap/m/GroupHeaderListItem.js

@@ -37,7 +37,7 @@ sap.ui.define(["sap/ui/core/library", "./library", "./ListItemBase", "./GroupHea
 	 * @implements sap.m.ITableItem
 
 	 * @author SAP SE
-	 * @version 1.138.0
+	 * @version 1.139.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/m/GrowingList.js

@@ -22,7 +22,7 @@ sap.ui.define(['./List', './library', './GrowingListRenderer', 'sap/ui/core/Conf
 	 * @extends sap.m.List
 	 *
 	 * @author SAP SE
-	 * @version 1.138.0
+	 * @version 1.139.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.138.0
+	 * @version 1.139.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.138.0
+		 * @version 1.139.0
 		 *
 		 * @public
 		 * @alias sap.m.HeaderContainer

package/src/sap/m/HeaderContainerItemNavigator.js

@@ -16,7 +16,7 @@ sap.ui.define([
 	 * @since 1.67.0
 	 *
 	 * @author SAP SE
-	 * @version 1.138.0
+	 * @version 1.139.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.138.0
+	 * @version 1.139.0
 	 *
 	 * @public
 	 * @alias sap.m.IconTabBar

package/src/sap/m/IconTabBarSelectList.js

@@ -40,7 +40,7 @@ sap.ui.define([
 	 * @extends sap.ui.core.Control
 	 *
 	 * @author SAP SE
-	 * @version 1.138.0
+	 * @version 1.139.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.138.0
+	 * @version 1.139.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.138.0
+	 * @version 1.139.0
 	 *
 	 * @constructor
 	 * @private

package/src/sap/m/IconTabHeader.js

@@ -85,7 +85,7 @@ sap.ui.define([
 	 * @extends sap.ui.core.Control
 	 *
 	 * @author SAP SE
-	 * @version 1.138.0
+	 * @version 1.139.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/m/IconTabSeparator.js

@@ -33,7 +33,7 @@ sap.ui.define([
 	 * @implements sap.m.IconTab
 	 *
 	 * @author SAP SE
-	 * @version 1.138.0
+	 * @version 1.139.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/m/IllustratedMessage.js

@@ -92,7 +92,7 @@ sap.ui.define([
 	 * @extends sap.ui.core.Control
 	 *
 	 * @author SAP SE
-	 * @version 1.138.0
+	 * @version 1.139.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.138.0
+	 * @version 1.139.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/m/Image.js

@@ -63,7 +63,7 @@ sap.ui.define([
 	 * @borrows sap.ui.core.IFormContent.getFormDoNotAdjustWidth as #getFormDoNotAdjustWidth
 	 *
 	 * @author SAP SE
-	 * @version 1.138.0
+	 * @version 1.139.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.138.0
+	 * @version 1.139.0
 	 * @since 1.38
 	 *
 	 * @public

package/src/sap/m/Input.js

@@ -162,7 +162,7 @@ function(
 	 * @extends sap.m.InputBase
 	 * @implements sap.ui.core.IAccessKeySupport
 	 * @author SAP SE
-	 * @version 1.138.0
+	 * @version 1.139.0
 	 *
 	 * @constructor
 	 * @public
@@ -2014,7 +2014,7 @@ function(
 
 			// update Accessibility text for suggestion
 			this._oInvisibleMessage?.announce(sAriaText, CoreLibrary.InvisibleMessageMode.Polite);
-		}.bind(this), 0);
+		}.bind(this), 100);
 	};
 
 	/**

package/src/sap/m/InputBase.js

@@ -78,7 +78,7 @@ function(
 	 * @borrows sap.ui.core.ILabelable.hasLabelableHTMLElement as #hasLabelableHTMLElement
 	 *
 	 * @author SAP SE
-	 * @version 1.138.0
+	 * @version 1.139.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/m/InputListItem.js

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

package/src/sap/m/Label.js

@@ -78,7 +78,7 @@ function(
 	 * @implements sap.ui.core.Label, sap.ui.core.IShrinkable, sap.ui.core.IAccessKeySupport, sap.ui.core.ILabelable
 	 *
 	 * @author SAP SE
-	 * @version 1.138.0
+	 * @version 1.139.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/m/LightBox.js

@@ -96,7 +96,7 @@ sap.ui.define([
 	 * @extends sap.ui.core.Control
 	 *
 	 * @author SAP SE
-	 * @version 1.138.0
+	 * @version 1.139.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.138.0
+	 * @version 1.139.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/m/Link.js

@@ -101,7 +101,7 @@ function(
 	 * @borrows sap.ui.core.ILabelable.hasLabelableHTMLElement as #hasLabelableHTMLElement
 	 *
 	 * @author SAP SE
-	 * @version 1.138.0
+	 * @version 1.139.0
 	 *
 	 * @constructor
 	 * @public
@@ -483,13 +483,15 @@ function(
 	 */
 	Link.prototype._handlePress = function(oEvent) {
 		var oTarget = oEvent.target,
+			parentElement,
 			bEmptyHref;
 
 		if (this.getEnabled()) {
 			// mark the event for components that needs to know if the event was handled by the link
 			oEvent.setMarked();
 
-			bEmptyHref = (oTarget.classList.contains("sapMLnk") || oTarget.parentElement.classList.contains("sapMLnk")) && (oTarget.getAttribute("href") == "#" || oTarget.parentElement.getAttribute("href") == "#");
+			parentElement = oTarget.parentElement && oTarget.parentElement.classList.contains("sapMLnkText") ? oTarget.parentElement.parentElement : oTarget.parentElement;
+			bEmptyHref = (oTarget.classList.contains("sapMLnk") || parentElement.classList.contains("sapMLnk")) && (oTarget.getAttribute("href") == "#" || parentElement.getAttribute("href") == "#");
 			if (!this.firePress({ctrlKey: !!oEvent.ctrlKey, metaKey: !!oEvent.metaKey}) || bEmptyHref) { // fire event and check return value whether default action should be prevented
 				oEvent.preventDefault();
 			}

package/src/sap/m/LinkTileContent.js

@@ -18,7 +18,7 @@ sap.ui.define([
 	 * @class This element is used within the GenericTile control that generates a combination of an icon and a linkl
 	 * @extends sap.ui.core.Element
 	 * @author SAP SE
-	 * @version 1.138.0
+	 * @version 1.139.0
 	 * @constructor
 	 * @public
 	 * @since 1.120

package/src/sap/m/List.js

@@ -30,7 +30,7 @@ sap.ui.define(["./library", "./ListBase", "./ListRenderer", "sap/ui/core/Lib", "
 	 * @extends sap.m.ListBase
 	 *
 	 * @author SAP SE
-	 * @version 1.138.0
+	 * @version 1.139.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/m/ListBase.js

@@ -105,7 +105,7 @@ function(
 	 * @extends sap.ui.core.Control
 	 *
 	 * @author SAP SE
-	 * @version 1.138.0
+	 * @version 1.139.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/m/ListItemAction.js

@@ -33,7 +33,7 @@ sap.ui.define([
 	 *
 	 * @extends sap.ui.core.ListItemActionBase
 	 * @author SAP SE
-	 * @version 1.138.0
+	 * @version 1.139.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/m/ListItemActionBase.js

@@ -18,7 +18,7 @@ sap.ui.define(["sap/ui/core/Element"], function(Element) {
 	 * @extends sap.ui.core.Element
 	 *
 	 * @author SAP SE
-	 * @version 1.138.0
+	 * @version 1.139.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/m/ListItemBase.js

@@ -66,7 +66,7 @@ function(
 	 * @extends sap.ui.core.Control
 	 *
 	 * @author SAP SE
-	 * @version 1.138.0
+	 * @version 1.139.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/m/ListRenderer.js

@@ -35,7 +35,7 @@ sap.ui.define(["sap/ui/core/Renderer", "./ListBaseRenderer", "sap/ui/core/Invisi
 			aDescribedBy.push(sBaseDescribedBy);
 		}
 
-		return aDescribedBy.length ? aDescribedBy.join(" ") : null;
+		return aDescribedBy.length ? aDescribedBy.join(" ") : undefined;
 	};
 
 	return ListRenderer;

package/src/sap/m/MaskEnabler.js

@@ -28,7 +28,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.138.0
+	 * @version 1.139.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.138.0
+	 * @version 1.139.0
 	 *
 	 * @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.138.0
+	 * @version 1.139.0
 	 * @public
 	 * @constructor
 	 * @since 1.34.0