@openui5/sap.m 1.126.1 → 1.128.0

package/src/sap/m/Toolbar.js

@@ -78,7 +78,7 @@ function(
 	 * @implements sap.ui.core.Toolbar,sap.m.IBar
 	 *
 	 * @author SAP SE
-	 * @version 1.126.1
+	 * @version 1.128.0
 	 *
 	 * @constructor
 	 * @public
@@ -249,6 +249,10 @@ function(
 	 * @returns {true|false|undefined|Object}
 	 */
 	Toolbar.checkShrinkable = function(oControl, sShrinkClass) {
+		if (oControl.isA("sap.ui.core.HTML")) {
+			return;
+		}
+
 		if (oControl instanceof ToolbarSpacer) {
 			return this.isRelativeWidth(oControl.getWidth());
 		}
@@ -333,10 +337,110 @@ function(
 		this._oContentDelegate = {
 			onAfterRendering: this._onAfterContentRendering
 		};
+
+		this._handleKeyNavigationBound =  this._handleKeyNavigation.bind(this);
+
 	};
 
 	Toolbar.prototype.onAfterRendering = function() {
 		this._checkContents();
+
+		//Attach event listened needed for the arrow key navigation
+		if (this.getDomRef()) {
+			this.getDomRef().removeEventListener("keydown", this._handleKeyNavigationBound);
+			this.getDomRef().addEventListener("keydown", this._handleKeyNavigationBound);
+		}
+	};
+
+	Toolbar.prototype._handleKeyNavigation = function(oEvent) {
+		const focusedElement = document.activeElement;
+		const toolbarDom = this.getDomRef();
+		if (toolbarDom.contains(focusedElement)) {
+			if (oEvent.keyCode === KeyCodes.ARROW_RIGHT || oEvent.keyCode === KeyCodes.ARROW_DOWN) {
+				this._moveFocus("forward", oEvent);
+			} else if (oEvent.keyCode === KeyCodes.ARROW_LEFT || oEvent.keyCode === KeyCodes.ARROW_UP) {
+				this._moveFocus("backward", oEvent);
+			}
+		}
+	};
+
+	Toolbar.prototype._moveFocus = function(sDirection, oEvent) {
+		var aFocusableElements = this._getToolbarInteractiveControls(),
+			oActiveElement = Element.getActiveElement(),
+			oActiveDomElement = document.activeElement;
+
+		while (oActiveElement && oActiveElement.getParent() !== this) {
+			oActiveElement = oActiveElement.getParent();
+		}
+
+		var iCurrentIndex = aFocusableElements.indexOf(oActiveElement),
+			iNextIndex = this._calculateNextIndex(sDirection, iCurrentIndex, aFocusableElements.length),
+			bIsFirst = this._isFirst(sDirection, iCurrentIndex),
+			bIsLast = this._isLast(sDirection, iCurrentIndex, aFocusableElements);
+
+			if (this._shouldAllowDefaultBehavior(oActiveElement, oEvent)) {
+				return;
+			}
+
+		// Handle specific behaviour for the input based controls
+		if (this._isInputBasedControl(oActiveDomElement, oActiveElement, oEvent)) {
+            var bIsAtStart = oActiveDomElement.selectionStart === 0,
+                bIsAtEnd = oActiveDomElement.selectionStart === oActiveDomElement.value.length,
+                bTextSelected = oActiveDomElement.selectionStart !== oActiveDomElement.selectionEnd;
+
+            if (bTextSelected || (sDirection === "forward" && !bIsAtEnd) || (sDirection === "backward" && !bIsAtStart)) {
+                return;
+            }
+		}
+
+		if (aFocusableElements[iNextIndex] && !bIsFirst && !bIsLast) {
+			this._focusElement(aFocusableElements[iNextIndex], oEvent);
+		}
+	};
+
+	Toolbar.prototype._isInputBasedControl = function(oActiveDomElement) {
+		return oActiveDomElement.tagName === "INPUT" && !oActiveDomElement.readOnly;
+	};
+
+	Toolbar.prototype._isFirst = function(sDirection, iCurrentIndex) {
+		return (iCurrentIndex === 0) && (sDirection === "backward" || sDirection === "up");
+	};
+
+	Toolbar.prototype._isLast = function(sDirection, iCurrentIndex, aFocusableElements) {
+		return (iCurrentIndex === aFocusableElements.length - 1) && (sDirection === "forward" || sDirection === "down");
+	};
+
+	Toolbar.prototype._shouldAllowDefaultBehavior = function(oActiveElement, oEvent) {
+		var sActiveElementName = oActiveElement.getMetadata().getName(),
+			bIsSelectOrCombobox = ["sap.m.Select", "sap.m.ComboBox"].includes(sActiveElementName),
+			bIsUpOrDownArrowKey = [KeyCodes.ARROW_UP, KeyCodes.ARROW_DOWN].includes(oEvent.keyCode),
+			bIsBreadcrumbs = sActiveElementName === "sap.m.Breadcrumbs";
+
+		if (bIsUpOrDownArrowKey && bIsSelectOrCombobox || bIsBreadcrumbs) {
+			return true;
+		}
+
+		// If the control does not have its own navigation or the conditions are not met, return false
+		return false;
+	};
+
+	Toolbar.prototype._calculateNextIndex = function(sDirection, iCurrentIndex, length) {
+		if (sDirection === "forward") {
+			return (iCurrentIndex + 1) % length;
+		}
+
+		return (iCurrentIndex - 1 + length) % length;
+	};
+
+	Toolbar.prototype._focusElement = function(element, oEvent) {
+		element.focus();
+
+		if (document.activeElement.tagName === 'INPUT') {
+			document.activeElement.select(); // Optionally select text in input field
+		}
+
+		// Prevent the default behavior to avoid any further automatic focus movement
+		oEvent.preventDefault();
 	};
 
 	Toolbar.prototype.onLayoutDataChange = function() {
@@ -374,7 +478,7 @@ function(
 			this.firePress({
 				srcControl : oEvent.srcControl
 			});
-			this.focus();
+			this.focus({preventScroll: true});
 		}
 	};
 
@@ -503,11 +607,21 @@ function(
 	 * @private
 	 */
 	Toolbar.prototype._getToolbarInteractiveControlsCount = function () {
+		return this._getToolbarInteractiveControls().length;
+	};
+
+	/**
+	 *
+	 * @returns {Array} Toolbar interactive Controls
+	 * @private
+	 */
+
+	Toolbar.prototype._getToolbarInteractiveControls = function () {
 		return this.getContent().filter(function (oControl) {
 			return oControl.getVisible()
 				&& oControl.isA("sap.m.IToolbarInteractiveControl")
 				&& typeof (oControl._getToolbarInteractive) === "function" && oControl._getToolbarInteractive();
-		}).length;
+		});
 	};
 
 	Toolbar.prototype._getActiveButton = function() {

package/src/sap/m/ToolbarLayoutData.js

@@ -25,7 +25,7 @@ sap.ui.define(['./library', 'sap/ui/core/LayoutData'],
 	 * <code>layoutData</code> aggregation of {@link sap.m.ToolbarSpacer} and if set it's ignored.
 	 *
 	 * @extends sap.ui.core.LayoutData
-	 * @version 1.126.1
+	 * @version 1.128.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/m/ToolbarRenderer.js

@@ -77,21 +77,25 @@ sap.ui.define(['./BarInPageEnabler'],
 		oRm.style("height", oToolbar.getHeight());
 	};
 
-	ToolbarRenderer.renderBarContent = function(rm, oToolbar) {
+	ToolbarRenderer.renderBarContent = function (rm, oToolbar) {
 		var oFirstVisibleControl = null;
 
 		if (oToolbar.getActive()) {
 			rm.renderControl(oToolbar._getActiveButton());
 		}
-		oToolbar.getContent().forEach(function(oControl) {
-			BarInPageEnabler.addChildClassTo(oControl, oToolbar);
-			if (!oFirstVisibleControl && oControl.getVisible()) {
-				oControl.addStyleClass("sapMBarChildFirstChild");
-				oFirstVisibleControl = oControl;
+		oToolbar.getContent().forEach(function (oControl) {
+			if (oControl.isA("sap.ui.core.HTML")) {
+				rm.renderControl(oControl);
 			} else {
-				oControl.removeStyleClass("sapMBarChildFirstChild");
+				BarInPageEnabler.addChildClassTo(oControl, oToolbar);
+				if (!oFirstVisibleControl && oControl.getVisible()) {
+					oControl.addStyleClass("sapMBarChildFirstChild");
+					oFirstVisibleControl = oControl;
+				} else {
+					oControl.removeStyleClass("sapMBarChildFirstChild");
+				}
+				rm.renderControl(oControl);
 			}
-			rm.renderControl(oControl);
 		});
 	};
 

package/src/sap/m/ToolbarSeparator.js

@@ -23,7 +23,7 @@ sap.ui.define(['./library', 'sap/ui/core/Control', './ToolbarSeparatorRenderer']
 	 * @extends sap.ui.core.Control
 	 *
 	 * @author SAP SE
-	 * @version 1.126.1
+	 * @version 1.128.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/m/ToolbarSpacer.js

@@ -29,7 +29,7 @@ function(library, Control, ToolbarSpacerRenderer, Log) {
 	 * @extends sap.ui.core.Control
 	 *
 	 * @author SAP SE
-	 * @version 1.126.1
+	 * @version 1.128.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/m/Tree.js

@@ -43,7 +43,7 @@ function(
 	 * @extends sap.m.ListBase
 	 *
 	 * @author SAP SE
-	 * @version 1.126.1
+	 * @version 1.128.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/m/TreeItemBase.js

@@ -31,7 +31,7 @@ sap.ui.define([
 	 * @extends sap.m.ListItemBase
 	 *
 	 * @author SAP SE
-	 * @version 1.126.1
+	 * @version 1.128.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/m/UploadCollection.js

@@ -97,7 +97,7 @@ sap.ui.define([
 	 * @extends sap.ui.core.Control
 	 *
 	 * @author SAP SE
-	 * @version 1.126.1
+	 * @version 1.128.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/m/UploadCollectionItem.js

@@ -28,7 +28,7 @@ sap.ui.define([
 	 * @extends sap.ui.core.Element
 	 *
 	 * @author SAP SE
-	 * @version 1.126.1
+	 * @version 1.128.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/m/UploadCollectionParameter.js

@@ -20,7 +20,7 @@ sap.ui.define([
 	 * @extends sap.ui.core.Element
 	 *
 	 * @author SAP SE
-	 * @version 1.126.1
+	 * @version 1.128.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/m/UploadCollectionToolbarPlaceholder.js

@@ -22,7 +22,7 @@ sap.ui.define([
 	 * @extends sap.ui.core.Control
 	 *
 	 * @author SAP SE
-	 * @version 1.126.1
+	 * @version 1.128.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/m/VBox.js

@@ -26,7 +26,7 @@ sap.ui.define(['./FlexBox', './library', "./VBoxRenderer"],
 	 * @extends sap.m.FlexBox
 	 *
 	 * @author SAP SE
-	 * @version 1.126.1
+	 * @version 1.128.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/m/ValueStateHeader.js

@@ -23,7 +23,7 @@ sap.ui.define(
 		 *
 		 * @extends sap.ui.core.Control
 		 * @author SAP SE
-		 * @version 1.126.1
+		 * @version 1.128.0
 		 *
 		 * @constructor
 		 * @private

package/src/sap/m/VariantItem.js

@@ -182,6 +182,7 @@ sap.ui.define([
 
 	VariantItem.prototype.setTitle = function(sValue) {
 		this.setProperty("title", sValue);
+		this.setProperty("text", sValue);
 		if (!this._bOriginalTitleSet) {
 			this._bOriginalTitleSet = true;
 			this._setOriginalTitle(sValue);

package/src/sap/m/ViewSettingsCustomItem.js

@@ -22,7 +22,7 @@ sap.ui.define(['./ViewSettingsItem', './library'],
 	 * @extends sap.m.ViewSettingsItem
 	 *
 	 * @author SAP SE
-	 * @version 1.126.1
+	 * @version 1.128.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/m/ViewSettingsCustomTab.js

@@ -20,7 +20,7 @@ sap.ui.define(['./library', 'sap/ui/core/Item', 'sap/ui/core/IconPool'],
 			 * @extends sap.ui.core.Item
 			 *
 			 * @author SAP SE
-			 * @version 1.126.1
+			 * @version 1.128.0
 			 *
 			 * @constructor
 			 * @public

package/src/sap/m/ViewSettingsDialog.js

@@ -155,7 +155,7 @@ function(
 	 * @extends sap.ui.core.Control
 	 *
 	 * @author SAP SE
-	 * @version 1.126.1
+	 * @version 1.128.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/m/ViewSettingsFilterItem.js

@@ -24,7 +24,7 @@ sap.ui.define(['./ViewSettingsItem', './library'],
 	 * @extends sap.m.ViewSettingsItem
 	 *
 	 * @author SAP SE
-	 * @version 1.126.1
+	 * @version 1.128.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/m/ViewSettingsItem.js

@@ -25,7 +25,7 @@ sap.ui.define(['./library', 'sap/ui/core/Item'],
 	 * @extends sap.ui.core.Item
 	 *
 	 * @author SAP SE
-	 * @version 1.126.1
+	 * @version 1.128.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/m/VisibleItem.js

@@ -21,7 +21,7 @@ sap.ui.define(['./library', 'sap/ui/core/Item', "sap/ui/thirdparty/jquery"],
 		 * @extends sap.ui.core.Item
 		 *
 		 * @author SAP SE
-		 * @version 1.126.1
+		 * @version 1.128.0
 		 *
 		 * @constructor
 		 * @private

package/src/sap/m/WheelSlider.js

@@ -30,7 +30,7 @@ sap.ui.define([
 		 * @extends sap.ui.core.Control
 		 *
 		 * @author SAP SE
-		 * @version 1.126.1
+		 * @version 1.128.0
 		 *
 		 * @constructor
 		 * @public

package/src/sap/m/WheelSliderContainer.js

@@ -31,7 +31,7 @@ sap.ui.define([
 		 * @extends sap.ui.core.Control
 		 *
 		 * @author SAP SE
-		 * @version 1.126.1
+		 * @version 1.128.0
 		 *
 		 * @constructor
 		 * @public

package/src/sap/m/Wizard.js

@@ -104,7 +104,7 @@ sap.ui.define([
 		 *
 		 * @extends sap.ui.core.Control
 		 * @author SAP SE
-		 * @version 1.126.1
+		 * @version 1.128.0
 		 *
 		 * @constructor
 		 * @public

package/src/sap/m/WizardProgressNavigator.js

@@ -45,7 +45,7 @@ function(
 	 * @extends sap.ui.core.Control
 	 *
 	 * @author SAP SE
-	 * @version 1.126.1
+	 * @version 1.128.0
 	 *
 	 * @constructor
 	 * @private

package/src/sap/m/WizardStep.js

@@ -42,7 +42,7 @@ sap.ui.define([
 	 * <li>If the execution needs to branch after a given step, you should set all possible next steps in the <code>subsequentSteps</code> aggregation.
 	 * @extends sap.ui.core.Control
 	 * @author SAP SE
-	 * @version 1.126.1
+	 * @version 1.128.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/m/changeHandler/AddTableColumn.js

@@ -40,7 +40,7 @@ sap.ui.define([
 	 * @constructor
 	 * @alias sap.m.changeHandler.AddTableColumn
 	 * @author SAP SE
-	 * @version 1.126.1
+	 * @version 1.128.0
 	 * @private
 	 * @since 1.51.0
 	 */

package/src/sap/m/changeHandler/ChangeLinkTarget.js

@@ -12,7 +12,7 @@ sap.ui.define(function () {
 	 *
 	 * @alias sap.m.changeHandler.ChangeLinkTarget
 	 * @author SAP SE
-	 * @version 1.126.1
+	 * @version 1.128.0
 	 * @experimental Since 1.71
 	 */
 	var ChangeLinkTarget = {};

package/src/sap/m/changeHandler/CombineButtons.js

@@ -18,7 +18,7 @@ sap.ui.define([
 	 *
 	 * @alias sap.m.changeHandler.CombineButtons
 	 * @author SAP SE
-	 * @version 1.126.1
+	 * @version 1.128.0
 	 * @experimental Since 1.48
 	 */
 	var CombineButtons = {};

package/src/sap/m/changeHandler/MoveTableColumns.js

@@ -13,7 +13,7 @@ sap.ui.define(["sap/base/Log"], function(Log) {
 	 *
 	 * @alias sap.m.changeHandler.MoveTableColumns
 	 * @author SAP SE
-	 * @version 1.126.1
+	 * @version 1.128.0
 	 * @private
 	 * @since 1.48.0
 	 */

package/src/sap/m/changeHandler/SelectIconTabBarFilter.js

@@ -14,7 +14,7 @@ sap.ui.define([
 	 *
 	 * @alias sap.m.changeHandler.SelectIconTabBarFilter
 	 * @author SAP SE
-	 * @version 1.126.1
+	 * @version 1.128.0
 	 * @since 1.96
 	 * @private
 	 */

package/src/sap/m/changeHandler/SplitMenuButton.js

@@ -20,7 +20,7 @@ sap.ui.define([
 	 *
 	 * @alias sap.m.changeHandler.SplitMenuButton
 	 * @author SAP SE
-	 * @version 1.126.1
+	 * @version 1.128.0
 	 * @experimental Since 1.48
 	 */
 	var SplitMenuButton = {};

package/src/sap/m/designtime/messagebundle_en_US_sappsd.properties

@@ -43,9 +43,9 @@ LINK_NAME=[[[\u013B\u012F\u014B\u0137]]]
 
 LINK_NAME_PLURAL=[[[\u013B\u012F\u014B\u0137\u015F\u2219\u2219\u2219\u2219\u2219\u2219\u2219\u2219\u2219]]]
 
-LINK_CHANGE_TARGET=[[[\u0108\u0125\u0105\u014B\u011F\u0113 \u0163\u0105\u0157\u011F\u0113\u0163\u2219\u2219\u2219\u2219\u2219\u2219]]]
+LINK_CHANGE_TARGET=[[[\u0108\u0125\u0105\u014B\u011F\u0113 \u0162\u0105\u0157\u011F\u0113\u0163\u2219\u2219\u2219\u2219\u2219\u2219]]]
 
-LINK_DIALOG_TITLE_CHANGE_TARGET=[[[\u015C\u0113\u013A\u0113\u010B\u0163 \u0163\u0105\u0157\u011F\u0113\u0163 \u014F\u0157 \u0113\u014B\u0163\u0113\u0157 \u0192\u0157\u0105\u0271\u0113\u2219\u2219\u2219\u2219\u2219\u2219\u2219\u2219]]]
+LINK_DIALOG_TITLE_CHANGE_TARGET=[[[\u015C\u0113\u013A\u0113\u010B\u0163 \u0162\u0105\u0157\u011F\u0113\u0163 \u014F\u0157 \u0114\u014B\u0163\u0113\u0157 \u0191\u0157\u0105\u0271\u0113\u2219\u2219\u2219\u2219\u2219\u2219\u2219\u2219]]]
 
 LINK_DIALOG_CANCEL_BTN=[[[\u0108\u0105\u014B\u010B\u0113\u013A\u2219\u2219\u2219\u2219\u2219\u2219\u2219\u2219]]]
 
@@ -95,9 +95,9 @@ COLUMN_NAME=[[[\u0108\u014F\u013A\u0171\u0271\u014B\u2219\u2219\u2219\u2219\u221
 
 COLUMN_NAME_PLURAL=[[[\u0108\u014F\u013A\u0171\u0271\u014B\u015F\u2219\u2219\u2219\u2219\u2219\u2219\u2219]]]
 
-FLEX_BOX_NAME=[[[\u0191\u013A\u0113\u03C7 \u0183\u014F\u03C7\u2219\u2219\u2219\u2219\u2219\u2219]]]
+FLEX_BOX_NAME=[[[\u0191\u013A\u0113\u03C7 \u0181\u014F\u03C7\u2219\u2219\u2219\u2219\u2219\u2219]]]
 
-FLEX_BOX_NAME_PLURAL=[[[\u0191\u013A\u0113\u03C7 \u0183\u014F\u03C7\u0113\u015F\u2219\u2219\u2219\u2219]]]
+FLEX_BOX_NAME_PLURAL=[[[\u0191\u013A\u0113\u03C7 \u0181\u014F\u03C7\u0113\u015F\u2219\u2219\u2219\u2219]]]
 
 MULTIINPUT_NAME=[[[\u039C\u0171\u013A\u0163\u012F \u012C\u014B\u03C1\u0171\u0163\u2219\u2219\u2219\u2219\u2219\u2219\u2219\u2219]]]