remaining consoles removed

Author: LALeer

app/src/components/ContextAPIManager/CreateTab/components/AddContextForm.tsx

@@ -25,9 +25,6 @@ const AddContextForm = ({
   const { allContext } = contextStore;
   const [btnDisabled, setBtnDisabled] = useState(false);
   const [open, setOpen] = useState(false);
-  // const { state } = useSelector((store: RootState) => ({
-  //   state: store.appState
-  // }));
 
   const color = 'white';
 

app/src/components/bottom/StylesEditor.tsx

@@ -23,7 +23,7 @@ const StylesEditor: React.FC<{
     (state: RootState) => state.appState.stylesheet
   );
   const roomCode = useSelector((store: RootState) => store.roomSlice.roomCode);
-  
+
   //sets state for what text is currently in the csseditor
   const [css, setCss] = useState(stylesheet);
 
@@ -35,7 +35,6 @@ const StylesEditor: React.FC<{
     dispatch(updateStylesheet(css));
     if (roomCode) {
       emitEvent('updateCSSAction', roomCode, css);
-      console.log('emit updateCSSAction event is triggered in StylesEditor.tsx');
     }
   };
 

app/src/components/left/DragDropPanel.tsx

@@ -22,13 +22,15 @@ Hook state:
 const DragDropPanel = (props): JSX.Element => {
   const dispatch = useDispatch();
 
-  const state = useSelector((store: RootState) => store.appState);
-  const contextParam = useSelector((store: RootState) => store.contextSlice);
-  const roomCode = useSelector((store: RootState) => store.roomSlice.roomCode);
+  const state = useSelector((store: RootState) => store.appState); // current state
+  const contextParam = useSelector((store: RootState) => store.contextSlice); // current contextParam
+  const roomCode = useSelector((store: RootState) => store.roomSlice.roomCode); // current roomCode
 
+  // function for delete element
   const handleDelete = (id: number): void => {
     dispatch(deleteElement({ id: id, contextParam: contextParam }));
     if (roomCode) {
+      // send delete element to server to broadcast to all users
       emitEvent('deleteElementAction', roomCode, {
         id,
         contextParam

app/src/components/main/Canvas.tsx

@@ -26,7 +26,6 @@ function Canvas(props: {}): JSX.Element {
   const userList = useSelector((store: RootState) => store.roomSlice.userList);
 
   //-------cursors tracking-------
-  console.log('canvas is rendered');
 
   //remote cursor data
   const [remoteCursors, setRemoteCursors] = useState([]);
@@ -57,7 +56,6 @@ function Canvas(props: {}): JSX.Element {
       const cursorIdx = prevState.findIndex(
         (cursor) => cursor.remoteUserName === remoteData.userName
       );
-      //[{x,y,remoteUserName, isVisible}, {...}, {...}]
       //existing user
       if (cursorIdx >= 0) {
         //check if cursor position has changed
@@ -93,7 +91,7 @@ function Canvas(props: {}): JSX.Element {
   };
 
   // console.log('userList:', userList);
-  //[{x,y,remoteUserName, isVisible}, {...}, {...}]
+  //[{x,y,remoteUserName, isVisible}, {...}, {...}];
   // console.log('remoteCursors:', remoteCursors);
 
   // Removes the mouse cursor of the user that leaves the collaboration room.
@@ -145,27 +143,27 @@ function Canvas(props: {}): JSX.Element {
   const socket = getSocket();
   //wrap the socket event listener in useEffect with dependency array as [socket], so the the effect will run only when: 1. After the initial rendering of the component 2. Every time the socket instance changes(connect, disconnect)
   useEffect(() => {
-    console.log(
-      'socket inside useEffect:',
-      socket ? 'connected' : 'not connected'
-    );
+    // console.log(
+    //   'socket inside useEffect:',
+    //   socket ? 'connected' : 'not connected'
+    // );
 
     if (socket) {
-      console.log('------setting up socket.on event listener-------');
+      // console.log('------setting up socket.on event listener-------');
       socket.on('remote cursor data from server', (remoteData) =>
         handleCursorDataFromServer(remoteData)
       );
     }
 
     return () => {
-      console.log('clean up cursor event listener after canvas unmount');
+      // console.log('clean up cursor event listener after canvas unmount');
       if (socket) socket.off('remote cursor data from server');
     };
   }, [socket]);
 
   useEffect(() => {
     handleCursorDeleteFromServer();
-    console.log('handle delete has been called');
+    // console.log('handle delete has been called');
   }, [userList]);
 
   //-----------------
@@ -246,9 +244,9 @@ function Canvas(props: {}): JSX.Element {
             contextParam: contextParam
           });
 
-          console.log(
-            `emit addChildAction event is triggered in canvas from ${socket.id}`
-          );
+          // console.log(
+          //   `emit addChildAction event is triggered in canvas from ${socket.id}`
+          // );
         }
       } else if (item.newInstance && item.instanceType === 'Component') {
         let hasDiffParent = false;
@@ -297,7 +295,7 @@ function Canvas(props: {}): JSX.Element {
             contextParam: contextParam
           });
 
-          console.log('emit addChildAction event is triggered in canvas');
+          // console.log('emit addChildAction event is triggered in canvas');
         }
       }
     },

app/src/components/main/DeleteButton.tsx

@@ -20,9 +20,9 @@ function DeleteButton({ id, name }: DeleteButtons) {
         contextParam
       });
 
-      console.log(
-        'emit deleteChildAction event is triggered in DeleteButton.tsx'
-      );
+      // console.log(
+      //   'emit deleteChildAction event is triggered in DeleteButton.tsx'
+      // );
     }
   };
 

app/src/components/main/DirectChildComponent.tsx

@@ -42,7 +42,7 @@ function DirectChildComponent({ childId, type, typeId, name }: ChildElement) {
         componentId: componentId,
         childId: childId
       });
-      console.log('emit focus event from DirectChildComponent');
+      // console.log('emit focus event from DirectChildComponent');
     }
   };
 

app/src/components/main/DirectChildHTML.tsx

@@ -44,7 +44,7 @@ function DirectChildHTML({ childId, name, type, typeId, style }: ChildElement) {
         componentId: componentId,
         childId: childId
       });
-      console.log('emit focus event from DirectChildHTML');
+      // console.log('emit focus event from DirectChildHTML');
     }
   };
 

app/src/components/main/DirectChildHTMLNestable.tsx

@@ -115,9 +115,9 @@ function DirectChildHTMLNestable({
               contextParam: contextParam
             });
 
-            console.log(
-              'emit addChildAction event is triggered in DirectChildHTMLNestable'
-            );
+            // console.log(
+            //   'emit addChildAction event is triggered in DirectChildHTMLNestable'
+            // );
           }
         }
       }
@@ -139,9 +139,9 @@ function DirectChildHTMLNestable({
               contextParam: contextParam
             });
 
-            console.log(
-              'emit changePosition event is triggered in DirectChildHTMLNestable'
-            );
+            // console.log(
+            //   'emit changePosition event is triggered in DirectChildHTMLNestable'
+            // );
           }
         }
       }
@@ -161,7 +161,7 @@ function DirectChildHTMLNestable({
         componentId: componentId,
         childId: childId
       });
-      console.log('emit focus event from DirectChildHTMLNestable');
+      // console.log('emit focus event from DirectChildHTMLNestable');
     }
   };
 

app/src/components/main/SeparatorChild.tsx

@@ -91,9 +91,9 @@ function DirectChildHTMLNestable({
               contextParam: contextParam
             });
 
-            console.log(
-              'emit addChildAction event is triggered in SeparatorChild'
-            );
+            // console.log(
+            //   'emit addChildAction event is triggered in SeparatorChild'
+            // );
           }
         }
       }
@@ -115,9 +115,9 @@ function DirectChildHTMLNestable({
               contextParam: contextParam
             });
 
-            console.log(
-              'emit changePosition event is triggered in SeparatorChild'
-            );
+            // console.log(
+            //   'emit changePosition event is triggered in SeparatorChild'
+            // );
           }
         }
       }

app/src/helperFunctions/socket.ts

@@ -9,8 +9,8 @@ export const initializeSocket = () => {
     // will force new socket connection if re-joining to prevent double emits
     forceNew: true
   });
-  console.log('A user connected');
-  console.log('socket:', socket);
+  // console.log('A user connected');
+  // console.log('socket:', socket);
   // }
 };
 
@@ -22,7 +22,6 @@ export const getSocket = () => {
 export const disconnectSocket = () => {
   if (socket) {
     socket.disconnect();
-    socket = null;
   }
 };