Only show loading state for the first list container call (#199)

Co-authored-by: Tio Pramayudi <[email protected]>
caraml-dev · Nov 29, 2021 · 4623dc6 · 4623dc6
1 parent 68a2d65
commit 4623dc6
Showing 1 changed file with 8 additions and 1 deletion.
diff --git a/ui/src/components/logs/ContainerLogsView.js b/ui/src/components/logs/ContainerLogsView.js
@@ -46,6 +46,8 @@ export const ContainerLogsView = ({
     tail_lines: "1000"
   });
 
+  const [containerHaveBeenLoaded, setContainerHaveBeenLoaded] = useState(false);
+
   const [
     { data: containers, isLoaded: containersLoaded },
     getContainers
@@ -65,6 +67,9 @@ export const ContainerLogsView = ({
 
   useEffect(
     () => {
+      if (!containerHaveBeenLoaded) {
+        setContainerHaveBeenLoaded(containersLoaded);
+      }
       if (containersLoaded && params.component_type === "") {
         if (
           containers.find(
@@ -176,7 +181,9 @@ export const ContainerLogsView = ({
       </EuiTitle>
 
       <EuiPageContent>
-        {componentTypes && componentTypes.length === 0 ? (
+        {!containerHaveBeenLoaded &&
+        componentTypes &&
+        componentTypes.length === 0 ? (
           <EuiLoadingContent lines={4} />
         ) : logUrl && stackdriverUrl ? (
           <EuiFlexGroup direction="column" gutterSize="none">