Fixed wrong issue position when create a quick issue on a rotated shape (#5162)

* Fixed wrong issue position when create a quick issue on a rotated shape

* Updated changelog

CHANGELOG.md

@@ -54,6 +54,7 @@ non-ascii paths while adding files from "Connected file share" (issue #4428)
 - Restoring CVAT in case of React-renderning fail (<https://github.com/opencv/cvat/pull/5134>)
 - Deleted frames become restored if a user deletes frames from another job of the same task
 (<https://github.com/opencv/cvat/pull/5138>)
+- Wrong issue position when create a quick issue on a rotated shape (<https://github.com/opencv/cvat/pull/5162>)
 - Skeleton points exported out of order in the COCO Keypoints format
   (<https://github.com/opencv/cvat/issues/5048>)
 - Changing an object causes current z layer to be set to the maximum (<https://github.com/opencv/cvat/pull/5145>)

cvat-ui/src/components/annotation-page/canvas/canvas-context-menu.tsx

@@ -1,4 +1,5 @@
 // Copyright (C) 2021-2022 Intel Corporation
+// Copyright (C) 2022 CVAT.ai Corporation
 //
 // SPDX-License-Identifier: MIT
 
@@ -105,10 +106,7 @@ export default function CanvasContextMenu(props: Props): JSX.Element | null {
                 left={left}
                 latestComments={latestComments}
                 onClick={(param: MenuInfo) => {
-                    const [state] = objectStates.filter(
+                    const state = objectStates.find((_state: any): boolean => _state.clientID === contextMenuClientID);
-                        (_state: any): boolean => _state.clientID === contextMenuClientID,
-                    );
-                    if (param.key === ReviewContextMenuKeys.OPEN_ISSUE) {
                     if (state) {
                         let { points } = state;
                         if (['ellipse', 'rectangle'].includes(state.shapeType)) {
@@ -132,22 +130,17 @@ export default function CanvasContextMenu(props: Props): JSX.Element | null {
                             ].map(([x, y]: number[]) => rotatePoint(x, y, state.rotation, cx, cy)).flat();
                         }
 
+                        if (param.key === ReviewContextMenuKeys.OPEN_ISSUE) {
                             onStartIssue(points);
-                        }
                         } else if (param.key === ReviewContextMenuKeys.QUICK_ISSUE_POSITION) {
-                        if (state) {
+                            openIssue(points, consts.QUICK_ISSUE_INCORRECT_POSITION_TEXT);
-                            openIssue(state.points, consts.QUICK_ISSUE_INCORRECT_POSITION_TEXT);
-                        }
                         } else if (param.key === ReviewContextMenuKeys.QUICK_ISSUE_ATTRIBUTE) {
-                        if (state) {
+                            openIssue(points, consts.QUICK_ISSUE_INCORRECT_ATTRIBUTE_TEXT);
-                            openIssue(state.points, consts.QUICK_ISSUE_INCORRECT_ATTRIBUTE_TEXT);
-                        }
                         } else if (
                             param.keyPath.length === 2 &&
                             param.keyPath[1] === ReviewContextMenuKeys.QUICK_ISSUE_FROM_LATEST
                         ) {
-                        if (state) {
+                            openIssue(points, latestComments[+param.keyPath[0]]);
-                            openIssue(state.points, latestComments[+param.keyPath[0]]);
                         }
                     }
                 }}