From 58e83d9dccffd64a2e76da334c4c9299cbd4c346 Mon Sep 17 00:00:00 2001
From: Filip Hlavac <filip.hlavac01@gmail.com>
Date: Thu, 9 Nov 2023 18:37:23 +0100
Subject: [PATCH 1/6] fix(props): Add iconSize param to TagCount

---
 packages/module/src/TagCount/TagCount.tsx | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/packages/module/src/TagCount/TagCount.tsx b/packages/module/src/TagCount/TagCount.tsx
index c0f09757..a854cb69 100644
--- a/packages/module/src/TagCount/TagCount.tsx
+++ b/packages/module/src/TagCount/TagCount.tsx
@@ -1,6 +1,6 @@
 import React from 'react';
 import clsx from 'clsx';
-import { Button, ButtonProps, Icon } from '@patternfly/react-core';
+import { BaseSizes, Button, ButtonProps, Icon, IconComponentProps, IconProps } from '@patternfly/react-core';
 import { TagIcon } from '@patternfly/react-icons';
 import { createUseStyles } from 'react-jss'
 
@@ -21,17 +21,20 @@ export interface TagCountProps extends ButtonProps {
   count?: number;
   /** Additional classes added to the tag count component */
   className?: string;
+  /** Icon size */
+  iconSize?: 'sm' | 'md' | 'lg' | 'xl';
 }
 
 const TagCount: React.FunctionComponent<TagCountProps> = (
   { count, 
     className,
+    iconSize= 'md',
     ...props }: TagCountProps) => {
   const classes = useStyles();
   const tagClasses = clsx(classes.buttonTagCount, className);
   return (
     <Button aria-label="Tag count" {...props} variant="plain" isDisabled={!count} className={tagClasses}>
-      <Icon size="md">
+      <Icon iconSize={iconSize} >
         <TagIcon/>
       </Icon>
       <span className={classes.tagText}>{count}</span>

From dc671422f789a0b0e0325bafe5c708f28e1f6354 Mon Sep 17 00:00:00 2001
From: Filip Hlavac <filip.hlavac01@gmail.com>
Date: Thu, 9 Nov 2023 18:45:16 +0100
Subject: [PATCH 2/6] fix(export): Fix SkeletonTable export

---
 packages/module/src/SkeletonTable/index.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/packages/module/src/SkeletonTable/index.ts b/packages/module/src/SkeletonTable/index.ts
index 7e0a19d0..0e7ab23e 100644
--- a/packages/module/src/SkeletonTable/index.ts
+++ b/packages/module/src/SkeletonTable/index.ts
@@ -1,2 +1,2 @@
 export { default } from './SkeletonTable';
-export { default as SkeletonTable } from './SkeletonTable';
+export * from './SkeletonTable';

From 489c8c3168edd8996218dd552c30aeb9e12ef5c7 Mon Sep 17 00:00:00 2001
From: Filip Hlavac <filip.hlavac01@gmail.com>
Date: Mon, 13 Nov 2023 18:38:17 +0100
Subject: [PATCH 3/6] fix(props): Allow passing className to ErrorStack

---
 packages/module/src/ErrorBoundary/ErrorBoundary.tsx |  2 +-
 packages/module/src/ErrorStack/ErrorStack.tsx       | 11 ++++++++---
 2 files changed, 9 insertions(+), 4 deletions(-)

diff --git a/packages/module/src/ErrorBoundary/ErrorBoundary.tsx b/packages/module/src/ErrorBoundary/ErrorBoundary.tsx
index 9c7846f6..9c47cb29 100644
--- a/packages/module/src/ErrorBoundary/ErrorBoundary.tsx
+++ b/packages/module/src/ErrorBoundary/ErrorBoundary.tsx
@@ -4,7 +4,7 @@ import ErrorState from '../ErrorState';
 import ErrorStack from '../ErrorStack';
 
 export interface ErrorPageProps {
-  /** The title to display on the error page */
+  /** Title to display on the error page */
   headerTitle: string;
   /** Indicates if this is a silent error */
   silent?: boolean;
diff --git a/packages/module/src/ErrorStack/ErrorStack.tsx b/packages/module/src/ErrorStack/ErrorStack.tsx
index 49ad4712..1d84bb1d 100644
--- a/packages/module/src/ErrorStack/ErrorStack.tsx
+++ b/packages/module/src/ErrorStack/ErrorStack.tsx
@@ -1,9 +1,13 @@
 import * as React from 'react';
+import clsx from 'clsx';
 import { Text } from '@patternfly/react-core';
 import { createUseStyles } from 'react-jss';
 
 export interface ErrorStackProps {
+  /** Error object to be displayed in the stack */
   error: Error;
+  /** Custom className */
+  className?: string;
 }
 
 const useStyles = createUseStyles({
@@ -19,11 +23,12 @@ const useStyles = createUseStyles({
   },
 })
 
-export const ErrorStack: React.FunctionComponent<ErrorStackProps> = ({ error }) => {
+export const ErrorStack: React.FunctionComponent<ErrorStackProps> = ({ error, className, ...props }) => {
   const classes = useStyles();
+
   if (error.stack) {
     return (
-      <Text className={classes.errorStack}>
+      <Text className={clsx(classes.errorStack, className)} {...props} >
         {error.stack.split('\n').map((line) => (
           <div key={line}>{line}</div>
         ))}
@@ -35,7 +40,7 @@ export const ErrorStack: React.FunctionComponent<ErrorStackProps> = ({ error })
     return (
       <>
         <Text component="h6">{error.name}</Text>
-        <Text className={classes.errorStack} component="blockquote">
+        <Text className={clsx(classes.errorStack, className)} component="blockquote" {...props}>
           {error.message}
         </Text>
       </>

From 3454a3c75304ce3e8f98161d1434ea567a4e911d Mon Sep 17 00:00:00 2001
From: Filip Hlavac <filip.hlavac01@gmail.com>
Date: Mon, 13 Nov 2023 18:42:02 +0100
Subject: [PATCH 4/6] fix(imports): Remove extra imports

---
 packages/module/src/TagCount/TagCount.tsx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/packages/module/src/TagCount/TagCount.tsx b/packages/module/src/TagCount/TagCount.tsx
index a854cb69..9c015332 100644
--- a/packages/module/src/TagCount/TagCount.tsx
+++ b/packages/module/src/TagCount/TagCount.tsx
@@ -1,6 +1,6 @@
 import React from 'react';
 import clsx from 'clsx';
-import { BaseSizes, Button, ButtonProps, Icon, IconComponentProps, IconProps } from '@patternfly/react-core';
+import { Button, ButtonProps, Icon } from '@patternfly/react-core';
 import { TagIcon } from '@patternfly/react-icons';
 import { createUseStyles } from 'react-jss'
 

From 3d3702e7aaba8867f982c4a96f234cb73f0964e1 Mon Sep 17 00:00:00 2001
From: Filip Hlavac <filip.hlavac01@gmail.com>
Date: Tue, 14 Nov 2023 15:14:40 +0100
Subject: [PATCH 5/6] fix(tests): Update TagCount snapshot

---
 .../__snapshots__/TagCount.test.tsx.snap         | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/packages/module/src/TagCount/__snapshots__/TagCount.test.tsx.snap b/packages/module/src/TagCount/__snapshots__/TagCount.test.tsx.snap
index 481232f9..4ea97b3c 100644
--- a/packages/module/src/TagCount/__snapshots__/TagCount.test.tsx.snap
+++ b/packages/module/src/TagCount/__snapshots__/TagCount.test.tsx.snap
@@ -16,10 +16,10 @@ exports[`TagCount component should render a disabled tag count with no value 1`]
         type="button"
       >
         <span
-          class="pf-v5-c-icon pf-m-md"
+          class="pf-v5-c-icon"
         >
           <span
-            class="pf-v5-c-icon__content"
+            class="pf-v5-c-icon__content pf-m-md"
           >
             <svg
               aria-hidden="true"
@@ -54,10 +54,10 @@ exports[`TagCount component should render a disabled tag count with no value 1`]
       type="button"
     >
       <span
-        class="pf-v5-c-icon pf-m-md"
+        class="pf-v5-c-icon"
       >
         <span
-          class="pf-v5-c-icon__content"
+          class="pf-v5-c-icon__content pf-m-md"
         >
           <svg
             aria-hidden="true"
@@ -148,10 +148,10 @@ exports[`TagCount component should render a tag count of 11 1`] = `
         type="button"
       >
         <span
-          class="pf-v5-c-icon pf-m-md"
+          class="pf-v5-c-icon"
         >
           <span
-            class="pf-v5-c-icon__content"
+            class="pf-v5-c-icon__content pf-m-md"
           >
             <svg
               aria-hidden="true"
@@ -187,10 +187,10 @@ exports[`TagCount component should render a tag count of 11 1`] = `
       type="button"
     >
       <span
-        class="pf-v5-c-icon pf-m-md"
+        class="pf-v5-c-icon"
       >
         <span
-          class="pf-v5-c-icon__content"
+          class="pf-v5-c-icon__content pf-m-md"
         >
           <svg
             aria-hidden="true"

From a5f687268f3972f7ec957794d336cd9625b58ec5 Mon Sep 17 00:00:00 2001
From: Filip Hlavac <filip.hlavac01@gmail.com>
Date: Tue, 14 Nov 2023 15:17:49 +0100
Subject: [PATCH 6/6] fix(props): Update primaryAction description

---
 packages/module/src/NotAuthorized/NotAuthorized.tsx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/packages/module/src/NotAuthorized/NotAuthorized.tsx b/packages/module/src/NotAuthorized/NotAuthorized.tsx
index f4691095..dcb86117 100644
--- a/packages/module/src/NotAuthorized/NotAuthorized.tsx
+++ b/packages/module/src/NotAuthorized/NotAuthorized.tsx
@@ -17,7 +17,7 @@ export interface NotAuthorizedProps extends Omit<EmptyStateProps, 'children' | '
   className?: string;
   /** Custom title */
   title?: React.ReactNode;
-  /** Custom primary action there should only be one primary action*/
+  /** Custom primary action - there should only be one defined */
   primaryAction?: React.ReactNode;
   /** Custom secondary actions */
   secondaryActions?: React.ReactNode;