diff --git a/src/components/FileBrowser.tsx b/src/components/FileBrowser.tsx index 32ec9a5..a382a3a 100644 --- a/src/components/FileBrowser.tsx +++ b/src/components/FileBrowser.tsx @@ -8,7 +8,7 @@ import uploadFile from "../functions/uploadFile" import { useCopyMutation, useCreateDirMutation, useDeleteDirMutation, useDeleteFileMutation, useMoveMutation, useOpenDirQuery } from "../generated/graphql" import Breadcrum from "./Breadcrum" import CreateDirButton from "./CreateDirButton" -import DragAndDrop from "./DragAndDrop" +import GlobalDragAndDrop from "./GlobalDragAndDrop" import FileBrowserContextMenu, { Action, CONTEXT_MENU_DIR, CONTEXT_MENU_FILE } from "./FileBrowserContextMenu" import FileOpen from "./FileOpen" import FileUploadButton from "./FileUploadButton" @@ -103,84 +103,82 @@ const FileBrowser: React.FC = (props) => { onSelect={onContextSelect} pasteActive={!!srcID} /> - { await handleDrop(files) - }} - > -
- { - props.history.push(newPath.toURI()) - }}/> -
- { - const dirID = new ObjID(path.bucket,path.key + dirName + "/") - await createDirMutation({variables:{path: dirID}}) - refetchDir() - }} - /> -
-
- handleDrop(files)} - /> -
-
- -
- -
-
- { - props.history.push(path.toURI()) - }} - - onDirContext={(e,path)=>{ - e.preventDefault() - showDirContext(e,{props:{id:path}}) - }} - - onFileClick={(e,id)=>{ - setOpenFileId(id) - setShowFile(true) - }} - - onFileContext={(e,id)=>{ - e.preventDefault() - showFileContext(e,{props:{id}}) - }} - - editId={editID} - editEnable={editEnable} - - onRenameDone={async (id,changed,newName)=>{ - setEditEnable(false) - if (changed){ - // TODO: maybe change the name on client so it seems more smooth rather then haveing it refetch - // TODO: input check & error handling - await moveMutation({variables:{ - src:id, - dest: id.rename(newName) - }}) - refetchDir() - } + }}/> +
+ { + props.history.push(newPath.toURI()) + }}/> +
+ { + const dirID = new ObjID(path.bucket,path.key + dirName + "/") + await createDirMutation({variables:{path: dirID}}) + refetchDir() }} /> - {loading && -
- -
- }
- { - setShowFile(false) - }} /> - +
+ handleDrop(files)} + /> +
+
+ +
+ +
+
+ { + props.history.push(path.toURI()) + }} + + onDirContext={(e,path)=>{ + e.preventDefault() + showDirContext(e,{props:{id:path}}) + }} + + onFileClick={(e,id)=>{ + setOpenFileId(id) + setShowFile(true) + }} + + onFileContext={(e,id)=>{ + e.preventDefault() + showFileContext(e,{props:{id}}) + }} + + editId={editID} + editEnable={editEnable} + + onRenameDone={async (id,changed,newName)=>{ + setEditEnable(false) + if (changed){ + // TODO: maybe change the name on client so it seems more smooth rather then haveing it refetch + // TODO: input check & error handling + await moveMutation({variables:{ + src:id, + dest: id.rename(newName) + }}) + refetchDir() + } + }} + /> + {loading && +
+ +
+ } +
+ { + setShowFile(false) + }} />
) } diff --git a/src/components/DragAndDrop.tsx b/src/components/GlobalDragAndDrop.tsx similarity index 51% rename from src/components/DragAndDrop.tsx rename to src/components/GlobalDragAndDrop.tsx index 36baef3..1efb756 100644 --- a/src/components/DragAndDrop.tsx +++ b/src/components/GlobalDragAndDrop.tsx @@ -10,12 +10,10 @@ interface Props { handleDrop?: (files: FileList)=>void } -const DragAndDrop: React.FC = (props) => { - const dropRef = React.createRef() +const GlobalDragAndDrop: React.FC = (props) => { const [hover,setHover] = useState(false) function handleDragEnter(event: DragEvent) { - // console.debug("dragenter",event) event.preventDefault() event.stopPropagation() setHover(true) @@ -23,7 +21,6 @@ const DragAndDrop: React.FC = (props) => { } function handleDragLeave(event: DragEvent) { - // console.debug("dragleave",event) event.preventDefault() event.stopPropagation() setHover(false) @@ -38,7 +35,6 @@ const DragAndDrop: React.FC = (props) => { } function handleDrop(event: DragEvent) { - console.debug("drop",event) event.preventDefault() event.stopPropagation() setHover(false) @@ -49,31 +45,25 @@ const DragAndDrop: React.FC = (props) => { } useEffect(()=>{ - if(dropRef.current){ - dropRef.current.addEventListener("dragenter",handleDragEnter) - dropRef.current.addEventListener("dragleave",handleDragLeave) - dropRef.current.addEventListener("dragover",handleDragOver) - dropRef.current.addEventListener("drop",handleDrop) + document.addEventListener("dragenter",handleDragEnter) + document.addEventListener("dragleave",handleDragLeave) + document.addEventListener("dragover",handleDragOver) + document.addEventListener("drop",handleDrop) - return ()=>{ - if (dropRef.current){ - dropRef.current.removeEventListener("dragenter",handleDragEnter) - dropRef.current.removeEventListener("dragleave",handleDragLeave) - dropRef.current.removeEventListener("dragover",handleDragOver) - dropRef.current.removeEventListener("drop",handleDrop) - } - } + return ()=>{ + document.removeEventListener("dragenter",handleDragEnter) + document.removeEventListener("dragleave",handleDragLeave) + document.removeEventListener("dragover",handleDragOver) + document.removeEventListener("drop",handleDrop) + } + },[]) return ( -
- {props.children} -
+
) } -export default DragAndDrop +export default GlobalDragAndDrop