From ea9b42f6184b6df75e766ba4e76c1fd71daff1f3 Mon Sep 17 00:00:00 2001
From: CD Cabrera <ccabrera@redhat.com>
Date: Wed, 20 Jan 2021 11:21:34 -0500
Subject: [PATCH] fix(inventoryList): issues/476 show field on zero results
 (#556)

---
 .../__snapshots__/inventoryList.test.js.snap  | 19 ++++++++++++++-----
 src/components/inventoryList/inventoryList.js |  7 ++-----
 .../inventorySubscriptions.test.js.snap       | 19 ++++++++++++++-----
 .../inventorySubscriptions.js                 |  7 ++-----
 4 files changed, 32 insertions(+), 20 deletions(-)

diff --git a/src/components/inventoryList/__tests__/__snapshots__/inventoryList.test.js.snap b/src/components/inventoryList/__tests__/__snapshots__/inventoryList.test.js.snap
index 0b624a89b..ab3e4dd91 100644
--- a/src/components/inventoryList/__tests__/__snapshots__/inventoryList.test.js.snap
+++ b/src/components/inventoryList/__tests__/__snapshots__/inventoryList.test.js.snap
@@ -236,7 +236,10 @@ exports[`InventoryList Component should handle variations in data: filtered data
           viewId="inventoryList"
         />
       </CardHeaderMain>
-      <CardActions>
+      <CardActions
+        aria-hidden={false}
+        className=""
+      >
         <Pagination
           dropDirection="down"
           isCompact={true}
@@ -352,7 +355,10 @@ exports[`InventoryList Component should handle variations in data: variable data
           viewId="inventoryList"
         />
       </CardHeaderMain>
-      <CardActions>
+      <CardActions
+        aria-hidden={false}
+        className=""
+      >
         <Pagination
           dropDirection="down"
           isCompact={true}
@@ -454,8 +460,8 @@ exports[`InventoryList Component should render a non-connected component: non-co
     updateOnResize={true}
   >
     <CardHeader
-      aria-hidden={true}
-      className="transparent"
+      aria-hidden={false}
+      className=""
     >
       <CardHeaderMain>
         <ToolbarFieldDisplayName
@@ -464,7 +470,10 @@ exports[`InventoryList Component should render a non-connected component: non-co
           viewId="inventoryList"
         />
       </CardHeaderMain>
-      <CardActions>
+      <CardActions
+        aria-hidden={true}
+        className="transparent"
+      >
         <Pagination
           dropDirection="down"
           isCompact={true}
diff --git a/src/components/inventoryList/inventoryList.js b/src/components/inventoryList/inventoryList.js
index 7f080f10e..8f3072d2c 100644
--- a/src/components/inventoryList/inventoryList.js
+++ b/src/components/inventoryList/inventoryList.js
@@ -222,14 +222,11 @@ class InventoryList extends React.Component {
     return (
       <Card className="curiosity-inventory-card">
         <MinHeight key="headerMinHeight" updateOnContent>
-          <CardHeader
-            className={(error && 'hidden') || (!itemCount && 'transparent') || ''}
-            aria-hidden={error || !itemCount || false}
-          >
+          <CardHeader className={(error && 'hidden') || ''} aria-hidden={error || false}>
             <CardHeaderMain>
               <ToolbarFieldDisplayName viewId={viewId} />
             </CardHeaderMain>
-            <CardActions>
+            <CardActions className={(!itemCount && 'transparent') || ''} aria-hidden={!itemCount || false}>
               <Pagination
                 isCompact
                 isDisabled={pending || error}
diff --git a/src/components/inventorySubscriptions/__tests__/__snapshots__/inventorySubscriptions.test.js.snap b/src/components/inventorySubscriptions/__tests__/__snapshots__/inventorySubscriptions.test.js.snap
index 27beba990..14d7294bc 100644
--- a/src/components/inventorySubscriptions/__tests__/__snapshots__/inventorySubscriptions.test.js.snap
+++ b/src/components/inventorySubscriptions/__tests__/__snapshots__/inventorySubscriptions.test.js.snap
@@ -122,7 +122,10 @@ exports[`InventorySubscriptions Component should handle variations in data: filt
       aria-hidden={false}
       className=""
     >
-      <CardActions>
+      <CardActions
+        aria-hidden={false}
+        className=""
+      >
         <Pagination
           dropDirection="down"
           isCompact={true}
@@ -229,7 +232,10 @@ exports[`InventorySubscriptions Component should handle variations in data: vari
       aria-hidden={false}
       className=""
     >
-      <CardActions>
+      <CardActions
+        aria-hidden={false}
+        className=""
+      >
         <Pagination
           dropDirection="down"
           isCompact={true}
@@ -329,10 +335,13 @@ exports[`InventorySubscriptions Component should render a non-connected componen
     updateOnResize={true}
   >
     <CardHeader
-      aria-hidden={true}
-      className="transparent"
+      aria-hidden={false}
+      className=""
     >
-      <CardActions>
+      <CardActions
+        aria-hidden={true}
+        className="transparent"
+      >
         <Pagination
           dropDirection="down"
           isCompact={true}
diff --git a/src/components/inventorySubscriptions/inventorySubscriptions.js b/src/components/inventorySubscriptions/inventorySubscriptions.js
index bb94b7b9b..b6f749b06 100644
--- a/src/components/inventorySubscriptions/inventorySubscriptions.js
+++ b/src/components/inventorySubscriptions/inventorySubscriptions.js
@@ -200,11 +200,8 @@ class InventorySubscriptions extends React.Component {
     return (
       <Card className="curiosity-inventory-card">
         <MinHeight key="headerMinHeight" updateOnContent>
-          <CardHeader
-            className={(error && 'hidden') || (!itemCount && 'transparent') || ''}
-            aria-hidden={error || !itemCount || false}
-          >
-            <CardActions>
+          <CardHeader className={(error && 'hidden') || ''} aria-hidden={error || false}>
+            <CardActions className={(!itemCount && 'transparent') || ''} aria-hidden={!itemCount || false}>
               <Pagination
                 isCompact
                 isDisabled={pending || error}