From 24fa73d12b8ba906bff1948870ebe37e8ba45ed2 Mon Sep 17 00:00:00 2001
From: PKulkoRaccoonGang <peter.kulko@raccoongang.com>
Date: Thu, 21 Dec 2023 17:11:52 +0200
Subject: [PATCH] refactor: corrected PropTypes

---
 src/Button/index.jsx        | 4 ++--
 src/Dropzone/index.jsx      | 2 +-
 src/Icon/index.jsx          | 2 +-
 src/IconButton/index.jsx    | 2 +-
 src/Menu/MenuItem.jsx       | 4 ++--
 src/Overlay/README.md       | 2 +-
 src/Overlay/index.jsx       | 2 +-
 www/src/components/Menu.tsx | 4 ++--
 8 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/src/Button/index.jsx b/src/Button/index.jsx
index 7b497901ee..7180ff3bfa 100644
--- a/src/Button/index.jsx
+++ b/src/Button/index.jsx
@@ -52,10 +52,10 @@ Button.propTypes = {
   variant: PropTypes.string,
   /** An icon component to render.
   * Example import of a Paragon icon component: `import { Check } from '@edx/paragon/icons';` */
-  iconBefore: PropTypes.oneOfType([PropTypes.func, PropTypes.node]),
+  iconBefore: PropTypes.oneOfType([PropTypes.elementType, PropTypes.node]),
   /** An icon component to render.
   * Example import of a Paragon icon component: `import { Check } from '@edx/paragon/icons';` */
-  iconAfter: PropTypes.oneOfType([PropTypes.func, PropTypes.node]),
+  iconAfter: PropTypes.oneOfType([PropTypes.elementType, PropTypes.node]),
 };
 
 Button.defaultProps = {
diff --git a/src/Dropzone/index.jsx b/src/Dropzone/index.jsx
index 8712f54301..1eecc7ac1f 100644
--- a/src/Dropzone/index.jsx
+++ b/src/Dropzone/index.jsx
@@ -270,7 +270,7 @@ Dropzone.propTypes = {
    */
   validator: PropTypes.func,
   /** A component to display initial state of the `Dropzone`. */
-  inputComponent: PropTypes.oneOfType([PropTypes.func, PropTypes.node]),
+  inputComponent: PropTypes.oneOfType([PropTypes.elementType, PropTypes.node]),
 };
 
 export default Dropzone;
diff --git a/src/Icon/index.jsx b/src/Icon/index.jsx
index cb01fb1ad0..717f421c9a 100644
--- a/src/Icon/index.jsx
+++ b/src/Icon/index.jsx
@@ -72,7 +72,7 @@ function Icon({
 Icon.propTypes = {
   // eslint-disable-next-line max-len
   /** An icon component to render. Example import of a Paragon icon component: `import { Check } from '@edx/paragon/dist/icon';` */
-  src: PropTypes.oneOfType([PropTypes.element, PropTypes.func]),
+  src: PropTypes.oneOfType([PropTypes.element, PropTypes.elementType]),
   /** HTML element attributes to pass through to the underlying svg element */
   svgAttrs: PropTypes.shape({
     'aria-label': PropTypes.string,
diff --git a/src/IconButton/index.jsx b/src/IconButton/index.jsx
index acb22265e8..4a20174050 100644
--- a/src/IconButton/index.jsx
+++ b/src/IconButton/index.jsx
@@ -80,7 +80,7 @@ IconButton.propTypes = {
   /** An icon component to render. Example import of a Paragon icon component:
    * `import { Check } from '@edx/paragon/dist/icon';`
    * */
-  src: PropTypes.oneOfType([PropTypes.element, PropTypes.func]),
+  src: PropTypes.oneOfType([PropTypes.element, PropTypes.elementType]),
   /** Alt text for your icon. For best practice, avoid using alt text to describe
    * the image in the `IconButton`. Instead, we recommend describing the function
    * of the button. */
diff --git a/src/Menu/MenuItem.jsx b/src/Menu/MenuItem.jsx
index a8034a18e7..165fc8f5a2 100644
--- a/src/Menu/MenuItem.jsx
+++ b/src/Menu/MenuItem.jsx
@@ -40,9 +40,9 @@ MenuItem.propTypes = {
   /** Specifies the base element */
   as: PropTypes.elementType,
   /** Specifies the jsx before the content of the ``MenuItem`` */
-  iconBefore: PropTypes.oneOfType([PropTypes.element, PropTypes.func]),
+  iconBefore: PropTypes.oneOfType([PropTypes.element, PropTypes.elementType]),
   /** Specifies the jsx after the content of the ``MenuItem`` */
-  iconAfter: PropTypes.oneOfType([PropTypes.element, PropTypes.func]),
+  iconAfter: PropTypes.oneOfType([PropTypes.element, PropTypes.elementType]),
 };
 
 MenuItem.defaultProps = {
diff --git a/src/Overlay/README.md b/src/Overlay/README.md
index da86dc330c..af259d0d0d 100644
--- a/src/Overlay/README.md
+++ b/src/Overlay/README.md
@@ -38,7 +38,7 @@ This component is used to power Tooltips and Popovers.
           Click me to see
         </Button>
       </div>
-      <Overlay target={target.current} show={open} placement="right">
+      <Overlay target={target.current} show={isOpen} placement="right">
         {({ placement, arrowProps, show: _show, popper, ...props }) => (
           <div
             {...props}
diff --git a/src/Overlay/index.jsx b/src/Overlay/index.jsx
index 38ce1a0fce..6e3fb7c83d 100644
--- a/src/Overlay/index.jsx
+++ b/src/Overlay/index.jsx
@@ -89,7 +89,7 @@ Overlay.propTypes = {
    * Animate the entering and exiting of the Overlay. `true` will use the `<Fade>` transition,
    * or a custom react-transition-group `<Transition>` component can be provided.
    */
-  transition: PropTypes.func,
+  transition: PropTypes.oneOfType([PropTypes.object, PropTypes.bool]),
 };
 
 OverlayTrigger.propTypes = {
diff --git a/www/src/components/Menu.tsx b/www/src/components/Menu.tsx
index 74afa7a0b3..2bb44ffdfb 100644
--- a/www/src/components/Menu.tsx
+++ b/www/src/components/Menu.tsx
@@ -70,7 +70,7 @@ export interface IComponentNavItem {
     type: string,
     status?: string,
   },
-  isActive: boolean,
+  isActive?: boolean,
 }
 
 export function ComponentNavItem({
@@ -114,7 +114,7 @@ ComponentNavItem.propTypes = {
     title: PropTypes.string.isRequired,
     status: PropTypes.string,
   }).isRequired,
-  isActive: PropTypes.bool.isRequired,
+  isActive: PropTypes.bool,
 };
 
 export type MenuComponentListTypes = {