Открывается и закрывается #12
Conversation
|
♻️ Я собрал ваш пулреквест. Посмотреть можно здесь. |
…менованы. в create-miniatuere добавли айди. в main добавлена констанста. в mas создаются новые элементы для комментариев. в full-picture основной когд, где прописана логика открытия и закрытия полноэкранного изображения.
|
♻️ Я собрал ваш пулреквест. Посмотреть можно здесь. |
js/createMiniature.js
Outdated
| @@ -3,32 +3,20 @@ import { finalMas } from './mas.js'; | |||
| const template = document.querySelector('#picture').content.querySelector('.picture'); | |||
| const container = document.querySelector('.pictures'); | |||
|
|
|||
| const similarFinalMas = finalMas(); | |||
|
|
|||
| const fragment = document.createDocumentFragment(); | |||
|
|
|||
There was a problem hiding this comment.
const renderThumbs = (arr) => {
js/createMiniature.js
Outdated
| /* <section class="pictures container"> | ||
| <h2 class="pictures__title visually-hidden">Фотографии других пользователей</h2> */ | ||
|
|
||
| export{finalMas}; |
js/main.js
Outdated
| similarFinalMas(); | ||
| import {finalMas} from './createMiniature.js'; | ||
| finalMas(); | ||
| import './fullPicture.js'; |
There was a problem hiding this comment.
const data = finalMas();
renderThumbs(data);
| const template = document.querySelector('#picture').content.querySelector('.picture'); | ||
| const container = document.querySelector('.pictures'); | ||
|
|
||
| const fragment = document.createDocumentFragment(); |
There was a problem hiding this comment.
фрагмент создай внутри функции
| }); | ||
| }; | ||
|
|
||
| container.appendChild(fragment); |
There was a problem hiding this comment.
это также внутри функции рендера
|
♻️ Я собрал ваш пулреквест. Посмотреть можно здесь. |
|
♻️ Я собрал ваш пулреквест. Посмотреть можно здесь. |
| document.addEventListener('keydown', onDocumentKeydown); | ||
| } | ||
|
|
||
| container.addEventListener('click', (evt) => { |
There was a problem hiding this comment.
слушатели раньше чем начались функции
| document.addEventListener('keydown', onDocumentKeydown); | ||
| } | ||
|
|
||
| container.addEventListener('click', (evt) => { |
There was a problem hiding this comment.
хочется чтоб действите было функцией, чтоб было точно понятно что тут происходит
а то у нас тут уже отрисоываются данные, а потом внезапно функция openPictureModal в которой только слушатели
получается что на container (но из названия не понятно что за контейнер) мы нажимаем чтоб получить элемень и его отобразить, что уже вроде действите "открыть"
| import { finalMas } from './mas.js'; | ||
|
|
||
| const dataThumbs = finalMas();renderThumbs(dataThumbs); | ||
| export {dataThumbs}; |
There was a problem hiding this comment.
а что вообще экспортируем мейн? должен ничего
мейн как основной узел, только при необходимости получает импорты и делает какие-то действия
There was a problem hiding this comment.
если ты хочешь передать полученый dataThumbs то экспортируй как функцию из рендера большого изображения, которое примит в себя как аргумент список тумб
| import {renderThumbs} from './create-miniature.js'; | ||
| import { finalMas } from './mas.js'; | ||
|
|
||
| const dataThumbs = finalMas();renderThumbs(dataThumbs); |
There was a problem hiding this comment.
странно что renderThumbs(dataThumbs); не на новой строке
🎓 Открывается и закрывается
💥 https://htmlacademy-javascript.github.io/2554297-kekstagram-2/12/