MastersAcademy / frontend-course-2021

Репозиторій з курсу «Advanced Front-end» проекту Masters Academy
MIT License
5 stars 29 forks source link

3-sync-first-commit #110

Closed MaksymSelenkov closed 3 years ago

MaksymSelenkov commented 3 years ago

sortInitial and filterData dont work. I am finding a resolve, why it doesnt work?

TArch64 commented 3 years ago

Роби на фікс кожного коментаря окремий комміт. Комміти мають бути невиличками частинами змін твого проекту і уникай коммітів з меседжами типу 3-async-first-fixed. По меседжу комміта має бути зрозуміло які ти робив зміни не відкриваючи його

TArch64 commented 3 years ago

Додай лінк на гітхаб пейджес

https://docs.github.com/en/pages/getting-started-with-github-pages/configuring-a-publishing-source-for-your-github-pages-site#choosing-a-publishing-source

MaksymSelenkov commented 3 years ago

https://maksymselenkov.github.io/frontend-course-2021/homeworks/maksym.selenkov_MaksymSelenkov/3-async/index.html

TArch64 commented 3 years ago

Не відповідає дизайну. Потрібно зробити так як там намальовано

image

TArch64 commented 3 years ago

Лодер занадто великий та піксельний. Зменши його та знайди імедж з більшим резолюшином image

TArch64 commented 3 years ago

Використовуй, коли можна, браузерні дропдауни. Кастомний це окрема розмова як його робити щоб він був адекватним і ним не больно було користовуватися. У браузера є тег <select> то його і використовуй

image

TArch64 commented 3 years ago

@MaksymSelenkov ping-ping