feat(Modal): fix CR comments

Author: YossiSaadi

packages/core/src/components/ModalNew/Modal/Modal.tsx

@@ -27,8 +27,8 @@ const Modal = forwardRef(
     }: ModalProps,
     ref: React.ForwardedRef<HTMLDivElement>
   ) => {
-    const [titleId, setTitleId] = useState<string>("");
-    const [descriptionId, setDescriptionId] = useState<string>("");
+    const [titleId, setTitleId] = useState<string>();
+    const [descriptionId, setDescriptionId] = useState<string>();
 
     const setTitleIdCallback = useCallback((id: string) => setTitleId(id), []);
     const setDescriptionIdCallback = useCallback((id: string) => setDescriptionId(id), []);
@@ -83,8 +83,8 @@ const Modal = forwardRef(
           data-testid={dataTestId || getTestId(ComponentDefaultTestId.MODAL_NEXT, id)}
           role="dialog"
           aria-modal
-          aria-labelledby={titleId || undefined}
-          aria-describedby={descriptionId || undefined}
+          aria-labelledby={titleId}
+          aria-describedby={descriptionId}
         >
           <ModalTopActions
             renderAction={renderHeaderAction}

packages/core/src/components/ModalNew/Modal/Modal.types.tsx

@@ -14,7 +14,7 @@ export interface ModalProps extends VibeComponentProps {
   size?: ModalSize;
   closeButtonTheme?: ModalTopActionsProps["color"];
   closeButtonAriaLabel?: ModalTopActionsProps["closeButtonAriaLabel"];
-  onClose?: (e: ModalCloseEvent) => void;
+  onClose?: (event: ModalCloseEvent) => void;
   renderHeaderAction?: ModalTopActionsProps["renderAction"];
   children: React.ReactNode;
 }

packages/core/src/components/ModalNew/ModalTopActions/ModalTopActions.types.ts

@@ -19,5 +19,5 @@ export interface ModalTopActionsProps {
     | ((color?: ModalTopActionsButtonColor) => React.ReactElement<typeof MenuButton | typeof IconButton>);
   color?: ModalTopActionsColor;
   closeButtonAriaLabel?: string;
-  onClose?: (e: React.MouseEvent<HTMLButtonElement>) => void;
+  onClose?: (event: React.MouseEvent<HTMLButtonElement>) => void;
 }