From 4c5832e0506f5d2fca7dfa873e793563a2ed9a82 Mon Sep 17 00:00:00 2001
From: ioedeveloper <daviddisu8@gmail.com>
Date: Wed, 24 Mar 2021 16:18:11 +0100
Subject: [PATCH] cleanup redundant code and console

---
 apps/remix-ide/src/migrateFileSystem.js       |  1 -
 .../workspace/src/lib/remix-ui-workspace.tsx  | 30 ++++---------------
 2 files changed, 5 insertions(+), 26 deletions(-)

diff --git a/apps/remix-ide/src/migrateFileSystem.js b/apps/remix-ide/src/migrateFileSystem.js
index 2d912ac3cb..ab47956743 100644
--- a/apps/remix-ide/src/migrateFileSystem.js
+++ b/apps/remix-ide/src/migrateFileSystem.js
@@ -15,7 +15,6 @@ export default (fileProvider) => {
       fileProvider.set(path, content)
       // TODO https://github.com/ethereum/remix-ide/issues/2377
       // fileStorage.remove(path) we don't want to remove it as we are still supporting the old version
-      console.log('file migrated', path)
     }
   })
   fileStorageBrowserFS.set(flag, 'done')
diff --git a/libs/remix-ui/workspace/src/lib/remix-ui-workspace.tsx b/libs/remix-ui/workspace/src/lib/remix-ui-workspace.tsx
index 42328b25c7..dbde392b20 100644
--- a/libs/remix-ui/workspace/src/lib/remix-ui-workspace.tsx
+++ b/libs/remix-ui/workspace/src/lib/remix-ui-workspace.tsx
@@ -299,12 +299,6 @@ export const Workspace = (props: WorkspaceProps) => {
     )
   }
 
-  // const handleWorkspaceSelect = (e) => {
-  //   const value = e.target.value
-
-  //   setWorkspace(value)
-  // }
-
   return (
     <div className='remixui_container'>
       <ModalDialog
@@ -358,15 +352,15 @@ export const Workspace = (props: WorkspaceProps) => {
                   title='Delete'>
                 </span>
               </span>
-              <select id="workspacesSelect" data-id="workspacesSelect" onChange={(e) => setWorkspace(e.target.value)} className="form-control custom-select">
+              <select id="workspacesSelect" value={state.currentWorkspace} data-id="workspacesSelect" onChange={(e) => setWorkspace(e.target.value)} className="form-control custom-select">
                 {
                   state.workspaces
-                    .map((folder) => {
-                      return <option selected={state.currentWorkspace === folder} value={folder}>{folder}</option>
+                    .map((folder, index) => {
+                      return <option key={index} value={folder}>{folder}</option>
                     })
                 }
-                <option selected={state.currentWorkspace === LOCALHOST} value={LOCALHOST}>{state.currentWorkspace === LOCALHOST ? 'localhost' : LOCALHOST}</option>
-                { state.workspaces.length <= 0 && <option selected={state.currentWorkspace === NO_WORKSPACE} value={NO_WORKSPACE}>{NO_WORKSPACE}</option> }
+                <option value={LOCALHOST}>{state.currentWorkspace === LOCALHOST ? 'localhost' : LOCALHOST}</option>
+                { state.workspaces.length <= 0 && <option value={NO_WORKSPACE}>{NO_WORKSPACE}</option> }
               </select>
             </div>
           </header>
@@ -404,20 +398,6 @@ export const Workspace = (props: WorkspaceProps) => {
                   }
                 </div>
             }
-            <div className='pl-2 remixui_treeview'>
-              { false && <FileExplorer
-                name='browser'
-                registry={props.registry}
-                filesProvider={props.browser}
-                menuItems={['createNewFile', 'createNewFolder', 'publishToGist', canUpload ? 'uploadFile' : '']}
-                plugin={props.plugin}
-                focusRoot={state.reset}
-                contextMenuItems={props.registeredMenuItems}
-                displayInput={state.displayNewFile}
-                externalUploads={state.uploadFileEvent}
-              />
-              }
-            </div>
           </div>
         </div>
       </div>