Feature/code clean up (#190)

* Remove some warnings

* Replace refine and review containers with view container

client/src/modules/assignment/components/assignments/assignments.js

@@ -12,8 +12,6 @@ const sortOrder = (a, b) => (
 );
 
 export const Assignments = ({ type, header, subheader, assignments, showEmpty = true }) => {
-  const enabledStatus = type === 'refine' ? 'active' : type;
-
   const enabled = ({ status, reviewer }) => 
     type === 'refine' ? status === 'active' :
     status !== 'active' || !reviewer?.login;

client/src/modules/view/components/index.js

@@ -3,4 +3,5 @@ export * from './slice';
 export * from './image';
 export * from './mask';
 export * from './surface';
-export * from './bounding-box';
+export * from './bounding-box';
+export * from './view-container';

client/src/modules/refine/components/refine-container/refine-container.js → client/src/modules/view/components/view-container.js

@@ -17,7 +17,7 @@ import { ClaimDialog, RemoveDialog, SplitDialog, MergeDialog, CreateDialog, Dele
 
 const { Column } = Grid;
 
-export const RefineContainer = () => {
+export const ViewContainer = ({ review = false }) => {
   const [{ imageData }, userDispatch] = useContext(UserContext);
   const [, annotateDispatch] = useContext(AnnotateContext);
   const [volumeView, setVolumeView] = useState();
@@ -240,7 +240,7 @@ export const RefineContainer = () => {
       </Grid>
       { !loading && 
         <>
-          <SaveButtons /> 
+          <SaveButtons review={ review } /> 
           <ClaimDialog />
           <RemoveDialog />
           <SplitDialog sliceView={ sliceView } />

client/src/modules/view/components/volume-view/volume-view.js

@@ -106,7 +106,7 @@ const applyActiveRegion = (region, surfaces) => {
 
 export function VolumeView(painter) {  
   // Callbacks
-  let onEdit = () => {};
+//  let onEdit = () => {};
   let onSliceChange = () => {};
 
   const renderWindow = RenderWindow();

client/src/modules/view/components/volume-view/widgets.js

@@ -41,8 +41,8 @@ export function Widgets(painter) {
   let regions = [];
   let backgroundRegions = [];
   let activeRegion = null;
-  let hoverLabel = null;
-  let highlightLabel = null;
+//  let hoverLabel = null;
+//  let highlightLabel = null;
 
   const getRegion = label => {
     const region = regions.concat(backgroundRegions).find(region => region.label === label);

client/src/pages/assignment.js

@@ -3,8 +3,7 @@ import { useNavigate } from 'react-router-dom';
 import { Message } from 'semantic-ui-react';
 import { UserContext } from 'contexts';
 import { RedirectMessage } from 'modules/common/components/redirect-message';
-import { RefineContainer } from 'modules/refine/components/refine-container';
-import { ReviewContainer } from 'modules/review/components/review-container';
+import { ViewContainer } from 'modules/view/components';
 
 export const Assignment = () => {
   const [{ user, assignment }] = useContext(UserContext);
@@ -21,9 +20,9 @@ export const Assignment = () => {
     : !assignment ? 
       <RedirectMessage message='No Assignment' /> 
     : assignment.status === 'active' ? 
-      <RefineContainer />
+      <ViewContainer />
     : assignment.status === 'review' ?
-      <ReviewContainer />
+      <ViewContainer review={ true } />
     :
       <Message>Unknown assignment type { assignment.type }</Message>
   );

client/src/vtk/filters/ninjato-painter/ninjato-painter.worker.js

@@ -194,18 +194,6 @@ function handlePaintFloodFill({ labels, label, labelConstraint, pointList, brush
   }
 }
 
-// XXX: Currently assuming z slice
-function handleErase({ pointList, brush }) {
-  if (pointList.length === 0) return;
-
-  // Paint points
-  pointList.forEach((point, i) => {
-    paint({ point, brush });
-
-    if (i === 0) globals.prevPoint = null;
-  });
-}
-
 function handleCrop({ p1, p2 }) {
   const xStart = p1[0];
   const xStop = p2[0];

client/src/vtk/widgets/brush-3d-representation/brush-3d-representation.js

@@ -146,4 +146,5 @@ export const newInstance = macro.newInstance(
   'vtkBrush3DRepresentation'
 );
 
+// eslint-disable-next-line import/no-anonymous-default-export
 export default { newInstance, extend };

client/src/vtk/widgets/brush-3d-widget/behavior.js

@@ -1,6 +1,6 @@
 import macro from '@kitware/vtk.js/macros';
 import { vec3 } from 'gl-matrix';
-import { getSurfaceLabel } from 'vtk/widgets/widget-utils';
+//import { getSurfaceLabel } from 'vtk/widgets/widget-utils';
 
 const toVoxelCenter = (p, spacing, extent) => {
   return p.map((v, i) => {