@openui5/sap.m 1.109.0 → 1.110.0

package/src/sap/m/p13n/Engine.js

@@ -1,6 +1,6 @@
 /*!
  * OpenUI5
- * (c) Copyright 2009-2022 SAP SE or an SAP affiliate company.
+ * (c) Copyright 2009-2023 SAP SE or an SAP affiliate company.
  * Licensed under the Apache License, Version 2.0 - see LICENSE.txt.
  */
 
@@ -50,21 +50,14 @@ sap.ui.define([
 	 * <li>{@link sap.m.p13n.GroupController GroupController}: Used to define a list of groupable properties</li>
 	 * </ul>
 	 *
-	 * The following persistence layers can be chosen for personalization services:
+	 * Can be used in combination with <code>sap.ui.fl.variants.VariantManagement</code> to persist a state in variants using <code>sap.ui.fl</code> capabilities.</li>
 	 *
-	 * <ul>
-	 * <li>{@link sap.m.p13n.modification.FlexModificationHandler FlexModificationHandler}: Used in combination with <code>sap.ui.fl.variants.VariantManagement</code> to persist a state in variants using <code>sap.ui.fl</code> capabilities.</li>
-	 * <li>{@link sap.m.p13n.modification.LocalStorageModificationHandler LocalStorageModificationHandler}: Used to store a personalization state in the local storage</li>
-	 * <li>{@link sap.m.p13n.modification.ModificationHandler ModificationHandler}: Used by default - this handler will not persist a state.</li>
-	 * </ul>
-	 *
-	 * @namespace
+	 * @class
 	 * @alias sap.m.p13n.Engine
-	 * @extends sap.m.p13n.AdaptationProvider
+	 * @extends sap.m.p13n.modules.AdaptationProvider
 	 * @author SAP SE
-	 * @version 1.109.0
+	 * @version 1.110.0
 	 * @public
-	 * @experimental Since 1.104. Please note that the API of this control is not yet finalized!
 	 * @since 1.104
 	 */
 	var Engine = AdaptationProvider.extend("sap.m.p13n.Engine", {
@@ -100,9 +93,10 @@ sap.ui.define([
 	 * @property {sap.m.p13n.ModificationHandler} modification The desired <code>{@link sap.m.p13n.modification.ModificationHandler ModificationHandler}</code> instance that is used for persistence.
 	 * @property {sap.m.p13n.MetadataHelper} helper The <code>{@link sap.m.p13n.modification.MetadataHelper MetadataHelper}</code> to provide metadata-specific information.
 	 * @property {object} controller A map of arbitrary keys that contain a controller instance as value. The key must be unique and needs to be provided for later access when using <code>Engine</code> functionality specific for one controller type.
-	 *
+	 */
+
+	/**
 	 * @public
-	 * @experimental Since 1.104. Please note that the API of this control is not yet finalized!
 	 *
 	 * @param {sap.ui.core.Control} oControl The control instance to be registered for adaptation
 	 * @param {sap.m.p13n.EngineRegistrationConfig} oConfig The Engine registration configuration
@@ -171,7 +165,6 @@ sap.ui.define([
 	 * such as the registered controllers, are destroyed.
 	 *
 	 * @public
-	 * @experimental Since 1.104. Please note that the API of this control is not yet finalized!
 	 *
 	 * @param {sap.ui.core.Control} oControl The registered control instance
 	 */
@@ -198,7 +191,6 @@ sap.ui.define([
 	 * Opens the personalization dialog.
 	 *
 	 * @public
-	 * @experimental Since 1.104. Please note that the API of this control is not yet finalized!
 	 *
 	 *
 	 * @param {sap.ui.core.Control} oControl The control instance that is personalized
@@ -221,7 +213,6 @@ sap.ui.define([
 	 * The event handler is fired every time a user triggers a personalization change for a control instance during runtime.
 	 *
 	 * @public
-	 * @experimental Since 1.104. Please note that the API of this control is not yet finalized!
 	 *
 	 * @param {function} fnStateEventHandler The handler function to call when the event occurs
 	 * @returns {this} Returns <code>this</code> to allow method chaining
@@ -235,7 +226,6 @@ sap.ui.define([
 	 * The passed parameters must match those used for registration with {@link sap.m.p13n.Engine#attachStateChange} beforehand.
 	 *
 	 * @public
-	 * @experimental Since 1.104. Please note that the API of this control is not yet finalized!
 	 *
 	 * @param {function} fnStateEventHandler The handler function to detach from the event
 	 * @returns {this} Returns <code>this</code> to allow method chaining
@@ -248,7 +238,6 @@ sap.ui.define([
 	 * This method can be used to trigger a reset to the provided control instance.
 	 *
 	 * @public
-	 * @experimental Since 1.104. Please note that the API of this control is not yet finalized!
 	 *
 	 * @param {sap.ui.core.Control} oControl The related control instance
 	 * @param {string} aKeys The key for the affected configuration
@@ -270,6 +259,14 @@ sap.ui.define([
 			selector: oControl
 		};
 
+		if (aKeys) {
+			var aChangeTypes = [];
+			aKeys.forEach(function(sKey){
+				aChangeTypes = aChangeTypes.concat(Object.values(this.getController(oControl, sKey).getChangeOperations()));
+			}.bind(this));
+			oResetConfig.changeTypes = [].concat.apply([], aChangeTypes);
+		}
+
 		var oModificationSetting = this._determineModification(oControl);
 		return oModificationSetting.handler.reset(oResetConfig, oModificationSetting.payload).then(function(){
 			this.stateHandlerRegistry.fireChange(oControl);
@@ -277,7 +274,9 @@ sap.ui.define([
 			return this.initAdaptation(oControl, aKeys).then(function(oPropertyHelper){
 				aKeys.forEach(function(sKey){
 					var oController = this.getController(oControl, sKey);
-					oController.update(oPropertyHelper);
+					if (this.hasActiveP13n(oControl)) {
+						oController.update(oPropertyHelper);
+					}
 				}.bind(this));
 			}.bind(this));
 		}.bind(this));
@@ -297,11 +296,10 @@ sap.ui.define([
 	 *
 	 * @param {sap.ui.core.Control} oControl The registered control instance
 	 * @param {object} oState The state object
-	 * @param {boolean} bApplyAbsolute Defines whether the state should be an additional delta of the current control state
 	 *
 	 * @returns {Promise} A Promise resolving after the state has been applied
 	 */
-	 Engine.prototype.applyState = function(oControl, oState, bApplyAbsolute) {
+	 Engine.prototype.applyState = function(oControl, oState) {
 
 		//Call retrieve only to ensure that the control is initialized and enabled for modification
 		return this.retrieveState(oControl).then(function(oCurrentState){
@@ -322,7 +320,8 @@ sap.ui.define([
 				var oController = this.getController(oControl, sControllerKey);
 
 				if (!oController){
-					//TODO: p13nMode <> register <> StateUtil key alignment
+					//In case no controller can be found, skip change creation & appliance
+					//to avoid errors and react gracefully
 					return;
 				}
 
@@ -331,7 +330,7 @@ sap.ui.define([
 					key: sControllerKey,
 					state: oController.sanityCheck(oState[sControllerKey]),
 					suppressAppliance: true,
-					applyAbsolute: bApplyAbsolute
+					applyAbsolute: false
 				});
 
 				aStatePromise.push(oStatePromise);
@@ -449,7 +448,7 @@ sap.ui.define([
 		var bSuppressCallback = !!mDiffParameters.suppressAppliance;
 
 		if (!sKey || !mDiffParameters.control || !vNewState) {
-			throw new Error("To create changes via Engine, atleast a 1)Control 2)Key and 3)State needs to be provided.");
+			return Promise.resolve([]);
 		}
 
 		var fDeltaHandling = function() {
@@ -573,7 +572,7 @@ sap.ui.define([
 	 * This method can be used in the designtime metadata of the control
 	 * for key user personalization.
 	 *
-	 * @ui5-restricted
+	 * @ui5-restricted sap.ui.mdc
 	 *
 	 * @param {sap.ui.core.Control} oControl The registered control instance
 	 * @param {object} mPropertyBag The property bag provided in the settings action
@@ -1201,24 +1200,28 @@ sap.ui.define([
 
 			var oController = this.getController(oControl, sControllerKey);
 
-			var p = this.createChanges({
-				control: oControl,
-				key: sControllerKey,
-				state: oController.getP13nData(),
-				suppressAppliance: true,
-				applyAbsolute: true
-			})
-			.then(function(aItemChanges){
+			var vP13nData = oController.getP13nData();
+			if (vP13nData) {
+				var p = this.createChanges({
+					control: oControl,
+					key: sControllerKey,
+					state: vP13nData,
+					suppressAppliance: true,
+					applyAbsolute: true
+				})
+				.then(function(aItemChanges){
 
-				return oController.getBeforeApply().then(function(aChanges){
+					return oController.getBeforeApply().then(function(aChanges){
 
-					var aComulatedChanges = aChanges ? aChanges.concat(aItemChanges) : aItemChanges;
-					return aComulatedChanges;
+						var aComulatedChanges = aChanges ? aChanges.concat(aItemChanges) : aItemChanges;
+						return aComulatedChanges;
 
+					});
 				});
-			});
 
-			pChanges.push(p);
+				pChanges.push(p);
+			}
+
 		}.bind(this));
 
 		return Promise.all(pChanges).then(function(aChangeMatrix){
@@ -1239,10 +1242,11 @@ sap.ui.define([
 	};
 
 	/**
+	 * This method is the central point of access to the Engine Singleton.
+	 *
 	 * @private
 	 * @ui5-restricted sap.m
 	 *
-	 * This method is the central point of access to the Engine Singleton.
 	 * @returns {sap.m.p13n.Engine} The Engine instance
 	 */
 	Engine.getInstance = function() {

package/src/sap/m/p13n/FlexUtil.js

@@ -1,6 +1,6 @@
 /*!
  * OpenUI5
- * (c) Copyright 2009-2022 SAP SE or an SAP affiliate company.
+ * (c) Copyright 2009-2023 SAP SE or an SAP affiliate company.
  * Licensed under the Apache License, Version 2.0 - see LICENSE.txt.
  */
 sap.ui.define([

package/src/sap/m/p13n/GroupController.js

@@ -1,6 +1,6 @@
 /*!
  * OpenUI5
- * (c) Copyright 2009-2022 SAP SE or an SAP affiliate company.
+ * (c) Copyright 2009-2023 SAP SE or an SAP affiliate company.
  * Licensed under the Apache License, Version 2.0 - see LICENSE.txt.
  */
 sap.ui.define([
@@ -21,9 +21,8 @@ sap.ui.define([
 	 * @extends sap.ui.base.Object
 	 *
 	 * @author SAP SE
-	 * @version 1.109.0
+	 * @version 1.110.0
 	 * @public
-	 * @experimental
 	 * @since 1.104
 	 * @alias sap.m.p13n.GroupController
 	 */

package/src/sap/m/p13n/GroupPanel.js

@@ -1,6 +1,6 @@
 /*!
  * OpenUI5
- * (c) Copyright 2009-2022 SAP SE or an SAP affiliate company.
+ * (c) Copyright 2009-2023 SAP SE or an SAP affiliate company.
  * Licensed under the Apache License, Version 2.0 - see LICENSE.txt.
  */
 sap.ui.define([
@@ -21,10 +21,9 @@ sap.ui.define([
 	 * @extends sap.m.p13n.QueryPanel
 	 *
 	 * @author SAP SE
-	 * @version 1.109.0
+	 * @version 1.110.0
 	 *
 	 * @public
-	 * @experimental Since 1.104. Please note that the API of this control is not yet finalized!
 	 * @alias sap.m.p13n.GroupPanel
 	 */
 	var GroupPanel = QueryPanel.extend("sap.m.p13n.GroupPanel", {
@@ -59,7 +58,6 @@ sap.ui.define([
 	/**
 	 * P13n <code>GroupItem</code> object type.
 	 *
-	 * @type {sap.m.p13n.GroupItem}
 	 * @static
 	 * @constant
 	 * @typedef {object} sap.m.p13n.GroupItem

package/src/sap/m/p13n/MetadataHelper.js

@@ -1,6 +1,6 @@
 /*!
  * OpenUI5
- * (c) Copyright 2009-2022 SAP SE or an SAP affiliate company.
+ * (c) Copyright 2009-2023 SAP SE or an SAP affiliate company.
  * Licensed under the Apache License, Version 2.0 - see LICENSE.txt.
  */
 
@@ -12,9 +12,9 @@ sap.ui.define([
     /**
      * The <code>MetadataHelper</code> entity offers utility functionality for providing service metadata during the <code>Engine#register</code> process.
      *
+     * @class
 	 * @author SAP SE
 	 * @public
-     * @experimental Since 1.104.
 	 * @alias sap.m.p13n.MetadataHelper
      */
     var MetadataHelper = BaseObject.extend("sap.m.p13n.MetadataHelper", {

package/src/sap/m/p13n/PersistenceProvider.js

@@ -1,6 +1,6 @@
 /*!
  * OpenUI5
- * (c) Copyright 2009-2022 SAP SE or an SAP affiliate company.
+ * (c) Copyright 2009-2023 SAP SE or an SAP affiliate company.
  * Licensed under the Apache License, Version 2.0 - see LICENSE.txt.
  */
 sap.ui.define([
@@ -14,9 +14,9 @@ sap.ui.define([
 	 * since any control that is a direct or indirect descendant of the provided <code>for</code> association is affected by this configuration.
 	 * For example, this controller can be used for <code>sap.ui.mdc</code> controls.
 	 *
-	 * @private
+	 * @class
+	 * @ui5-restricted sap.ui.mdc, sap.fe
 	 *
-	 * @experimental
 	 * @since 1.104
 	*/
 	var PersistenceProvider = CoreControl.extend("sap.m.p13n.PersistenceProvider", /** @lends sap.ui.mdc.p13n.PersistenceProvider.prototype */ {
@@ -67,6 +67,7 @@ sap.ui.define([
 
 		var oModel = this.getModel(ControlVariantApplyAPI.getVariantModelName());
 		if (oModel) {
+			this.reinitialize();
 			this._fnResolveModel(oModel);
 		}
 	};
@@ -140,6 +141,21 @@ sap.ui.define([
 		return this;
 	};
 
+	/**
+	 * This method reinitializes the inner <code>VariantManagement</code> control be providing the
+	 * variant model and triggering a reinitialize on the inner VM in the static area
+	 *
+	 * @ui5-restricted sap.m.p13n
+	 */
+	PersistenceProvider.prototype.reinitialize = function () {
+		var oVM = sap.ui.getCore().byId(this.getId() + "--vm");
+		if (this.getMode() === mode.Transient && oVM) {
+			var oVariantModel = this.getModel(ControlVariantApplyAPI.getVariantModelName());
+			oVM.setModel(oVariantModel, ControlVariantApplyAPI.getVariantModelName());
+			oVM.reinitialize();
+		}
+	};
+
 	PersistenceProvider.prototype.exit = function () {
 		if (this._oWrapper) {
 			var oStatic = sap.ui.getCore().getUIArea(sap.ui.getCore().getStaticAreaRef());

package/src/sap/m/p13n/Popup.js

@@ -1,6 +1,6 @@
 /*!
  * OpenUI5
- * (c) Copyright 2009-2022 SAP SE or an SAP affiliate company.
+ * (c) Copyright 2009-2023 SAP SE or an SAP affiliate company.
  * Licensed under the Apache License, Version 2.0 - see LICENSE.txt.
  */
 sap.ui.define([
@@ -30,14 +30,12 @@ sap.ui.define([
 	 * @class
 	 * This control can be used to show personalization-related content in different popup controls.
 	 *
-	 * @class
 	 * @extends sap.ui.core.Control
 	 *
 	 * @author SAP SE
-	 * @version 1.109.0
+	 * @version 1.110.0
 	 *
 	 * @public
-	 * @experimental Since 1.97.
 	 * @since 1.97
 	 * @alias sap.m.p13n.Popup
 	 */
@@ -287,7 +285,19 @@ sap.ui.define([
 		var aPanels = this.getPanels();
 		var bUseContainer = aPanels.length > 1;
 		var oResourceBundle = sap.ui.getCore().getLibraryResourceBundle("sap.m");
+
+		var oInitialFocusedControl;
+		if (aPanels.length > 0) {
+			var oContent = aPanels[0];
+			oInitialFocusedControl = oContent.getInitialFocusedControl && oContent.getInitialFocusedControl();
+			if (!oInitialFocusedControl && bUseContainer) {
+				// focus at least the iconTabBar first item
+				oInitialFocusedControl = this._getContainer()._getTabBar().getItems()[0];
+			}
+		}
+
 		var oContainer = new Dialog(this.getId() + "-dialog", {
+			initialFocus: oInitialFocusedControl,
 			title: this.getTitle(),
 			horizontalScrolling: mDialogSettings.hasOwnProperty("horizontalScrolling") ? mDialogSettings.horizontalScrolling : false,
 			verticalScrolling: !bUseContainer,

package/src/sap/m/p13n/QueryPanel.js

@@ -1,6 +1,6 @@
 /*!
  * OpenUI5
- * (c) Copyright 2009-2022 SAP SE or an SAP affiliate company.
+ * (c) Copyright 2009-2023 SAP SE or an SAP affiliate company.
  * Licensed under the Apache License, Version 2.0 - see LICENSE.txt.
  */
 sap.ui.define([
@@ -20,15 +20,14 @@ sap.ui.define([
 	 * @class
 	 * This control serves as base class for a query builder like personalization implementation.
 	 *
-	 * @class
 	 * @extends sap.m.p13n.BasePanel
 	 *
 	 * @author SAP SE
-	 * @version 1.109.0
+	 * @version 1.110.0
 	 *
 	 * @private
-	 * @ui5-restricted
-	 * @experimental Since 1.96.
+	 * @ui5-restricted sap.m, sap.ui.mdc
+	 *
 	 * @since 1.96
 	 * @alias sap.m.p13n.QueryPanel
 	 */
@@ -85,7 +84,7 @@ sap.ui.define([
 	QueryPanel.prototype.getP13nData = function (bOnlyActive) {
 		var aItems = [];
 		this._oListControl.getItems().forEach(function (oItem) {
-			var sKey = oItem.getContent()[0].getContent()[0]._key;
+			var sKey = this._getControlFromRow(oItem)._key;
 			if (sKey) {
 				var oField = this._getP13nModel().getProperty("/items").find(function (o) {
 					return o.name == sKey;
@@ -152,7 +151,7 @@ sap.ui.define([
 	};
 
 	QueryPanel.prototype._getModelEntry = function(oRow) {
-		var sKey = oRow.getContent()[0].getContent()[0]._key;
+		var sKey = this._getControlFromRow(oRow)._key;
 		var oField = this._getP13nModel().getProperty("/items").find(function (o) {
 			return o.name == sKey;
 		});
@@ -218,7 +217,7 @@ sap.ui.define([
 			this._addHover(oRow);
 		}
 
-		oRow.getContent()[0].getContent()[0]._key = oItem.name;
+		this._getControlFromRow(oRow)._key = oItem.name;
 
 		this._oListControl.addItem(oRow);
 
@@ -243,8 +242,7 @@ sap.ui.define([
 	QueryPanel.prototype._handleActivated = function(oHoveredItem) {
 		var oQueryRow = oHoveredItem.getContent()[0];
 		if (oQueryRow) {
-			var iItemLength = oQueryRow.getContent().length - 1;
-			var oButtonBox = oHoveredItem.getContent()[0].getContent()[iItemLength];
+			var oButtonBox = this._getControlFromRow(oHoveredItem, -1);
 
 			//Only add the buttons if 1) an hovered item is provided 2) the buttons are not already there
 			if (oHoveredItem && oButtonBox.getItems().length < 2) {
@@ -311,8 +309,7 @@ sap.ui.define([
 		var oListItem = oComboBox.getParent().getParent();
 		var bIsLastRow = this._oListControl.getItems().length - 1 == this._oListControl.getItems().indexOf(oListItem);
 
-		var aContent = oListItem.getContent()[0].getContent();
-		var oBtnContainer = aContent[aContent.length - 1];
+		var oBtnContainer = this._getControlFromRow(oListItem, -1);
 		oBtnContainer.setVisible(!(bIsLastRow && sNewKey == ""));
 
 		//Remove previous
@@ -350,14 +347,12 @@ sap.ui.define([
 						var bNewRowRequired = iQueries === 1 || iQueries == this.getP13nData(true).length;
 
 						this._oListControl.removeItem(oRow);
-						this._updatePresence(oRow.getContent()[0].getContent()[0]._key, false, undefined);
+						this._updatePresence(this._getControlFromRow(oRow)._key, false, undefined);
 						if (bNewRowRequired) {
 							this._addQueryRow();
-						} else {
-							//In case an item has been removed, focus the Select control of the new 'none' row
-							var iLastIndex = this._oListControl.getItems().length - 1;
-							this._oListControl.getItems()[iLastIndex].getContent()[0].getContent()[0].focus();
 						}
+						//In case an item has been removed, focus the Select control of the new 'none' row
+						this.getInitialFocusedControl().focus();
 
 						this._getP13nModel().checkUpdate(true);
 					}.bind(this)
@@ -395,6 +390,31 @@ sap.ui.define([
 		});
 	};
 
+	QueryPanel.prototype.getInitialFocusedControl = function() {
+		var oRow = this._getRow(-1);
+		return this._getControlFromRow(oRow);
+	};
+
+	QueryPanel.prototype._getRow = function(iIndex) {
+		var aItems = this._oListControl.getItems();
+		if (iIndex < 0) {
+			iIndex = aItems.length + iIndex;
+		}
+		return aItems[iIndex];
+	};
+
+	QueryPanel.prototype._getControlFromRow = function(oRow, iIndex) {
+		var aContent = oRow.getContent()[0].getContent();
+		if (iIndex === undefined) {
+			iIndex = 0;
+		}
+		if (iIndex < 0) {
+			iIndex = aContent.length + iIndex;
+		}
+		return aContent[iIndex];
+	};
+
+
 	return QueryPanel;
 
 });

package/src/sap/m/p13n/SelectionController.js

@@ -1,6 +1,6 @@
 /*!
  * OpenUI5
- * (c) Copyright 2009-2022 SAP SE or an SAP affiliate company.
+ * (c) Copyright 2009-2023 SAP SE or an SAP affiliate company.
  * Licensed under the Apache License, Version 2.0 - see LICENSE.txt.
  */
 
@@ -29,10 +29,9 @@ sap.ui.define([
 	 * @extends sap.ui.base.Object
 	 *
 	 * @author SAP SE
-	 * @version 1.109.0
+	 * @version 1.110.0
 	 *
 	 * @public
-	 * @experimental Since 1.104. Please note that the API of this control is not yet finalized!
 	 * @alias sap.m.p13n.SelectionController
 	 */
 	var SelectionController = BaseObject.extend("sap.m.p13n.SelectionController",{

package/src/sap/m/p13n/SelectionPanel.js

@@ -1,6 +1,6 @@
 /*!
  * OpenUI5
- * (c) Copyright 2009-2022 SAP SE or an SAP affiliate company.
+ * (c) Copyright 2009-2023 SAP SE or an SAP affiliate company.
  * Licensed under the Apache License, Version 2.0 - see LICENSE.txt.
  */
 sap.ui.define([
@@ -48,10 +48,9 @@ sap.ui.define([
 	 * @extends sap.m.p13n.BasePanel
 	 *
 	 * @author SAP SE
-	 * @version 1.109.0
+	 * @version 1.110.0
 	 *
 	 * @public
-	 * @experimental Since 1.96.
 	 * @since 1.96
 	 * @alias sap.m.p13n.SelectionPanel
 	 */
@@ -100,6 +99,9 @@ sap.ui.define([
 				/**
 				 * An optional callback that may be used to display additional custom content in each selectable item.
 				 * This factory can be toggled by executing the {@link sap.m.p13n.SelectionPanel#showFactory} method.
+				 *
+				 * <b>Note:</b>: The <code>getIdForLabel</code> method can be imlplemented on the returned control instance
+				 * to return a focusable children control to provide the <code>labelFor</code> reference for the associated text.
 				 */
 				itemFactory: {
 					type: "function"
@@ -132,7 +134,12 @@ sap.ui.define([
 	SelectionPanel.prototype._getListTemplate = function() {
 		var oColumnListItem = new ColumnListItem({
 			selected: "{" + this.P13N_MODEL + ">" + this.PRESENCE_ATTRIBUTE + "}",
-			type: ListType.Active,
+			type: {
+				path: this.P13N_MODEL + ">" + this.PRESENCE_ATTRIBUTE,
+				formatter: function(bSelected) {
+					return bSelected ? ListType.Active : ListType.Inactive;
+				}
+			},
 			cells: [
 				new VBox({
 					items: [
@@ -256,7 +263,9 @@ sap.ui.define([
 		}
 		//Store (new) hovered item and set its icon to visible: false + add move buttons to it
 		var oIcon = oHoveredItem.getCells()[1].getItems()[0];
-		oIcon.setVisible(false);
+		if (oHoveredItem.getSelected()) {
+			oIcon.setVisible(false);
+		}
 		this._oHoveredItem = oHoveredItem;
 		this._updateEnableOfMoveButtons(oHoveredItem, false);
 		this._addMoveButtons(oHoveredItem);

package/src/sap/m/p13n/SortController.js

@@ -1,6 +1,6 @@
 /*!
  * OpenUI5
- * (c) Copyright 2009-2022 SAP SE or an SAP affiliate company.
+ * (c) Copyright 2009-2023 SAP SE or an SAP affiliate company.
  * Licensed under the Apache License, Version 2.0 - see LICENSE.txt.
  */
 
@@ -19,12 +19,11 @@ sap.ui.define([
 	 * @class
 	 * The <code>SortController</code> entity serves as a base class to create personalization implementations that are specific to sorting.
 	 *
-	 * @extends sap.ui.base.Object
+	 * @extends sap.m.p13n.SelectionController
 	 *
 	 * @author SAP SE
-	 * @version 1.109.0
+	 * @version 1.110.0
 	 * @public
-	 * @experimental Since 1.104. Please note that the API of this control is not yet finalized!
 	 * @alias sap.m.p13n.SortController
 	 */
     var SortController = BaseController.extend("sap.m.p13n.SortController", {

package/src/sap/m/p13n/SortPanel.js

@@ -1,6 +1,6 @@
 /*!
  * OpenUI5
- * (c) Copyright 2009-2022 SAP SE or an SAP affiliate company.
+ * (c) Copyright 2009-2023 SAP SE or an SAP affiliate company.
  * Licensed under the Apache License, Version 2.0 - see LICENSE.txt.
  */
 sap.ui.define([
@@ -21,10 +21,9 @@ sap.ui.define([
 	 * @extends sap.m.p13n.QueryPanel
 	 *
 	 * @author SAP SE
-	 * @version 1.109.0
+	 * @version 1.110.0
 	 *
 	 * @public
-	 * @experimental Since 1.96.
 	 * @since 1.96
 	 * @alias sap.m.p13n.SortPanel
 	 */
@@ -54,7 +53,6 @@ sap.ui.define([
 	/**
 	 * P13n <code>SortItem</code> object type.
 	 *
-	 * @type {sap.m.p13n.SortItem}
 	 * @static
 	 * @constant
 	 * @typedef {object} sap.m.p13n.SortItem

package/src/sap/m/p13n/enum/PersistenceMode.js

@@ -1,6 +1,6 @@
 /*
  * OpenUI5
- * (c) Copyright 2009-2022 SAP SE or an SAP affiliate company.
+ * (c) Copyright 2009-2023 SAP SE or an SAP affiliate company.
  * Licensed under the Apache License, Version 2.0 - see LICENSE.txt.
  */
 sap.ui.define(function () {

package/src/sap/m/p13n/enum/ProcessingStrategy.js

@@ -1,6 +1,6 @@
 /*!
  * OpenUI5
- * (c) Copyright 2009-2022 SAP SE or an SAP affiliate company.
+ * (c) Copyright 2009-2023 SAP SE or an SAP affiliate company.
  * Licensed under the Apache License, Version 2.0 - see LICENSE.txt.
  */
 

package/src/sap/m/p13n/handler/xConfigHandler.js

@@ -1,6 +1,6 @@
 /*!
  * OpenUI5
- * (c) Copyright 2009-2022 SAP SE or an SAP affiliate company.
+ * (c) Copyright 2009-2023 SAP SE or an SAP affiliate company.
  * Licensed under the Apache License, Version 2.0 - see LICENSE.txt.
  */
 

package/src/sap/m/p13n/modification/FlexModificationHandler.js

@@ -1,6 +1,6 @@
 /*!
  * OpenUI5
- * (c) Copyright 2009-2022 SAP SE or an SAP affiliate company.
+ * (c) Copyright 2009-2023 SAP SE or an SAP affiliate company.
  * Licensed under the Apache License, Version 2.0 - see LICENSE.txt.
  */
 sap.ui.define([

package/src/sap/m/p13n/modification/LocalStorageModificationHandler.js

@@ -1,6 +1,6 @@
 /*!
  * OpenUI5
- * (c) Copyright 2009-2022 SAP SE or an SAP affiliate company.
+ * (c) Copyright 2009-2023 SAP SE or an SAP affiliate company.
  * Licensed under the Apache License, Version 2.0 - see LICENSE.txt.
  */
 sap.ui.define([