Remove role prop as we're not using it atm

[MAILPOET-2777]
This commit is contained in:
Jan Jakeš
2020-06-01 12:26:01 +02:00
committed by Veljko V
parent f58e6f1237
commit 75efdca29f
2 changed files with 1 additions and 8 deletions

View File

@@ -3,7 +3,6 @@ import classnames from 'classnames';
type Props = {
fullScreen?: boolean,
role?: string,
className?: string,
children: React.ReactNode,
};
@@ -11,7 +10,6 @@ type Props = {
function ModalFrame({
children,
className,
role,
fullScreen,
}: Props) {
return (
@@ -21,7 +19,7 @@ function ModalFrame({
{ 'mailpoet-modal-full-screen': fullScreen },
className
)}
role={role}
role="dialog"
tabIndex={-1}
>
{children}
@@ -30,7 +28,6 @@ function ModalFrame({
}
ModalFrame.defaultProps = {
role: 'dialog',
fullScreen: false,
className: '',
};

View File

@@ -15,7 +15,6 @@ type Props = {
fullScreen?: boolean,
shouldCloseOnEsc?: boolean,
shouldCloseOnClickOutside?: boolean,
role?: string,
children: React.ReactNode,
};
@@ -26,7 +25,6 @@ function Modal({
isDismissible,
shouldCloseOnEsc,
shouldCloseOnClickOutside,
role,
contentClassName,
overlayClassName,
fullScreen,
@@ -39,7 +37,6 @@ function Modal({
className={overlayClassName}
>
<ModalFrame
role={role}
className={contentClassName}
fullScreen={fullScreen}
>
@@ -63,7 +60,6 @@ function Modal({
Modal.defaultProps = {
onRequestClose: () => {},
role: 'dialog',
title: null,
shouldCloseOnEsc: true,
shouldCloseOnClickOutside: true,