7 Самсонов Иван #38
7 Самсонов Иван #38
Conversation
| ); | ||
| }; | ||
|
|
||
| ReactDom.render(<Form />, document.getElementById('userForm')); |
There was a problem hiding this comment.
Кажется, ты забыл закоммитить изменения в index.html, без него ничего не работает)
| return ( | ||
| <Modal onClose={close}> | ||
| <Modal.Header>Пользователь сохранен</Modal.Header> | ||
| <Modal.Body>{listItems.length !== 0 && saved && <div>Измененные данные: {listItems}</div>}</Modal.Body> |
There was a problem hiding this comment.
Когда рендеришь массив, у каждого из детей должно быть свойство key. Сейчас его нет, и в консоли из-за этого ошибки при открытии модалки
| setData({ ...data, current: { ...data.current, [field]: value } }); | ||
| } | ||
|
|
||
| function renderModal() { |
There was a problem hiding this comment.
Если у тебя в компоненте появляется функция, которая называется render..., это хороший звоночек, что это должен быть отдельный компонент :)
| <p> | ||
| Имя: было {data.saved.name} стало {data.current.name} | ||
| </p> |
There was a problem hiding this comment.
Это в целом тоже может быть мини-компонентом)
| function close() { | ||
| setOpened(false); | ||
| setData({ ...data, saved: data.current }); | ||
| setSaved(true); | ||
| } |
There was a problem hiding this comment.
Тут внутри столько всего происходит, что это скорее не просто close, а saveAndClose какой-нибудь) Ну или абстрактный closeHandler. Главное, чтобы название не вводило в заблуждение, что там происходит какое-то одно простое действие
О, и кстати в принципе, почему сохранение данных происходит по закрытию модалки, а не по кнопке "Сохранить"?)
| onClick={() => { | ||
| close(); | ||
| }} |
There was a problem hiding this comment.
А можно просто onClick={close}
| }); | ||
| const [saved, setSaved] = useState(false); | ||
| const [opened, setOpened] = useState(false); | ||
| const [panel, setPanel] = useState(false); |
No description provided.