Отрисуй меня полностью #6
Conversation
|
♻️ Я собрал ваш пулреквест. Посмотреть можно здесь. |
js/photos.js
Outdated
There was a problem hiding this comment.
Лучше всего стараться всегда писать весь код внутри функций. Даже если в файле больше нет функций. Так проще масштабировать код, экспортировать, изолировать переменные. Поэтому здесь тоже лучше создать как минимум одну функцию для создания мениатюр. Может быть ты захочешь создать и больше, чем одну, почему нет)
js/main.js
Outdated
| import {createArrayOfPhotos} from './data.js'; | ||
|
|
||
| createArrayOfPhotos(); | ||
| import './photos.js'; |
There was a problem hiding this comment.
Когда создашь функцию в модуле, стоит импортировать ее и вызвать в главном модуле)
js/photos.js
Outdated
| photos.forEach((photo) => { | ||
| const picture = pictureTemplate.cloneNode(true); | ||
|
|
||
| picture.querySelector(".picture__img").src = photo.url; |
There was a problem hiding this comment.
У фото еще alt нужно заполнить по заданию
There was a problem hiding this comment.
как-то пропустил, исправил)
js/photos.js
Outdated
|
|
||
| picture.querySelector(".picture__img").src = photo.url; | ||
| picture.querySelector(".picture__comments").textContent = photo.comments.length; | ||
| picture.querySelector(".picture__likes").textContent = photo.likes; |
There was a problem hiding this comment.
Можно в начале развернуть массив, чтобы проще было обращаться ко всем свойствам. Пример const [likes] = photo
There was a problem hiding this comment.
Заметил мою ошибку с массивом) Конечно же у нас объект, сделал все правильно
👍🏻
|
♻️ Я собрал ваш пулреквест. Посмотреть можно здесь. |
js/main.js
Outdated
| import {createArrayOfPhotos} from './data.js'; | ||
|
|
||
| createArrayOfPhotos(); | ||
| import {generatePictures} from './photos.js'; |
There was a problem hiding this comment.
Давай выберем или photo, или pictures, а то несогласованность какая-то)
|
♻️ Я собрал ваш пулреквест. Посмотреть можно здесь. |
keksobot
merged commit b231d5f
into
htmlacademy-univer-javascript-1:master
🎓 Отрисуй меня полностью
💥 https://htmlacademy-univer-javascript-1.github.io/2579201-kekstagram-5/6/