Непроставлен курсор на бургере (или курсор-текст это фича?)
4. Если бургер-меню на весь экран - зачем перенос слов (ширина контейнера менюшек
5. бордер инпута поиска - обрезается
6. неработает кнопка look (или я не понял что она должна делать)
7. Медленная и тормозная анимация скейла на инпутах (усугубляет, что она двигает за собой инпуты снизу)
8. Неотцентрованы надписи\кнопка look на зеленых инпутах
9. Ссылки в бургерменю частично не рабочие (articles -> 404)
10. В html не везде соблюдены корректно отступы у вложенных тегов
11. Спорный момент, но имхо если таблица в 4 колонки, то и ячейки (td) должно быть 4 (не уверен в этом моменте). И наверное лучше было бы сверстать этот блок на гридах\флекса, ну дело вкуса.
12. scss транспилится в свою же папку (исходники и стили в одной куче)
13. Не консистентное форматирование ксс (пропуск строк не везде, пробел перед скобкой тоже)
14. Плохо и не читаемо (pantigans.js): event.target.onclick = document.querySelector(".articles").querySelectorAll(".items").forEach((items)=> items.remove());
15. Load() - с большой буквы не принято называть функции (это классы)
16. Везде let, где по хорошему нужна const
17. find.js (10 строка) - невозможно прочитать. И вообще лучше не писать в стиле if (ооочень длинное условие) something else another, и всё в одну строку
18. Лучше использовать стрелочную функцию в качестве коллбеков , так читаемее
19. articles.js (6 строка) - невозможно прочитать
Сорри, без негатива))
Большинство из этого списка чинится линтером\преттиером)
жесть
Обсуждают сегодня