Открывается и закрывается #6
Conversation
js/bigPicture.js
Outdated
| }; | ||
|
|
||
| export const closeBigPicture = () => { | ||
| document.addEventListener('keydown', (event) => { |
There was a problem hiding this comment.
Обработчики событий нужно удалять с DOM-элементов, иначе они будут там копиться до бесконечности и забивать память (я говорил об этом на занятии 14го марта, можешь посмотреть запись)
js/bigPicture.js
Outdated
| @@ -0,0 +1,67 @@ | |||
| export const showBigPicture = (post) => { | |||
There was a problem hiding this comment.
Имеет смысл разбить эту функцию на несколько:
- Функция для удаления классов скрывающих модалку
- Функция для рендера правильной картинки/описания
- Функция для рендера комментариев
js/bigPicture.js
Outdated
| bigPicture.classList.add('hidden'); | ||
| }; | ||
|
|
||
| export const closeBigPicture = () => { |
There was a problem hiding this comment.
Комментарии нужно стирать после того как мы открыли новую картинку. Иначе они будут копиться до бесконечности (попробуй открыть два раза модалку с 3-4 комментами и поймешь о чем я)
| @@ -0,0 +1,84 @@ | |||
| const openModal = () => { | |||
| const bigPicture = document.querySelector('.big-picture'); | |||
There was a problem hiding this comment.
Давай в эту функцию добавим обработчики событий для закрытия модалки
js/bigPicture.js
Outdated
| const closeButton = document.getElementById('picture-cancel'); | ||
| const clickHandler = () => { | ||
| close(); | ||
| closeButton.removeEventListener('click', clickHandler); |
There was a problem hiding this comment.
Удаление обработчиков событий должно находиться в функции ответственной за закрытие модалки. Вообще, логика должна быть примерно следующей:
- Открыли модалку - повесили обработчики событий
- Закрыли модалку - удалили обработчики событий
There was a problem hiding this comment.
Сейчас у тебя перестают работать обработчики событий если несколько раз закрыть и открыть модалку
🎓 Открывается и закрывается