search

MastersAcademy / frontend-course-2021

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

8-css-variables #227

Closed ConstantinRusenko closed 2 years ago

ConstantinRusenko commented 2 years ago

https://constantinrusenko.github.io/frontend-course-2021/homeworks/konstantin.rusenko_ConstantinRusenko/8-css-variables/

Chebutalio commented 2 years ago

image на темній темі немає такої білої частини

Chebutalio commented 2 years ago

image при зменшенні ширини сторінки - ця картинка скукожується

Chebutalio commented 2 years ago

image скукожений текст, блоки не рівні по висоті, кнопка також якась страшна і не рівна, відступ дуже маленький. На такому екрані треба б ці блоки розмістити вертикально і все

Chebutalio commented 2 years ago

image На заголовку розміром зі скрін телефону я б і закінчив ознайомлення з твоїм сайтом в реальному житті

Chebutalio commented 2 years ago

image Меню просто страшне. Зроби його нормального розміру і точно не прозорим.

Chebutalio commented 2 years ago

image Меню просто страшне. Зроби його нормального розміру і точно не прозорим.

також меню має закриватися по тапу за межами меню

Chebutalio commented 2 years ago

image О, так тут є медіаквері для 500, де це виглядає все ще не правильно. Центруй текст, кнопку зроби нормальною

Chebutalio commented 2 years ago

image При розмірі менше 500 уже можна всі картинки робити по одній і вертикально

Chebutalio commented 2 years ago

image вище текст по центру вирівняний, а тут чомусь він вирівняний по лівому краю

Chebutalio commented 2 years ago

image скукожені рєбята

Chebutalio commented 2 years ago

image скукожений домік... як читати оцей білий текст на білому фоні?

Chebutalio commented 2 years ago

image на 500 брейкпоінті відцентруй

Chebutalio commented 2 years ago

image 320 - хедер страшненький. Також на цьому брейкпоінті всі картинки скукожені

Chebutalio commented 2 years ago

image на 320 брейкпоінті можна лого зробити зверху, а під ним уже список лінок

ConstantinRusenko commented 2 years ago

And resolved other noted issues

Chebutalio commented 2 years ago

image на дизайні не такий градієнт. На широкому екрані картинки скукожені по висоті

Chebutalio commented 2 years ago

image зроби бургер по середині

Chebutalio commented 2 years ago

image скукожені рєбята

до 1000 брейкпоінта скукожуються, далі вже просто обрізаються. Можна було б просто зменшувати всю картинку. З будинком нижче та ж сама історія.

Chebutalio commented 2 years ago

image верхня части на не по центру

ConstantinRusenko commented 2 years ago

Поправил картинки и центрировал верхнюю часть и хедер. С градиентом поигрался, вроде бы похоже. Думал что в фигме нормальный градиент, но он не валидный.

Chebutalio commented 2 years ago

Поправил картинки и центрировал верхнюю часть и хедер. С градиентом поигрался, вроде бы похоже. Думал что в фигме нормальный градиент, но он не валидный.

з фігми можна брати шрифти, кольори і дивитися падінги/марджини. Стилі звідти брати - це погана ідея. А ще хороша практика - звірити пункти в задачі з тим, що реалізовано, щоб потім не повертали недороблену задачу.

Chebutalio commented 2 years ago

image 1000 брейкпоінт тут картинки звужуються по ширині. На наступному брейкпоінті вони звужуються по висоті. Проблема з оцими двома картинками. Їх доробляй і вмержимо.

ConstantinRusenko commented 2 years ago

image 1000 брейкпоінт тут картинки звужуються по ширині. На наступному брейкпоінті вони звужуються по висоті. Проблема з оцими двома картинками. Їх доробляй і вмержимо.

Добавил, теперь и по ширине и высоте