Copilot commented on code in PR #8581:
URL: https://github.com/apache/texera/pull/8581#discussion_r4075583873
##########
frontend/src/app/workspace/component/workflow-editor/mini-map/mini-map.component.ts:
##########
@@ -17,13 +17,16 @@
* under the License.
*/
-import { AfterViewInit, Component, HostListener, OnDestroy, ViewChild } from
"@angular/core";
+import { AfterViewInit, Component, ElementRef, HostListener, OnDestroy,
ViewChild } from "@angular/core";
import { UntilDestroy, untilDestroyed } from "@ngneat/until-destroy";
import { WorkflowActionService } from
"../../../service/workflow-graph/model/workflow-action.service";
import { MAIN_CANVAS } from "../workflow-editor.component";
import * as joint from "jointjs";
import { JointGraphWrapper } from
"../../../service/workflow-graph/model/joint-graph-wrapper";
import { PanelService } from "../../../service/panel/panel.service";
+
+/** The main paper's events that move or resize its viewport, which is what
the navigator tracks. */
+const MAIN_PAPER_EVENTS = ["translate", "scale", "resize"] as const;
import { CdkDrag } from "@angular/cdk/drag-drop";
Review Comment:
`MAIN_PAPER_EVENTS` is declared between import statements, which is invalid
in TypeScript/ESM (imports must come before other top-level statements). Move
the `MAIN_PAPER_EVENTS` declaration below the final import (or reorder the
imports so all imports appear before any const/logic).
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
