Skip to content

Commit 13c1030

Browse files
authored
Merge pull request #5 from webdevia/homework-5
Homework 5 fix
2 parents f97a132 + d19616d commit 13c1030

File tree

2 files changed

+3
-8
lines changed

2 files changed

+3
-8
lines changed

src/shared/modal-form/ModalForm.module.scss

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,3 @@
3333
}
3434
}
3535

36-
.modal-hide {
37-
display: none;
38-
}
Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
11
import React, { ReactNode, MouseEvent } from 'react';
2-
import cn from 'clsx';
32
import s from './ModalForm.module.scss';
43

54
type ModalFormProps = {
@@ -8,9 +7,9 @@ type ModalFormProps = {
87
onClose?: (event: MouseEvent<HTMLButtonElement>) => void;
98
};
109

11-
export const ModalForm = ({ visible = true, onClose, children }: ModalFormProps) => {
12-
return (
13-
<div id="myModal" className={cn(s.modal, { [s['modal-hide']]: !visible })}>
10+
export const ModalForm = ({ visible = true, onClose, children }: ModalFormProps) =>
11+
!!visible && (
12+
<div className={s.modal}>
1413
<div className={s['modal-content']}>
1514
<span className={s.close} onClick={onClose}>
1615
&times;
@@ -19,4 +18,3 @@ export const ModalForm = ({ visible = true, onClose, children }: ModalFormProps)
1918
</div>
2019
</div>
2120
);
22-
};

0 commit comments

Comments
 (0)