Use effects for setroot instead of cached state

flopes89 · Feb 14, 2020 · e8e1cf7 · e8e1cf7
1 parent 6ddb99a
commit e8e1cf7
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 4 deletions.
diff --git a/src/client/components/files/filter.tsx b/src/client/components/files/filter.tsx
@@ -36,7 +36,7 @@ export const Filter: React.FC = () => {
 
     return (
         <Row>
-            <Col xs={3}>
+            <Col xs={1}>
                 <strong>Filters</strong>
             </Col>
             <Col>

diff --git a/src/client/components/files/index.tsx b/src/client/components/files/index.tsx
@@ -30,7 +30,7 @@ const FileList: React.FC = () => {
                 <Col>
                     <Filter />
                 </Col>
-                <Col xs={4} className="text-right">
+                <Col xs={2} className="text-right">
                     <Label check className="pl-2">
                         <Input
                             type="checkbox"

diff --git a/src/client/components/global/setRoot.tsx b/src/client/components/global/setRoot.tsx
@@ -1,4 +1,4 @@
-import React, { useState } from "react";
+import React, { useState, useEffect } from "react";
 import { Button, ButtonGroup, Col, Modal, ModalBody, ModalHeader, Row } from "reactstrap";
 import Octicon, { Inbox } from "@primer/octicons-react";
 import { Loading } from "../util";
@@ -109,7 +109,11 @@ const FolderSelection: React.FC<FolderSelectionProps> = (props) => {
 
 export const SetRoot: React.FC = () => {
     const { data } = useConfigQuery();
-    const [isOpen, setIsOpen] = useState(Boolean(data?.config.root));
+    const [isOpen, setIsOpen] = useState(false);
+
+    useEffect(() => {
+        setIsOpen(!Boolean(data?.config.root));
+    }, [data]);
 
     if (!data) {
         return <Loading />;