Отрисуй меня полностью #10
Conversation
|
♻️ Я собрал ваш пулреквест. Посмотреть можно здесь. |
|
♻️ Я собрал ваш пулреквест. Посмотреть можно здесь. |
|
♻️ Я собрал ваш пулреквест. Посмотреть можно здесь. |
|
♻️ Я собрал ваш пулреквест. Посмотреть можно здесь. |
| }); | ||
|
|
||
| container.appendChild(fragment); | ||
| export{similarFinalMas}; |
There was a problem hiding this comment.
формиатирование кода, не оставляй код без пробелов внутри фигурных скобок
все для читаемости
| @@ -4,7 +4,6 @@ const getRandomInteger = (a, b) => { | |||
| const result = Math.random() * (upper - lower + 1) + lower; | |||
| return Math.floor(result); | |||
| }; | |||
|
|
|||
There was a problem hiding this comment.
достаточно чтоб файл был не rand.js а utils.js
что будет использоваться для повторяющихся функций
| @@ -0,0 +1,20 @@ | |||
| import {getRandomArrayElement, getRandomInteger} from './rand.js'; | |||
| import {description, messageCommentator, nameCommentator, count} from './data.js'; | |||
|
|
|||
There was a problem hiding this comment.
что значит mas.js? не понимаю из названия файла
| likes: getRandomInteger(1, 200), | ||
| comments: Array.from({ length: getRandomInteger(1, 30) }, (__, indexx) => createComment(indexx)), | ||
| }); | ||
| const finalMas = () => Array.from({length:count}, (__, index) => createUsers(index)); |
There was a problem hiding this comment.
между функциями - отступ
| container.appendChild(fragment); | ||
| export{similarFinalMas}; | ||
|
|
||
| /* <template id="picture"> |
There was a problem hiding this comment.
комменты из кода стоит удалять, особенно комменты не используемого кода
|
|
||
| copy.querySelector('.picture__img').src = url; | ||
| copy.querySelector('.picture__img').alt = description; | ||
| copy.querySelector('.picture__comments').textContent = comments.length;// |
There was a problem hiding this comment.
// и на 16 строке тоже, для чего осталось?
| // eslint-disable-next-line no-console | ||
| console.log(finalMas); | ||
| import {similarFinalMas} from './createMiniature.js'; | ||
| similarFinalMas(); |
There was a problem hiding this comment.
в идеале хочется чтоб в main.js было формирование данных и после вызов функции для отрисовки полученных данных
There was a problem hiding this comment.
не так чтоб файл js/createMiniature.js сам по себе генерил их
🎓 Отрисуй меня полностью
💥 https://htmlacademy-javascript.github.io/2554297-kekstagram-2/10/