@sapui5/sap.ui.vk 1.96.0 → 1.97.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sapui5/sap.ui.vk",
-  "version": "1.96.0",
+  "version": "1.97.0",
   "description": "SAPUI5 Library sap.ui.vk",
   "homepage": "https://sap.github.io/ui5-tooling/pages/SAPUI5/",
   "author": "SAP SE (https://www.sap.com)",

package/src/sap/ui/vk/.library

@@ -7,7 +7,7 @@
 
         (c) Copyright 2009-2015 SAP SE. All rights reserved
     </copyright>
-    <version>1.96.0</version>
+    <version>1.97.0</version>
 
     <documentation>SAPUI5 Visualization Toolkit.</documentation>
 

package/src/sap/ui/vk/AnimationPlayback.js

@@ -25,7 +25,7 @@ sap.ui.define([
 	 *
 	 * @private
 	 * @author SAP SE
-	 * @version 1.96.0
+	 * @version 1.97.0
 	 * @extends sap.ui.base.EventProvider
 	 * @alias sap.ui.vk.AnimationPlayback
 	 * @experimental Since 1.67.0 This class is experimental and might be modified or removed in future versions.

package/src/sap/ui/vk/AnimationPlayer.js

@@ -31,7 +31,7 @@ sap.ui.define([
 	 *
 	 * @public
 	 * @author SAP SE
-	 * @version 1.96.0
+	 * @version 1.97.0
 	 * @extends sap.ui.core.Element
 	 * @alias sap.ui.vk.AnimationPlayer
 	 * @experimental Since 1.67.0 This class is experimental and might be modified or removed in future versions.
@@ -127,7 +127,7 @@ sap.ui.define([
 	};
 
 	AnimationPlayer.prototype._onViewApplied = function(channel, eventId, event) {
-		if (event.source.getId() != this.getViewStateManager()) {
+		if (event.source == null || event.source.getId() !== this.getViewStateManager()) {
 			return;
 		}
 

package/src/sap/ui/vk/AnimationSequence.js

@@ -31,7 +31,7 @@ sap.ui.define([
 	 *
 	 * @private
 	 * @author SAP SE
-	 * @version 1.96.0
+	 * @version 1.97.0
 	 * @extends sap.ui.base.Object
 	 * @alias sap.ui.vk.AnimationSequence
 	 * @implements sap.ui.vk.IJointCollection

package/src/sap/ui/vk/AnimationTimeSlider.js

@@ -24,7 +24,7 @@ sap.ui.define([
 	 * @extends sap.m.Slider
 	 *
 	 * @author SAP SE
-	 * @version 1.96.0
+	 * @version 1.97.0
 	 *
 	 * @constructor
 	 * @private

package/src/sap/ui/vk/AnimationTrack.js

@@ -30,7 +30,7 @@ sap.ui.define([
 	 * @param {boolean} parameters.infinite infinite flag
 	 *
 	 * @author SAP SE
-	 * @version 1.96.0
+	 * @version 1.97.0
 	 * @extends sap.ui.base.Object
 	 * @alias sap.ui.vk.AnimationTrack
 	 *

package/src/sap/ui/vk/Annotation.js

@@ -36,7 +36,7 @@ sap.ui.define([
 	 *
 	 * @public
 	 * @author SAP SE
-	 * @version 1.96.0
+	 * @version 1.97.0
 	 * @extends sap.ui.core.Control
 	 * @alias sap.ui.vk.Annotation
 	 * @experimental Since 1.76.0 This class is experimental and might be modified or removed in future versions.

package/src/sap/ui/vk/BaseNodeProxy.js

@@ -34,7 +34,7 @@ sap.ui.define([
 	 * @public
 	 * @abstract
 	 * @author SAP SE
-	 * @version 1.96.0
+	 * @version 1.97.0
 	 * @extends sap.ui.base.Object
 	 * @implements sap.ui.base.Poolable, sap.ui.vk.BaseNodeProxy
 	 * @alias sap.ui.vk.BaseNodeProxy

package/src/sap/ui/vk/Camera.js

@@ -21,7 +21,7 @@ sap.ui.define([
 	 * @public
 	 * @abstract
 	 * @author SAP SE
-	 * @version 1.96.0
+	 * @version 1.97.0
 	 * @extends sap.ui.base.ManagedObject
 	 * @alias sap.ui.vk.Camera
 	 * @since 1.52.0

package/src/sap/ui/vk/ContentConnector.js

@@ -36,7 +36,7 @@ sap.ui.define([
 	 * @param {object} [mSettings] Initial settings for the new ContentConnector object.
 	 * @public
 	 * @author SAP SE
-	 * @version 1.96.0
+	 * @version 1.97.0
 	 * @extends sap.ui.core.Element
 	 * @alias sap.ui.vk.ContentConnector
 	 */

package/src/sap/ui/vk/ContentManager.js

@@ -24,7 +24,7 @@ sap.ui.define([
 	 * @protected
 	 * @abstract
 	 * @author SAP SE
-	 * @version 1.96.0
+	 * @version 1.97.0
 	 * @extends sap.ui.base.ManagedObject
 	 * @alias sap.ui.vk.ContentManager
 	 * @since 1.50.0

package/src/sap/ui/vk/ContentResource.js

@@ -25,7 +25,7 @@ sap.ui.define([
 	 * @param {object} [oScope] scope An object for resolving string-based type and formatter references in bindings.
 	 * @public
 	 * @author SAP SE
-	 * @version 1.96.0
+	 * @version 1.97.0
 	 * @extends sap.ui.base.ManagedObject
 	 * @alias sap.ui.vk.ContentResource
 	 */
@@ -166,10 +166,12 @@ sap.ui.define([
 
 				/**
 				 * Optional parameter to load UsageIds. Valid values:
-				 *   true - Load all UsageIds. This is potentially very expensive, as there could be many UsageIds defined for the tenant which are not relevant to the current application.
-				 *   false - Do not load any UsageIds.
-				 *   string - A single non-empty string which defines the name of the requested UsageId.
-				 *   string[] - An array of non-empty strings which define the names of requested UsageIds.
+				 * <ul>
+				 *   <li><code>true</code> - Load all UsageIds. This is potentially very expensive, as there could be many UsageIds defined for the tenant which are not relevant to the current application.
+				 *   <li><code>false</code> - Do not load any UsageIds.
+				 *   <li><code>string</code> - A single non-empty string which defines the name of the requested UsageId.
+				 *   <li><code>string[]</code> - An array of non-empty strings which define the names of requested UsageIds.
+				 * </ul>
 				 */
 				includeUsageId: {
 					type: "sap.ui.vk.IncludeUsageIdType",

package/src/sap/ui/vk/Core.js

@@ -37,7 +37,7 @@ sap.ui.define([
 	 * @final
 	 * @private
 	 * @author SAP SE
-	 * @version 1.96.0
+	 * @version 1.97.0
 	 * @extends sap.ui.base.ManagedObject
 	 * @alias sap.ui.vk.Core
 	 * @since 1.50.0

package/src/sap/ui/vk/DownloadManager.js

@@ -37,7 +37,7 @@ sap.ui.define([
 	 * 				If left undefined, the number of retry attempts will default to 1. Passing in 0 disables any retry attempts.
 	 * @private
 	 * @author SAP SE
-	 * @version 1.96.0
+	 * @version 1.97.0
 	 * @extends sap.ui.base.EventProvider
 	 * @alias sap.ui.vk.DownloadManager
 	 * @since 1.32.0

package/src/sap/ui/vk/DrawerToolbar.js

@@ -64,7 +64,7 @@ sap.ui.define([
 	 * @extends sap.ui.core.Control
 	 *
 	 * @author SAP
-	 * @version 1.96.0
+	 * @version 1.97.0
 	 *
 	 * @public
 	 * @alias sap.ui.vk.DrawerToolbar
@@ -305,7 +305,7 @@ sap.ui.define([
 	DrawerToolbar.prototype._getViewport = function() {
 		var viewport = core.byId(this.getViewport());
 		viewport = viewport instanceof Viewport && viewport.getImplementation() || viewport;
-		return viewport && (isViewport3D(viewport) || isSvgViewport(viewport)) ? viewport : null;
+		return viewport && (isViewport3D(viewport) || isSvgViewport(viewport) || isNativeViewport(viewport)) ? viewport : null;
 	};
 
 	DrawerToolbar.prototype._getViewStateManager = function() {
@@ -627,7 +627,9 @@ sap.ui.define([
 
 		var showHideSeparator = new ToolbarSeparator();
 		var rectSelectSeparator = new ToolbarSeparator();
+		var fitToViewSeparator = new ToolbarSeparator();
 		this._itemsShowHideSelect = [show, hide, showHideSeparator, this._rectSelectionButton, rectSelectSeparator];
+		this._itemsFitToView = [fitToView, fitToViewSeparator];
 
 		this._items3D = [
 			this._gestureButtons[0],
@@ -649,7 +651,7 @@ sap.ui.define([
 			this._gestureButtons[3],
 			new ToolbarSeparator(),
 			fitToView,
-			new ToolbarSeparator(),
+			fitToViewSeparator,
 			this._rectSelectionButton,
 			rectSelectSeparator,
 			crossSectionButton,
@@ -749,12 +751,11 @@ sap.ui.define([
 					setGestureIcon(this.viewport, this._mode);
 					var dx = p.x - this._x;
 					var dy = p.y - this._y;
-					var cameraController = isViewport3D(this.viewport) ? this.viewport._viewportGestureHandler._cameraController : this.viewport;
 					switch (this._mode) {
-						case 0: cameraController.rotate(dx, dy, true); break;
-						case 1: cameraController.rotate(dx, dy, false); break;
-						case 2: cameraController.pan(dx, dy); break;
-						case 3: cameraController.zoom(1 + dy * 0.005); break;
+						case 0: this.viewport.rotate(dx, dy, true); break;
+						case 1: this.viewport.rotate(dx, dy, false); break;
+						case 2: this.viewport.pan(dx, dy); break;
+						case 3: this.viewport.zoom(1 + dy * 0.005); break;
 						default: break;
 					}
 					this._x = p.x;
@@ -828,7 +829,7 @@ sap.ui.define([
 		var viewport = this._getViewport();
 		this.setVisible(!!viewport);
 		if (viewport) {
-			var is3D = isViewport3D(viewport);
+			var is3D = isViewport3D(viewport) && !viewport._redlineHandler;
 			var isNative = isNativeViewport(viewport);
 
 			this._ignoreVisiblityChange = true;
@@ -837,6 +838,10 @@ sap.ui.define([
 				item.setVisible(!isNative && this._itemsVisibility.get(item));
 			}.bind(this));
 
+			this._itemsFitToView.forEach(function(item) {
+				item.setVisible(!viewport._redlineHandler && this._itemsVisibility.get(item));
+			}.bind(this));
+
 			this._items3D.forEach(function(item) {
 				item.setVisible(is3D && this._itemsVisibility.get(item));
 			}.bind(this));

package/src/sap/ui/vk/DvlException.js

@@ -16,7 +16,7 @@ sap.ui.define(["sap/ui/base/Exception"], function(Exception) {
 	 * @param {string} message The error message.
 	 * @private
 	 * @author SAP SE
-	 * @version 1.96.0
+	 * @version 1.97.0
 	 * @extends sap.ui.base.Exception
 	 * @alias sap.ui.vk.DvlException
 	 */

package/src/sap/ui/vk/FlexibleControl.js

@@ -28,7 +28,7 @@ sap.ui.define([
 	 * @extends sap.ui.core.Control
 	 *
 	 * @author SAP SE
-	 * @version 1.96.0
+	 * @version 1.97.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/ui/vk/FlexibleControlLayoutData.js

@@ -25,7 +25,7 @@ sap.ui.define([
 	 * (The CSS value "auto" is used internally to recalculate the size of the content
 	 * dynamically and is not directly set as style property.)
 	 * @extends sap.ui.core.LayoutData
-	 * @version 1.96.0
+	 * @version 1.97.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/ui/vk/Highlight.js

@@ -24,7 +24,7 @@ sap.ui.define([
 	 *
 	 * @private
 	 * @author SAP SE
-	 * @version 1.96.0
+	 * @version 1.97.0
 	 * @extends sap.ui.base.Object
 	 * @alias sap.ui.vk.Highlight
 	 * @since 1.73.0

package/src/sap/ui/vk/ImageContentManager.js

@@ -25,7 +25,7 @@ sap.ui.define([
 	 * @param {object} [mSettings] Initial settings for the new ImageContentManager object.
 	 * @public
 	 * @author SAP SE
-	 * @version 1.96.0
+	 * @version 1.97.0
 	 * @extends sap.ui.vk.ContentManager
 	 * @alias sap.ui.vk.ImageContentManager
 	 * @since 1.50.0

package/src/sap/ui/vk/IncludeUsageIdType.js

@@ -3,6 +3,23 @@ sap.ui.define([
 ], function(DataType) {
 	"use strict";
 
+	/**
+	 * @classdesc
+	 * This type defines possible valid values for property {@link sap.ui.vk.ContentResource#getIncludeUsageId ContentResource.includeUsageId}.
+	 *
+	 * The possible valid values:
+	 * <ul>
+	 *   <li><code>true</code> - Load all UsageIds. This is potentially very expensive, as there could be many UsageIds defined for the tenant which are not relevant to the current application.
+	 *   <li><code>false</code> - Do not load any UsageIds.
+	 *   <li><code>string</code> - A single non-empty string which defines the name of the requested UsageId.
+	 *   <li><code>string[]</code> - An array of non-empty strings which define the names of requested UsageIds.
+	 * </ul>
+	 * @namespace
+	 * @final
+	 * @public
+	 * @alias sap.ui.vk.IncludeUsageIdType
+	 * @since 1.96.0
+	 */
 	var IncludeUsageIdType = DataType.createType(
 		"sap.ui.vk.IncludeUsageIdType",
 		{

package/src/sap/ui/vk/JointUtils.js

@@ -18,7 +18,7 @@ sap.ui.define([
 	 *
 	 * @private
 	 * @author SAP SE
-	 * @version 1.96.0
+	 * @version 1.97.0
 	 * @alias sap.ui.vk.JointUtils
 	 * @since 1.74
 	 * @experimental Since 1.74 This class is experimental and might be modified or removed in future versions.

package/src/sap/ui/vk/LayerProxy.js

@@ -28,7 +28,7 @@ sap.ui.define([
 	 * @public
 	 * @abstract
 	 * @author SAP SE
-	 * @version 1.96.0
+	 * @version 1.97.0
 	 * @extends sap.ui.base.Object
 	 * @alias sap.ui.vk.LayerProxy
 	 */

package/src/sap/ui/vk/Loco.js

@@ -19,7 +19,7 @@ sap.ui.define([
 	 * @extends sap.ui.base.EventProvider
 	 *
 	 * @author SAP SE
-	 * @version 1.96.0
+	 * @version 1.97.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/ui/vk/Material.js

@@ -21,7 +21,7 @@ sap.ui.define([
 	 *
 	 * @public
 	 * @author SAP SE
-	 * @version 1.96.0
+	 * @version 1.97.0
 	 * @extends sap.ui.base.ManagedObject
 	 * @alias sap.ui.vk.Material
 	 * @experimental Since 1.60.0 This class is experimental and might be modified or removed in future versions.

package/src/sap/ui/vk/NativeViewport.js

@@ -46,7 +46,7 @@ sap.ui.define([
 	 *
 	 * @extends sap.ui.vk.ViewportBase
 	 * @author SAP SE
-	 * @version 1.96.0
+	 * @version 1.97.0
 	 * @constructor
 	 * @public
 	 * @alias sap.ui.vk.NativeViewport
@@ -164,7 +164,7 @@ sap.ui.define([
 	};
 
 	NativeViewport.prototype.setShouldRenderFrame = function() {
-		this.rerender();
+		this.invalidate();
 	};
 
 	/**
@@ -494,6 +494,10 @@ sap.ui.define([
 			return this;
 		}
 
+		if (this._redlineHandler) {
+			this._redlineHandler.pan(dx, dy);
+		}
+
 		this._x += dx;
 		this._y += dy;
 		this._update();
@@ -517,22 +521,7 @@ sap.ui.define([
 	 * @public
 	 */
 	NativeViewport.prototype.rotate = function(dx, dy) {
-		if (this._svgError) {
-			return this;
-		}
-
-		this._x += dx;
-		this._y += dy;
-		this._update();
-		this.fireMove({
-			pan: {
-				x: dx,
-				y: dy
-			},
-			zoom: 1.0
-		});
-
-		return this;
+		return this.pan(dx, dy);
 	};
 
 	NativeViewport.prototype._getZoomInLimit = function() {
@@ -568,6 +557,10 @@ sap.ui.define([
 		this._s = Math.min(Math.max(this._s * z, this._getZoomOutLimit()), this._getZoomInLimit());
 		z = this._s / oldScale;
 
+		if (this._redlineHandler) {
+			this._redlineHandler.zoom(z);
+		}
+
 		var gxn = this._gx * this._s;
 		var gyn = this._gy * this._s;
 		var dx = gxo - gxn;

package/src/sap/ui/vk/NodeHierarchy.js

@@ -34,7 +34,7 @@ sap.ui.define([
 	 * @public
 	 * @abstract
 	 * @author SAP SE
-	 * @version 1.96.0
+	 * @version 1.97.0
 	 * @extends sap.ui.base.ManagedObject
 	 * @alias sap.ui.vk.NodeHierarchy
 	 */

package/src/sap/ui/vk/NodeProxy.js

@@ -22,7 +22,7 @@ sap.ui.define([
 	 *
 	 * @public
 	 * @author SAP SE
-	 * @version 1.96.0
+	 * @version 1.97.0
 	 * @extends sap.ui.base.ManagedObject
 	 * @alias sap.ui.vk.NodeProxy
 	 * @since 1.32.0

package/src/sap/ui/vk/NodeUtils.js

@@ -13,7 +13,7 @@ sap.ui.define([
 	 *
 	 * @private
 	 * @author SAP SE
-	 * @version 1.96.0
+	 * @version 1.97.0
 	 * @alias sap.ui.vk.NodeUtils
 	 * @since 1.74
 	 * @experimental Since 1.74 This class is experimental and might be modified or removed in future versions.

package/src/sap/ui/vk/Notifications.js

@@ -33,7 +33,7 @@ sap.ui.define([
 	 * @param {object} [mSettings] Initial settings for the new Notifications control.
 	 * @public
 	 * @author SAP SE
-	 * @version 1.96.0
+	 * @version 1.97.0
 	 * @extends sap.ui.core.Control
 	 * @alias sap.ui.vk.Notifications
 	 * @since 1.38.0

package/src/sap/ui/vk/OrthographicCamera.js

@@ -20,7 +20,7 @@ sap.ui.define([
 	 *
 	 * @public
 	 * @author SAP SE
-	 * @version 1.96.0
+	 * @version 1.97.0
 	 * @extends sap.ui.vk.Camera
 	 * @alias sap.ui.vk.OrthographicCamera
 	 * @since 1.52.0

package/src/sap/ui/vk/PerspectiveCamera.js

@@ -20,7 +20,7 @@ sap.ui.define([
 	 *
 	 * @public
 	 * @author SAP SE
-	 * @version 1.96.0
+	 * @version 1.97.0
 	 * @extends sap.ui.vk.Camera
 	 * @alias sap.ui.vk.PerspectiveCamera
 	 * @since 1.52.0

package/src/sap/ui/vk/ProgressIndicator.js

@@ -27,7 +27,7 @@ sap.ui.define([
 	 * @extends sap.m.ProgressIndicator
 	 *
 	 * @author SAP SE
-	 * @version 1.96.0
+	 * @version 1.97.0
 	 *
 	 * @constructor
 	 * @public

package/src/sap/ui/vk/RedlineCollaboration.js

@@ -30,7 +30,7 @@ sap.ui.define([
 	 *
 	 * @public
 	 * @author SAP SE
-	 * @version 1.96.0s
+	 * @version 1.97.0s
 	 * @extends sap.ui.core.Element
 	 * @alias sap.ui.vk.RedlineCollaboration
 	 * @since 1.89.0
@@ -261,8 +261,12 @@ sap.ui.define([
 				vkCore.getEventBus().subscribe("sap.ui.vk", "readyForAnimation", this._readyForAnimation, this);
 				var vsm = core.byId(viewport.getViewStateManager());
 				var vm = core.byId(vsm.getViewManager());
-				vm.setAutoPlayAnimation(false);
-				vm.activateView(view);
+				vm.setAllowAutoPlayAnimation(false);
+				this._activatingView = true;
+				if (this._tool.getActive()) {
+					this._tool.setActive(false, viewport); // RedlineTool must be deactivated before activateView
+				}
+				vm.activateView(view, true);
 			}
 			if (!this._tool.getActive()) {
 				this._tool.setActive(true, viewport);
@@ -297,10 +301,11 @@ sap.ui.define([
 			}
 			viewport.setViewInfo(viewInfo);
 		}
+		this._activatingView = false;
 		vkCore.getEventBus().unsubscribe("sap.ui.vk", "readyForAnimation", this._readyForAnimation, this);
 		var vsm = core.byId(viewport.getViewStateManager());
 		var vm = core.byId(vsm.getViewManager());
-		vm.setAutoPlayAnimation(true);
+		vm.setAllowAutoPlayAnimation(true);
 	};
 
 	/**
@@ -406,11 +411,7 @@ sap.ui.define([
 			}
 		}
 
-		var conversation = core.byId(this.getActiveConversation());
-		if (conversation) {
-			this._tool.destroyRedlineElements();
-			conversation._activate(this._tool);
-		}
+		this._tool.getGizmo().rerender();
 
 		return this;
 	};
@@ -445,11 +446,7 @@ sap.ui.define([
 			normalizedArray[i].setHalo(false);
 		}
 
-		var conversation = core.byId(this.getActiveConversation());
-		if (conversation) {
-			this._tool.destroyRedlineElements();
-			conversation._activate(this._tool);
-		}
+		this._tool.getGizmo().rerender();
 
 		return this;
 	};
@@ -613,7 +610,7 @@ sap.ui.define([
 		var activeId = "";
 		if (this.getActiveConversation()) {
 			var activeConversation = core.byId(this.getActiveConversation());
-			if (activeConversation.getViewInfo()) {
+			if (!this._activatingView && activeConversation.getViewInfo()) {
 				var result = this._updateViewInfo();
 				activeConversation.setViewInfo(result);
 			}

package/src/sap/ui/vk/RedlineConversation.js

@@ -25,7 +25,7 @@ function(
 	 *
 	 * @public
 	 * @author SAP SE
-	 * @version 1.96.0
+	 * @version 1.97.0
 	 * @extends sap.ui.core.Element
 	 * @alias sap.ui.vk.RedlineConversation
 	 * @since 1.89.0

package/src/sap/ui/vk/RedlineDesign.js

@@ -29,7 +29,7 @@ sap.ui.define([
 		 *
 		 * @public
 		 * @author SAP SE
-		 * @version 1.96.0
+		 * @version 1.97.0
 		 * @extends sap.ui.vk.RedlineSurface
 		 * @alias sap.ui.vk.RedlineDesign
 		 * @since 1.40.0

package/src/sap/ui/vk/RedlineElement.js

@@ -19,7 +19,7 @@ sap.ui.define([
 	 *
 	 * @public
 	 * @author SAP SE
-	 * @version 1.96.0
+	 * @version 1.97.0
 	 * @extends sap.ui.core.Element
 	 * @alias sap.ui.vk.RedlineElement
 	 * @since 1.40.0
@@ -132,6 +132,24 @@ sap.ui.define([
 		return rgba;
 	};
 
+	RedlineElement.prototype._getHaloFilter = function(color) {
+		var rgba = this._colorToArray(this.getHaloColor());
+		var colorId = rgba.join("");
+		return "url(#halo" + colorId + ")";
+	};
+
+	RedlineElement.prototype.setHalo = function(halo) {
+		this.setProperty("halo", halo, true);
+		var domRef = this.getDomRef();
+		if (domRef) {
+			if (halo) {
+				domRef.setAttribute("filter", this._getHaloFilter());
+			} else {
+				domRef.removeAttribute("filter");
+			}
+		}
+	};
+
 	/**
 	 * Exports all the relevant data contained in the redline element to a JSON-like object.
 	 * @returns {object} JSON Relevant data that can be serialized and later used to restore the redline element.

package/src/sap/ui/vk/RedlineElementComment.js

@@ -20,7 +20,7 @@ sap.ui.define([
 	 *
 	 * @public
 	 * @author SAP SE
-	 * @version 1.96.0
+	 * @version 1.97.0
 	 * @extends sap.ui.core.Element
 	 * @alias sap.ui.vk.RedlineElementComment
 	 * @since 1.89.0

package/src/sap/ui/vk/RedlineElementEllipse.js

@@ -22,7 +22,7 @@ sap.ui.define([
 	 *
 	 * @public
 	 * @author SAP SE
-	 * @version 1.96.0
+	 * @version 1.97.0
 	 * @extends sap.ui.vk.RedlineElement
 	 * @alias sap.ui.vk.RedlineElementEllipse
 	 * @since 1.40.0
@@ -139,9 +139,7 @@ sap.ui.define([
 		}
 		renderManager.writeAttribute("opacity", this.getOpacity());
 		if (halo) {
-			var rgba = this._colorToArray(this.getHaloColor());
-			var colorId = rgba.join("");
-			renderManager.writeAttribute("filter", "url(#halo" + colorId + ")");
+			renderManager.writeAttribute("filter", this._getHaloFilter());
 		}
 		renderManager.write("></ellipse>");