fixup! fixup! Change default properties from null to undefined (#662)

Author: lukasbriza

src/components/Modal/Modal.jsx

@@ -121,7 +121,7 @@ export const Modal = ({
     onMouseDown,
   };
 
-  if (portalId === null) {
+  if (portalId === undefined) {
     return preRender(
       children,
       color,

src/components/Modal/_helpers/dialogOnCancelHandler.js

@@ -17,7 +17,7 @@ export const dialogOnCancelHandler = (e, closeButtonRef, onCancelHandler = undef
   e.preventDefault();
 
   // If the close button is not disabled, close the modal.
-  if (closeButtonRef?.current !== undefined && closeButtonRef?.current?.disabled === false) {
+  if (closeButtonRef?.current != null && closeButtonRef?.current?.disabled === false) {
     closeButtonRef.current.click();
   }
 

src/components/Modal/_helpers/dialogOnClickHandler.js

@@ -46,7 +46,7 @@ export const dialogOnClickHandler = (
   }
 
   // If the close button is not disabled, close the modal.
-  if (closeButtonRef?.current !== undefined && closeButtonRef?.current?.disabled === false) {
+  if (closeButtonRef?.current != null && closeButtonRef?.current?.disabled === false) {
     closeButtonRef.current.click();
   }
 };

src/components/Modal/_helpers/dialogOnCloseHandler.js

@@ -17,7 +17,7 @@ export const dialogOnCloseHandler = (e, closeButtonRef, onCloseHandler = undefin
   e.preventDefault();
 
   // If the close button is not disabled, close the modal.
-  if (closeButtonRef?.current !== undefined && closeButtonRef?.current?.disabled === false) {
+  if (closeButtonRef?.current != null && closeButtonRef?.current?.disabled === false) {
     closeButtonRef.current.click();
   }
 

src/components/Modal/_helpers/dialogOnKeyDownHandler.js

@@ -35,7 +35,7 @@ export const dialogOnKeyDownHandler = (
     // of close event in happy-dom library. When this is fixed, the `else` statement can be removed
     // as the `closeButtonRef.current.click()` will be handled by `dialogOnCancelHandler.js`.
     if (
-      closeButtonRef?.current === undefined
+      closeButtonRef?.current == null
       || closeButtonRef?.current?.disabled === true
       || !allowCloseOnEscapeKey
     ) {
@@ -52,7 +52,7 @@ export const dialogOnKeyDownHandler = (
   // 4. The focused element is an input or select (text area is omitted as Enter key is used for new line)
   if (
     e.key === 'Enter'
-    && primaryButtonRef?.current !== undefined
+    && primaryButtonRef?.current != null
     && primaryButtonRef?.current?.disabled === false
     && allowPrimaryActionOnEnterKey
     && ['INPUT', 'SELECT'].includes(e.target.nodeName)

src/components/Modal/_hooks/useModalFocus.js

@@ -16,7 +16,7 @@ export const useModalFocus = (
 
       const dialogElement = dialogRef.current;
 
-      if (dialogElement === undefined) {
+      if (dialogElement == null) {
         return () => {};
       }
 
@@ -41,7 +41,7 @@ export const useModalFocus = (
         return () => {};
       }
 
-      if (primaryButtonRef?.current !== undefined && primaryButtonRef?.current?.disabled === false) {
+      if (primaryButtonRef?.current != null && primaryButtonRef?.current?.disabled === false) {
         primaryButtonRef.current.focus();
         return () => {};
       }