@openui5/sap.ui.table 1.128.0 → 1.129.0

package/src/sap/ui/table/Table.js

@@ -102,7 +102,7 @@ sap.ui.define([
 	 *     the data model and binding being used.
 	 * </p>
 	 * @extends sap.ui.core.Control
-	 * @version 1.128.0
+	 * @version 1.129.0
 	 *
 	 * @constructor
 	 * @public
@@ -237,9 +237,11 @@ sap.ui.define([
 			 * If the <code>scrollThreshold</code> is lower than the number of rows in the scrollable area (number of rows minus
 			 * number of fixed rows), this number is used as the <code>scrollThreshold</code>. If the value is 0, no threshold is
 			 * applied during scrolling. The value -1 applies the same value as the <code>threshold</code> property.
-			 *
+			 * <br/>
 			 * <b>Note:</b> This property only takes effect if it is set to a positive integer value.
 			 *
+			 * The value of the <code>scrollThreshold</code> should be higher than the <code>threshold</code> value to avoid unnecessary requests.
+			 *
 			 * For <code>AnalyticalTable</code> and <code>TreeTable</code>, the <code>scrollThreshold</code> property must be
 			 * higher than the <code>threshold</code> property to take effect.
 			 *
@@ -2410,6 +2412,8 @@ sap.ui.define([
 		const mRowCounts = this._getRowCounts();
 		let iThreshold = this._bScrolled ? this._getScrollThreshold() : this.getThreshold();
 
+		this._bScrolled = false;
+
 		iRequestLength = iRequestLength == null ? mRowCounts.count : iRequestLength;
 
 		// If the threshold is not explicitly disabled by setting it to 0, the threshold should be at least the number of scrollable rows.
@@ -2533,7 +2537,6 @@ sap.ui.define([
 	 */
 	Table.prototype.refreshRows = function(sReason) {
 		this._bContextsAvailable = false;
-		this._bScrolled = false;
 
 		if (sReason === ChangeReason.Sort || sReason === ChangeReason.Filter) {
 			this.setFirstVisibleRow(0);
@@ -2558,8 +2561,6 @@ sap.ui.define([
 			return;
 		}
 
-		this._bScrolled = false;
-
 		if (oEventInfo.detailedReason === "AddVirtualContext") {
 			createVirtualRow(this);
 			return;

package/src/sap/ui/table/TablePersoController.js

@@ -34,7 +34,7 @@ sap.ui.define([
 	 * @extends sap.ui.base.ManagedObject
 	 *
 	 * @author SAP SE
-	 * @version 1.128.0
+	 * @version 1.129.0
 	 * @since 1.21.1
 	 *
 	 * @constructor

package/src/sap/ui/table/TableRenderer.js

@@ -959,13 +959,6 @@ sap.ui.define([
 			rm.class("sapUiTableGroupIcon");
 			rm.openEnd();
 			rm.close("div");
-
-			if (TableUtils.Grouping.showGroupMenuButton(oTable)) {
-				const oIconInfo = IconPool.getIconInfo("sap-icon://drop-down-list");
-				rm.openStart("div").class("sapUiTableGroupMenuButton").openEnd();
-				rm.text(oIconInfo.content);
-				rm.close("div");
-			}
 		}
 	};
 

package/src/sap/ui/table/TreeTable.js

@@ -42,7 +42,7 @@ sap.ui.define([
 	 * @class
 	 * The TreeTable control provides a comprehensive set of features to display hierarchical data.
 	 * @extends sap.ui.table.Table
-	 * @version 1.128.0
+	 * @version 1.129.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/ui/table/extensions/Accessibility.js

@@ -969,7 +969,7 @@ sap.ui.define([
 	 * @class Extension for sap.ui.table.Table which handles ACC related things.
 	 * @extends sap.ui.table.extensions.ExtensionBase
 	 * @author SAP SE
-	 * @version 1.128.0
+	 * @version 1.129.0
 	 * @constructor
 	 * @private
 	 * @alias sap.ui.table.extensions.Accessibility

package/src/sap/ui/table/extensions/AccessibilityRender.js

@@ -44,7 +44,7 @@ sap.ui.define([
 	 * @class Extension for sap.ui.table.TableRenderer which handles ACC related things.
 	 * @extends sap.ui.table.extensions.ExtensionBase
 	 * @author SAP SE
-	 * @version 1.128.0
+	 * @version 1.129.0
 	 * @constructor
 	 * @private
 	 * @alias sap.ui.table.extensions.AccessibilityRender

package/src/sap/ui/table/extensions/DragAndDrop.js

@@ -274,7 +274,7 @@ sap.ui.define([
 	 *
 	 * @extends sap.ui.table.extensions.ExtensionBase
 	 * @author SAP SE
-	 * @version 1.128.0
+	 * @version 1.129.0
 	 * @constructor
 	 * @private
 	 * @alias sap.ui.table.extensions.DragAndDrop

package/src/sap/ui/table/extensions/ExtensionBase.js

@@ -19,7 +19,7 @@ sap.ui.define([
 	 * @abstract
 	 * @extends sap.ui.base.Object
 	 * @author SAP SE
-	 * @version 1.128.0
+	 * @version 1.129.0
 	 * @constructor
 	 * @private
 	 * @alias sap.ui.table.extensions.ExtensionBase

package/src/sap/ui/table/extensions/Keyboard.js

@@ -267,7 +267,7 @@ sap.ui.define([
 	 * @class Extension for sap.ui.table.Table which handles keyboard related things.
 	 * @extends sap.ui.table.extensions.ExtensionBase
 	 * @author SAP SE
-	 * @version 1.128.0
+	 * @version 1.129.0
 	 * @constructor
 	 * @private
 	 * @alias sap.ui.table.extensions.Keyboard

package/src/sap/ui/table/extensions/KeyboardDelegate.js

@@ -65,7 +65,7 @@ sap.ui.define([
 	 *
 	 * @extends sap.ui.base.Object
 	 * @author SAP SE
-	 * @version 1.128.0
+	 * @version 1.129.0
 	 * @constructor
 	 * @private
 	 * @alias sap.ui.table.extensions.KeyboardDelegate

package/src/sap/ui/table/extensions/Pointer.js

@@ -32,36 +32,6 @@ sap.ui.define([
 	 */
 	const ExtensionHelper = {
 
-		/*
-		 * Returns the pageX and pageY position of the given mouse/touch event.
-		 */
-		_getEventPosition: function(oEvent, oTable) {
-			const oPosition = getTouchObject(oEvent) || oEvent;
-
-			function getTouchObject(oTouchEvent) {
-				if (!oTable._isTouchEvent(oTouchEvent)) {
-					return null;
-				}
-
-				const aTouchEventObjectNames = ["touches", "targetTouches", "changedTouches"];
-
-				for (let i = 0; i < aTouchEventObjectNames.length; i++) {
-					const sTouchEventObjectName = aTouchEventObjectNames[i];
-
-					if (oEvent[sTouchEventObjectName] && oEvent[sTouchEventObjectName][0]) {
-						return oEvent[sTouchEventObjectName][0];
-					}
-					if (oEvent.originalEvent[sTouchEventObjectName] && oEvent.originalEvent[sTouchEventObjectName][0]) {
-						return oEvent.originalEvent[sTouchEventObjectName][0];
-					}
-				}
-
-				return null;
-			}
-
-			return {x: oPosition.pageX, y: oPosition.pageY};
-		},
-
 		/*
 		 * Returns true, when the given click event should be skipped because it happened on a
 		 * interactive control inside a table cell.
@@ -125,7 +95,7 @@ sap.ui.define([
 
 			oTable._bIsColumnResizerMoving = true;
 			oTable._bColumnResizerMoved = false;
-			oTable._iColumnResizeStart = ExtensionHelper._getEventPosition(oEvent, oTable).x;
+			oTable._iColumnResizeStart = TableUtils.getEventPosition(oEvent, oTable).x;
 			oTable.$().toggleClass("sapUiTableResizing", true);
 
 			const $Document = jQuery(document);
@@ -145,7 +115,7 @@ sap.ui.define([
 		 * Drops the previous dragged column resize bar and recalculates the new column width.
 		 */
 		exitColumnResizing: function(oEvent) {
-			const iLocationX = ExtensionHelper._getEventPosition(oEvent, this).x;
+			const iLocationX = TableUtils.getEventPosition(oEvent, this).x;
 			const oColumn = this._getVisibleColumns()[this._iLastHoveredVisibleColumnIndex];
 			const $RelevantColumnElement = this.$().find("th[data-sap-ui-colid=\"" + oColumn.getId() + "\"]"); // Consider span and multi-header
 			const iColumnWidth = $RelevantColumnElement[0].offsetWidth; // the width of the column with padding and border
@@ -163,7 +133,7 @@ sap.ui.define([
 		 * Handler for the move events while dragging the column resize bar.
 		 */
 		onMouseMoveWhileColumnResizing: function(oEvent) {
-			const iLocationX = ExtensionHelper._getEventPosition(oEvent, this).x;
+			const iLocationX = TableUtils.getEventPosition(oEvent, this).x;
 			const iRszOffsetLeft = this.$().find(".sapUiTableCnt").offset().left;
 			const iRszLeft = Math.floor(iLocationX - iRszOffsetLeft);
 
@@ -302,7 +272,7 @@ sap.ui.define([
 		 * Reposition the ghost.
 		 */
 		onMouseMoveWhileReordering: function(oEvent) {
-			const oEventPosition = ExtensionHelper._getEventPosition(oEvent, this);
+			const oEventPosition = TableUtils.getEventPosition(oEvent, this);
 			const iLocationX = oEventPosition.x;
 			const iLocationY = oEventPosition.y;
 			const iOldColPos = this._iNewColPos;
@@ -650,9 +620,6 @@ sap.ui.define([
 			} else if (oRow && oRow.isSummary()) {
 				// Sum row cannot be selected
 				oEvent.preventDefault();
-			} else if ($Target.hasClass("sapUiTableGroupMenuButton")) {
-				// Analytical Table: Mobile group menu button in group header rows.
-				TableUtils.Menu.openContextMenu(this, oEvent);
 			} else if ($Target.hasClass("sapUiTableGroupIcon") || $Target.hasClass("sapUiTableTreeIcon")) {
 				// Expand/Collapse icon
 				oRow.toggleExpandedState();
@@ -703,7 +670,7 @@ sap.ui.define([
 	 * @class Extension for sap.ui.table.Table which handles mouse and touch related things.
 	 * @extends sap.ui.table.extensions.ExtensionBase
 	 * @author SAP SE
-	 * @version 1.128.0
+	 * @version 1.129.0
 	 * @constructor
 	 * @private
 	 * @alias sap.ui.table.extensions.Pointer

package/src/sap/ui/table/extensions/Scrolling.js

@@ -2295,7 +2295,7 @@ sap.ui.define([
 	 * @class Extension for sap.ui.table.Table which handles scrolling.
 	 * @extends sap.ui.table.extensions.ExtensionBase
 	 * @author SAP SE
-	 * @version 1.128.0
+	 * @version 1.129.0
 	 * @constructor
 	 * @private
 	 * @alias sap.ui.table.extensions.Scrolling

package/src/sap/ui/table/extensions/ScrollingIOS.js

@@ -27,7 +27,7 @@ sap.ui.define([
 	 * @class Extension for sap.ui.table.Table which handles the scrollbar on iOS.
 	 * @extends sap.ui.table.extensions.ExtensionBase
 	 * @author SAP SE
-	 * @version 1.128.0
+	 * @version 1.129.0
 	 * @constructor
 	 * @private
 	 * @alias sap.ui.table.extensions.ScrollingIOS

package/src/sap/ui/table/extensions/Synchronization.js

@@ -185,7 +185,7 @@ sap.ui.define([
 	 * @class Extension for sap.ui.table.Table that allows synchronization with a table.
 	 * @extends sap.ui.table.extensions.ExtensionBase
 	 * @author SAP SE
-	 * @version 1.128.0
+	 * @version 1.129.0
 	 * @constructor
 	 * @private
 	 * @alias sap.ui.table.extensions.Synchronization

package/src/sap/ui/table/library.js

@@ -27,14 +27,14 @@ sap.ui.define([
 	 * @namespace
 	 * @alias sap.ui.table
 	 * @author SAP SE
-	 * @version 1.128.0
+	 * @version 1.129.0
 	 * @since 0.8
 	 * @public
 	 */
 	const thisLib = Library.init({
 		name: "sap.ui.table",
 		apiVersion: 2,
-		version: "1.128.0",
+		version: "1.129.0",
 		dependencies: ["sap.ui.core", "sap.ui.unified"],
 		designtime: "sap/ui/table/designtime/library.designtime",
 		types: [
@@ -96,7 +96,7 @@ sap.ui.define([
 	/**
 	 * Navigation mode of the table
 	 *
-	 * @version 1.128.0
+	 * @version 1.129.0
 	 * @enum {string}
 	 * @deprecated As of version 1.38, the concept has been discarded.
 	 * @public
@@ -112,8 +112,8 @@ sap.ui.define([
 		/**
 		 * Uses the paginator control.
 		 * This option must no longer be used. Using a scrollbar is the only navigation mode which is supported by
-		 * the <code>sap.ui.table</code> library. The <code>navigationMode</code> property has always been a visual representation. No matter which navigation mode
-		 * is used, data fetched from an OData service is loaded page-wise.
+		 * the <code>sap.ui.table</code> library. The <code>navigationMode</code> property has always been a visual
+		 * representation. No matter which navigation mode is used, data fetched from an OData service is loaded page-wise.
 		 * @public
 		 * @deprecated As of version 1.38, replaced by {@link sap.ui.table.NavigationMode.Scrollbar}
 		 */
@@ -127,7 +127,7 @@ sap.ui.define([
 	/**
 	 * Row Action types.
 	 *
-	 * @version 1.128.0
+	 * @version 1.129.0
 	 * @enum {string}
 	 * @public
 	 */
@@ -158,7 +158,7 @@ sap.ui.define([
 	/**
 	 * Selection behavior of the table
 	 *
-	 * @version 1.128.0
+	 * @version 1.129.0
 	 * @enum {string}
 	 * @public
 	 */
@@ -189,7 +189,7 @@ sap.ui.define([
 	/**
 	 * Selection mode of the table
 	 *
-	 * @version 1.128.0
+	 * @version 1.129.0
 	 * @enum {string}
 	 * @public
 	 */
@@ -227,7 +227,7 @@ sap.ui.define([
 	/**
 	 * Sort order of a column
 	 *
-	 * @version 1.128.0
+	 * @version 1.129.0
 	 * @enum {string}
 	 * @public
 	 * @deprecated As of version 1.120, replaced with <code>sap.ui.core.SortOrder</code>
@@ -254,7 +254,7 @@ sap.ui.define([
 	/**
 	 * VisibleRowCountMode of the table
 	 *
-	 * @version 1.128.0
+	 * @version 1.129.0
 	 * @enum {string}
 	 * @deprecated As of version 1.119, see the <code>rowMode</code> aggregation of <code>sap.ui.table.Table</code> for more details.
 	 * @public
@@ -299,7 +299,7 @@ sap.ui.define([
 	 *
 	 * Contains IDs of shared DOM references, which should be accessible to inheriting controls via getDomRef() function.
 	 *
-	 * @version 1.128.0
+	 * @version 1.129.0
 	 * @enum {string}
 	 * @public
 	 */
@@ -400,7 +400,7 @@ sap.ui.define([
 	 *
 	 * This is an alias for {@link sap.ui.model.TreeAutoExpandMode} and kept for compatibility reasons.
 	 *
-	 * @version 1.128.0
+	 * @version 1.129.0
 	 * @typedef {sap.ui.model.TreeAutoExpandMode}
 	 * @public
 	 * @deprecated As of version 1.120, replaced by <code>sap.ui.model.TreeAutoExpandMode</code>
@@ -414,7 +414,7 @@ sap.ui.define([
 	/**
 	 * Mode of a selection plugin
 	 *
-	 * @version 1.128.0
+	 * @version 1.129.0
 	 * @enum {string}
 	 * @private
 	 */

package/src/sap/ui/table/menus/ColumnHeaderMenuAdapter.js

@@ -35,7 +35,7 @@ sap.ui.define([
 	 *
 	 * @extends sap.ui.base.Object
 	 * @author SAP SE
-	 * @version 1.128.0
+	 * @version 1.129.0
 	 * @private
 	 * @alias sap.ui.table.menus.ColumnHeaderMenuAdapter
 	 */
@@ -156,14 +156,14 @@ sap.ui.define([
 	/**
 	 * This hook is called when menu items should be removed from the column menu.
 	 *
-	 * @param {sap.ui.core.IColumnHeaderMenu} oColumnHeaderMenu
+	 * @param {sap.ui.core.IColumnHeaderMenu} oColumnHeaderMenu Instance of the column menu
 	 */
 	ColumnHeaderMenuAdapter.prototype.removeMenuItems = function(oColumnHeaderMenu) {};
 
 	/**
 	 * This hook is called after the column menu and injected menu items are destroyed.
 	 *
-	 * @param {sap.ui.core.IColumnHeaderMenu} oColumnHeaderMenu
+	 * @param {sap.ui.core.IColumnHeaderMenu} oColumnHeaderMenu Instance of the column menu
 	 */
 	ColumnHeaderMenuAdapter.prototype.onAfterMenuDestroyed = function(oColumnHeaderMenu) {};