From d9eac607b9ba4b8c76366adef879e8f44dacb64a Mon Sep 17 00:00:00 2001 From: Boris Sekachev <40690378+bsekachev@users.noreply.github.com> Date: Fri, 14 Dec 2018 01:15:35 +0300 Subject: [PATCH] Fixed: The function sortByZOrder() throws the exception if a frame consists any points (#236) --- cvat/apps/engine/static/engine/js/shapes.js | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/cvat/apps/engine/static/engine/js/shapes.js b/cvat/apps/engine/static/engine/js/shapes.js index b15df209..05ac6993 100644 --- a/cvat/apps/engine/static/engine/js/shapes.js +++ b/cvat/apps/engine/static/engine/js/shapes.js @@ -3116,14 +3116,15 @@ class PointsView extends PolyShapeView { return; } - this._uis.points = this._scenes.svg.group().fill(this._appearance.fill || this._appearance.colors.shape) + this._uis.points = this._scenes.svg.group() + .fill(this._appearance.fill || this._appearance.colors.shape) .on('click', () => { this._positionateMenus(); this._controller.click(); - }).attr({ - 'z_order': position.z_order }).addClass('pointTempGroup'); + this._uis.points.node.setAttribute('z_order', position.z_order) + let points = PolyShapeModel.convertStringToNumberArray(position.points); for (let point of points) { let radius = POINT_RADIUS * 2 / window.cvat.player.geometry.scale; @@ -3158,7 +3159,7 @@ class PointsView extends PolyShapeView { let interpolation = this._controller.interpolate(window.cvat.player.frames.current); if (interpolation.position.points) { let points = window.cvat.translate.points.actualToCanvas(interpolation.position.points); - this._drawPointMarkers(Object.assign(interpolation.position.points, {points: points})); + this._drawPointMarkers(Object.assign(interpolation.position, {points: points})); } } }