CELE-116 Code review changes

MetaCell · Dec 17, 2024 · 336c7ef · 336c7ef
1 parent aa03594
commit 336c7ef
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 8 deletions.
diff --git a/...viewers/ThreeD/CustomFormControlLabel.tsx → ...src/components/CustomFormControlLabel.tsx b/...viewers/ThreeD/CustomFormControlLabel.tsx → ...src/components/CustomFormControlLabel.tsx
@@ -1,7 +1,7 @@
 import HelpOutlineIcon from "@mui/icons-material/HelpOutline";
 import { Box, FormControlLabel, Stack, Tooltip, Typography } from "@mui/material";
-import { vars } from "../../../theme/variables.ts"; // Adjust the import path as needed
-import CustomSwitch from "../../ViewerContainer/CustomSwitch.tsx";
+import { vars } from "./../theme/variables.ts"; // Adjust the import path as needed
+import CustomSwitch from "./ViewerContainer/CustomSwitch.tsx";
 
 interface CustomFormControlLabel {
   label: React.ReactNode;

diff --git a/applications/visualizer/frontend/src/components/viewers/EM/EMStackTilesViewer.tsx b/applications/visualizer/frontend/src/components/viewers/EM/EMStackTilesViewer.tsx
@@ -105,8 +105,7 @@ function synapsesStyle(feature: FeatureLike, workspace: Workspace): Style {
 type LayerSelector = [VectorLayer<Feature> | undefined, (feature: Feature, next: () => void) => void];
 
 function selectAcrossLayers(position: Coordinate, ...selectors: LayerSelector[]) {
-  for (let i = 0; i < selectors.length; i++) {
-    const [layer, handler] = selectors[i];
+  for (const [layer, handler] of selectors) {
     const source = layer?.getSource();
     const features = source?.getFeaturesAtCoordinate(position);
     if (!features || features.length === 0) {

diff --git a/applications/visualizer/frontend/src/components/viewers/EM/SceneControls.tsx b/applications/visualizer/frontend/src/components/viewers/EM/SceneControls.tsx
@@ -4,7 +4,7 @@ import ZoomOutIcon from "@mui/icons-material/ZoomOut";
 import { Box, Divider, IconButton, Popover, Typography } from "@mui/material";
 import Tooltip from "@mui/material/Tooltip";
 import { useState } from "react";
-import CustomFormControlLabel from "../ThreeD/CustomFormControlLabel";
+import CustomFormControlLabel from "../../CustomFormControlLabel";
 import { vars } from "../../../theme/variables.ts";
 
 const { gray500 } = vars;
@@ -41,8 +41,7 @@ function SceneControls({ onZoomIn, onResetView, onZoomOut, onPrint, layers }: Sc
     setAnchorEl(null);
   };
 
-  const layersControlsElems = Object.keys(layers).map((key) => {
-    const { label, checked, onToggle } = layers[key as keyof LayersControlsHandlers];
+  const layersControlsElems = Object.entries(layers).map(([key, { label, checked, onToggle }]) => {
     return (
       <CustomFormControlLabel
         key={key}

diff --git a/applications/visualizer/frontend/src/components/viewers/ThreeD/SceneControls.tsx b/applications/visualizer/frontend/src/components/viewers/ThreeD/SceneControls.tsx
@@ -16,7 +16,7 @@ import { useEffect, useRef, useState } from "react";
 import { useSelectedWorkspace } from "../../../hooks/useSelectedWorkspace.ts";
 import { DARK_SCENE_BACKGROUND, LIGHT_SCENE_BACKGROUND } from "../../../settings/threeDSettings.ts";
 import { vars } from "../../../theme/variables.ts";
-import CustomFormControlLabel from "./CustomFormControlLabel.tsx";
+import CustomFormControlLabel from "../../CustomFormControlLabel.tsx";
 import { Recorder } from "./Recorder.ts";
 
 import { useGlobalContext } from "../../../contexts/GlobalContext.tsx";