Merge pull request #936 from ethereum/lintWorkspace

Linting react workspace
4 years ago · 432220c18b
parent c4c9297783 cbfd940958
commit 432220c18b
3 changed files with 95 additions and 95 deletions
--- a/libs/remix-ui/workspace/src/index.ts
+++ b/libs/remix-ui/workspace/src/index.ts
@ -1 +1 @@
-export * from './lib/remix-ui-workspace';
+export * from './lib/remix-ui-workspace'
--- a/libs/remix-ui/workspace/src/lib/remix-ui-workspace.tsx
+++ b/libs/remix-ui/workspace/src/lib/remix-ui-workspace.tsx
@ -1,6 +1,6 @@
-import React, { useState, useEffect, useRef } from 'react';
+import React, { useState, useEffect, useRef } from 'react' // eslint-disable-line
 import { FileExplorer } from '@remix-ui/file-explorer' // eslint-disable-line
-import './remix-ui-workspace.css';
+import './remix-ui-workspace.css'
 import { ModalDialog } from '@remix-ui/modal-dialog' // eslint-disable-line

 /* eslint-disable-next-line */
@ -79,11 +79,11 @@ export const Workspace = (props: WorkspaceProps) => {
  }, [props.workspaces])

  useEffect(() => {
-    props.localhost.event.register('connected', (event) => {
+    props.localhost.event.register('connected', () => {
      remixdExplorer.show()
    })
-  
-    props.localhost.event.register('disconnected', (event) => {
+
+    props.localhost.event.register('disconnected', () => {
      remixdExplorer.hide()
    })

@ -193,15 +193,15 @@ export const Workspace = (props: WorkspaceProps) => {
    props.browser.remove(workspacesPath + '/' + state.currentWorkspace)
    const name = state.currentWorkspace
    setWorkspace(NO_WORKSPACE)
-    props.workspaceDeleted({ name })    
+    props.workspaceDeleted({ name })
  }
-  /**** ****/
+  /** ** ****/

  const resetFocus = (reset) => {
    setState(prevState => {
      return { ...prevState, reset }
    })
-  }  
+  }

  const setWorkspace = async (name) => {
    if (name === LOCALHOST) {
@ -280,77 +280,77 @@ export const Workspace = (props: WorkspaceProps) => {

  //   setWorkspace(value)
  // }
-  
+
  return (
    <div className='remixui_container'>
-        <ModalDialog
-          id='workspacesModalDialog'
-          title={ state.modal.title }
-          message={ state.modal.message }
-          hide={ state.modal.hide }
-          ok={ state.modal.ok }
-          cancel={ state.modal.cancel }
-          handleHide={ handleHideModal }>
-            { (typeof state.modal.message !== 'string') && state.modal.message }
-        </ModalDialog>
-        <div className='remixui_fileexplorer' onClick={() => resetFocus(true)}>
-          <div>
-            <header>
-              <div className="mb-2">
-                <label className="form-check-label" htmlFor="workspacesSelect">
+      <ModalDialog
+        id='workspacesModalDialog'
+        title={ state.modal.title }
+        message={ state.modal.message }
+        hide={ state.modal.hide }
+        ok={ state.modal.ok }
+        cancel={ state.modal.cancel }
+        handleHide={ handleHideModal }>
+        { (typeof state.modal.message !== 'string') && state.modal.message }
+      </ModalDialog>
+      <div className='remixui_fileexplorer' onClick={() => resetFocus(true)}>
+        <div>
+          <header>
+            <div className="mb-2">
+              <label className="form-check-label" htmlFor="workspacesSelect">
                Workspaces
-                </label>
-                <span className="remixui_menu">
-                  <span
-                    id='workspaceCreate'
-                    data-id='workspaceCreate'
-                    onClick={(e) => {
-                      e.stopPropagation()
-                      createWorkspace()
-                    }}
-                    className='far fa-plus-square remixui_menuicon'
-                    title='Create a new Workspace'>
-                  </span>
-                  <span
-                    hidden={state.currentWorkspace === LOCALHOST || state.currentWorkspace === NO_WORKSPACE}
-                    id='workspaceRename'
-                    data-id='workspaceRename'
-                    onClick={(e) => {
-                      e.stopPropagation()
-                      renameCurrentWorkspace()
-                    }}
-                    className='far fa-edit remixui_menuicon'
-                    title='Rename current Workspace'>
-                  </span>
-                  <span
-                    hidden={state.currentWorkspace === LOCALHOST || state.currentWorkspace === NO_WORKSPACE}
-                    id='workspaceDelete'
-                    data-id='workspaceDelete'
-                    onClick={(e) => {
-                      e.stopPropagation()
-                      deleteCurrentWorkspace()
-                    }}
-                    className='fas fa-trash'
-                    title='Delete current Workspace'>
-                  </span>
+              </label>
+              <span className="remixui_menu">
+                <span
+                  id='workspaceCreate'
+                  data-id='workspaceCreate'
+                  onClick={(e) => {
+                    e.stopPropagation()
+                    createWorkspace()
+                  }}
+                  className='far fa-plus-square remixui_menuicon'
+                  title='Create a new Workspace'>
                </span>
-                <select id="workspacesSelect" data-id="workspacesSelect" onChange={(e) => setWorkspace(e.target.value)} className="form-control custom-select">
+                <span
+                  hidden={state.currentWorkspace === LOCALHOST || state.currentWorkspace === NO_WORKSPACE}
+                  id='workspaceRename'
+                  data-id='workspaceRename'
+                  onClick={(e) => {
+                    e.stopPropagation()
+                    renameCurrentWorkspace()
+                  }}
+                  className='far fa-edit remixui_menuicon'
+                  title='Rename current Workspace'>
+                </span>
+                <span
+                  hidden={state.currentWorkspace === LOCALHOST || state.currentWorkspace === NO_WORKSPACE}
+                  id='workspaceDelete'
+                  data-id='workspaceDelete'
+                  onClick={(e) => {
+                    e.stopPropagation()
+                    deleteCurrentWorkspace()
+                  }}
+                  className='fas fa-trash'
+                  title='Delete current Workspace'>
+                </span>
+              </span>
+              <select id="workspacesSelect" 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) => {
+                      return <option selected={state.currentWorkspace === folder} value={folder}>{folder}</option>
+                    })
                }
                <option selected={state.currentWorkspace === LOCALHOST} value={LOCALHOST}>{LOCALHOST}</option>
                { state.workspaces.length <= 0 && <option selected={state.currentWorkspace === NO_WORKSPACE} value={NO_WORKSPACE}>{NO_WORKSPACE}</option> }
-                </select>
-              </div>
-            </header>
-          </div>
-          <div className='remixui_fileExplorerTree'>
-            <div>
-              <div className='pl-2 remixui_treeview' data-id='filePanelFileExplorerTree'>
-                { state.hideRemixdExplorer && state.currentWorkspace && state.currentWorkspace !== NO_WORKSPACE && state.currentWorkspace !== LOCALHOST &&
+              </select>
+            </div>
+          </header>
+        </div>
+        <div className='remixui_fileExplorerTree'>
+          <div>
+            <div className='pl-2 remixui_treeview' data-id='filePanelFileExplorerTree'>
+              { state.hideRemixdExplorer && state.currentWorkspace && state.currentWorkspace !== NO_WORKSPACE && state.currentWorkspace !== LOCALHOST &&
                  <FileExplorer
                    name={state.currentWorkspace}
                    registry={props.registry}
@ -362,10 +362,10 @@ export const Workspace = (props: WorkspaceProps) => {
                    displayInput={state.displayNewFile}
                    externalUploads={state.uploadFileEvent}
                  />
-                }
-              </div>
-              <div className='pl-2 filesystemexplorer remixui_treeview'>
-                { !state.hideRemixdExplorer &&
+              }
+            </div>
+            <div className='pl-2 filesystemexplorer remixui_treeview'>
+              { !state.hideRemixdExplorer &&
                  <FileExplorer
                    name='localhost'
                    registry={props.registry}
@ -375,27 +375,27 @@ export const Workspace = (props: WorkspaceProps) => {
                    focusRoot={state.reset}
                    contextMenuItems={props.registeredMenuItems}
                  />
-                }
-              </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 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>
-  );
-};
+    </div>
+  )
+}

-export default Workspace;
+export default Workspace
--- a/package.json
+++ b/package.json
@ -41,7 +41,7 @@
    "workspace-schematic": "nx workspace-schematic",
    "dep-graph": "nx dep-graph",
    "help": "nx help",
-    "lint:libs": "nx run-many --target=lint --projects=remix-analyzer,remix-astwalker,remix-debug,remix-lib,remix-simulator,remix-solidity,remix-tests,remix-url-resolver,remixd,remix-ui-tree-view,remix-ui-modal-dialog,remix-ui-toaster,remix-ui-file-explorer,remix-ui-debugger-ui",
+    "lint:libs": "nx run-many --target=lint --projects=remix-analyzer,remix-astwalker,remix-debug,remix-lib,remix-simulator,remix-solidity,remix-tests,remix-url-resolver,remixd,remix-ui-tree-view,remix-ui-modal-dialog,remix-ui-toaster,remix-ui-file-explorer,remix-ui-debugger-ui,remix-ui-workspace",
    "build:libs": "nx run-many --target=build --parallel=false --with-deps=true --projects=remix-analyzer,remix-astwalker,remix-debug,remix-lib,remix-simulator,remix-solidity,remix-tests,remix-url-resolver,remixd",
    "test:libs": "nx run-many --target=test --projects=remix-analyzer,remix-astwalker,remix-debug,remix-lib,remix-simulator,remix-solidity,remix-tests,remix-url-resolver,remixd",
    "publish:libs": "npm run build:libs & lerna publish --skip-git & npm run bumpVersion:libs",