NCL-7216 Render 'Add selected' and 'Remove selected' buttons only whe…

…n data are available
project-ncl · Oct 4, 2023 · 1fe6be8 · 1fe6be8
1 parent 6f617e9
commit 1fe6be8
Show file tree

Hide file tree

Showing 2 changed files with 42 additions and 38 deletions.
diff --git a/src/components/ProductVersionBuildConfigsEditPage/BuildConfigsAddList.tsx b/src/components/ProductVersionBuildConfigsEditPage/BuildConfigsAddList.tsx
@@ -94,27 +94,29 @@ export const BuildConfigsAddList = ({
         </ToolbarItem>
       </Toolbar>
 
-      <Toolbar disablePaddingTop>
-        <ToolbarItem>
-          <Button
-            variant="tertiary"
-            onClick={() => {
-              checkedItems.forEach((checkedItem) => {
-                onBuildConfigAdd(checkedItem);
-                toggleItemCheck(checkedItem, false);
-              });
-            }}
-            isDisabled={!checkedItems.length}
-          >
-            Add selected
-          </Button>
-        </ToolbarItem>
-        {!!checkedItems.length && (
+      {!!serviceContainerBuildConfigs.data?.content?.length && (
+        <Toolbar disablePaddingTop>
           <ToolbarItem>
-            <b>{checkedItems.length}</b> to be added
+            <Button
+              variant="tertiary"
+              onClick={() => {
+                checkedItems.forEach((checkedItem) => {
+                  onBuildConfigAdd(checkedItem);
+                  toggleItemCheck(checkedItem, false);
+                });
+              }}
+              isDisabled={!checkedItems.length}
+            >
+              Add selected
+            </Button>
           </ToolbarItem>
-        )}
-      </Toolbar>
+          {!!checkedItems.length && (
+            <ToolbarItem>
+              <b>{checkedItems.length}</b> to be added
+            </ToolbarItem>
+          )}
+        </Toolbar>
+      )}
 
       <ContentBox borderTop>
         <ServiceContainerLoading {...serviceContainerBuildConfigs} title={PageTitles.buildConfigs}>

diff --git a/src/components/ProductVersionBuildConfigsEditPage/BuildConfigsRemoveList.tsx b/src/components/ProductVersionBuildConfigsEditPage/BuildConfigsRemoveList.tsx
@@ -90,27 +90,29 @@ export const BuildConfigsRemoveList = ({
         </ToolbarItem>
       </Toolbar>
 
-      <Toolbar disablePaddingTop>
-        <ToolbarItem>
-          <Button
-            variant="tertiary"
-            onClick={() => {
-              checkedItems.forEach((checkedItem) => {
-                onBuildConfigRemove(checkedItem);
-                toggleItemCheck(checkedItem, false);
-              });
-            }}
-            isDisabled={!checkedItems.length}
-          >
-            Remove selected
-          </Button>
-        </ToolbarItem>
-        {!!checkedItems.length && (
+      {!!serviceContainerBuildConfigs.data?.content?.length && (
+        <Toolbar disablePaddingTop>
           <ToolbarItem>
-            <b>{checkedItems.length}</b> to be removed
+            <Button
+              variant="tertiary"
+              onClick={() => {
+                checkedItems.forEach((checkedItem) => {
+                  onBuildConfigRemove(checkedItem);
+                  toggleItemCheck(checkedItem, false);
+                });
+              }}
+              isDisabled={!checkedItems.length}
+            >
+              Remove selected
+            </Button>
           </ToolbarItem>
-        )}
-      </Toolbar>
+          {!!checkedItems.length && (
+            <ToolbarItem>
+              <b>{checkedItems.length}</b> to be removed
+            </ToolbarItem>
+          )}
+        </Toolbar>
+      )}
 
       <ContentBox borderTop>
         <ServiceContainerLoading {...serviceContainerBuildConfigs} title={PageTitles.buildConfigs}>