refactor: clean up useless comments

Author: atyrode

src/frontend/src/pad/editors/HtmlEditor.tsx

@@ -33,7 +33,7 @@ export const useHtmlEditor = (
   element: NonDeleted<ExcalidrawEmbeddableElement> | undefined,
   editorRef: React.RefObject<any>,
   excalidrawAPI?: any,
-  isActive: boolean = true // New parameter to control if the hook is active
+  isActive: boolean = true
 ) => {
   // Always initialize these hooks regardless of isActive
   const [createNew, setCreateNew] = useState(false);

src/frontend/src/ui/MainMenu.tsx

@@ -7,7 +7,7 @@ import { LogOut, SquarePlus, LayoutDashboard, User, Text, Settings, Terminal, Fi
 import md5 from 'crypto-js/md5';
 
 // Components
-import SettingsDialog from './SettingsDialog'; // Added import
+import SettingsDialog from './SettingsDialog';
 
 import { useLogout } from '../hooks/useLogout';
 import { useAuthStatus } from '../hooks/useAuthStatus';
@@ -135,7 +135,7 @@ export const MainMenuConfig: React.FC<MainMenuConfigProps> = ({
     setShowSettingsModal(true);
   };
 
-  const handleCloseSettingsModal = () => { // Added handler to close settings modal
+  const handleCloseSettingsModal = () => {
     setShowSettingsModal(false);
   };
 
@@ -151,7 +151,7 @@ export const MainMenuConfig: React.FC<MainMenuConfigProps> = ({
         const keycloakLogoutUrl = data.logout_url;
 
         const createIframeLoader = (url: string, debugName: string): Promise<void> => {
-          return new Promise<void>((resolve, reject) => { // Added reject for error handling
+          return new Promise<void>((resolve, reject) => {
             const iframe = document.createElement("iframe");
             iframe.style.display = "none";
             iframe.src = url;
@@ -204,7 +204,7 @@ export const MainMenuConfig: React.FC<MainMenuConfigProps> = ({
           onClose={() => setShowAccountModal(false)} 
         />
       )}
-      {showSettingsModal && ( // Added conditional rendering for SettingsDialog
+      {showSettingsModal && (
         <SettingsDialog
           excalidrawAPI={excalidrawAPI}
           onClose={handleCloseSettingsModal}

src/frontend/src/ui/TabContextMenu.tsx

@@ -219,7 +219,7 @@ class TabActionManager implements ActionManager {
     onRename: (padId: string, newName: string) => void,
     onDelete: (padId: string) => void, // This is for deleteOwnedPad
     onUpdateSharingPolicy: (padId: string, policy: string) => void,
-    onLeaveSharedPad: (padId: string) => void, // Moved before optional param
+    onLeaveSharedPad: (padId: string) => void,
     sharingPolicy?: string
   ) {
     this.padId = padId;
@@ -244,7 +244,7 @@ class TabActionManager implements ActionManager {
         console.debug('[pad.ws] User confirmed delete, calling onDelete');
         this.onDelete(this.padId); // Calls original onDelete for owned pads
       }
-    } else if (action.name === 'leaveSharedPad') { // New action for leaving
+    } else if (action.name === 'leaveSharedPad') {
       console.debug('[pad.ws] Attempting to leave shared pad:', this.padId, this.padName);
       if (window.confirm(`Are you sure you want to leave "${this.padName}"? This will remove it from your list of open pads.`)) {
         this.onLeaveSharedPad(this.padId); // Calls the new handler

src/frontend/src/ui/Tabs.tsx

@@ -20,7 +20,7 @@ interface TabsProps {
     createNewPadAsync: () => Promise<Tab | null | undefined>;
     renamePad: (args: { padId: string; newName: string }) => void;
     deletePad: (padId: string) => void;
-    leaveSharedPad: (padId: string) => void; // Added prop
+    leaveSharedPad: (padId: string) => void;
     updateSharingPolicy: (args: { padId: string; policy: string }) => void;
     selectTab: (tabId: string) => void;
 }
@@ -470,7 +470,7 @@ const Tabs: React.FC<TabsProps> = ({
                         }
                         deletePad(padId); // Calls the prop for deleting owned pad
                     }}
-                    onLeaveSharedPad={(padId: string) => { // New prop for 'leaveSharedPad'
+                    onLeaveSharedPad={(padId: string) => {
                         leaveSharedPad(padId); // Calls the prop for leaving shared pad
                     }}
                     onUpdateSharingPolicy={(padId: string, policy: string) => {