MastersAcademy / frontend-course-2021

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

6-adaptive #205

Closed ConstantinRusenko closed 2 years ago

ConstantinRusenko commented 2 years ago

https://constantinrusenko.github.io/frontend-course-2021/homeworks/konstantin.rusenko_ConstantinRusenko/6-adaptive/

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 У тебе тут взагалі якийсь хаос з відступами. Дивись як це на дизайні. Також відсутні ховер ефекти для лінок

Chebutalio commented 2 years ago

image знову відступи, текст має бути в два рядки, відступи між іконками-посиланнями на соцмережі мають бути меншими, відсутні ховер ефекти для лінок

Chebutalio commented 2 years ago

image Можна додати ще одну медіа квері для того, щоб до розміру в 768 пікселів сайт також виглядав нормально (ну або хоча б поправити вьорстку загалом, щоб вона стала респонсивною)

Chebutalio commented 2 years ago

image при 768 цей заголовок виглядає нєочєнь

Chebutalio commented 2 years ago

image 768 - футер виглядає дивно. Між логотипами також досить маленький відступ

Chebutalio commented 2 years ago

image 320 пікселів. Скажи, ти б такий сайт листав на телефоні?)

Chebutalio commented 2 years ago

image кривенько

ConstantinRusenko commented 2 years ago

image 768 - футер виглядає дивно. Між логотипами також досить маленький відступ

сделал получше

ConstantinRusenko commented 2 years ago

image 320 пікселів. Скажи, ти б такий сайт листав на телефоні?)

изменил. Просто иногда не очень понятно как сделать(не в этом месте) мобильный дизайн или таблет без примеров

ConstantinRusenko commented 2 years ago

image при 768 цей заголовок виглядає нєочєнь

сделал лучше

ConstantinRusenko commented 2 years ago

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

Есть

ConstantinRusenko commented 2 years ago

image Відсутній відступ між елементами

готов

ConstantinRusenko commented 2 years ago

image Ховер та ектів ефекти для кнопок відсутні. Текст в інпуті мав би мати якийсь мабуть темніший колір, бо його майже не видно.

ховеры есть, поинтер появляется на линках также. Просто не понятно по дизайну на какие цвета должно меняться

Chebutalio commented 2 years ago

В задачі описано, що на лінках і кнопках має бути ховер ефект (це стосується не лише курсора, а взагалі. Колір для ховер ефектів ти можеш обирати сам, який хочеш, головне. щоб це виглядало нормально.

Chebutalio commented 2 years ago

image

або чувачок наїжджає на текст, або текст наїжджає на чувачка... Щось тут не так точно

Chebutalio commented 2 years ago

image не пофікшено і в цілому брейкпоінт на 320 (який ніхто не використовує, я скинув тобі розміри) неюзабєльний.

Chebutalio commented 2 years ago

тег article використовується для вставки статті, блогу чи т.п. У тебе він використовується вскрізь. Що заважає використати просто дів, якщо немає підгодящого семантичного тегу?

https://developer.mozilla.org/ru/docs/Web/HTML/Element/article

Chebutalio commented 2 years ago

З приводу вставки svg була окрема лекція. В цій домашці треба використати те. що ви на ній пройшли. Всі коменти які не скриті - не виправлені + є кілька нових.

ConstantinRusenko commented 2 years ago

На лекции показывали несколько способов, один из них был img. Я пробовал и с use, но тогда html становиться очень большой(тысячи строк) а так же линтер много, очень много ругается на все эти строки html. А если в use вставить линку на файл, то image и использовать без сервера не получиться из-за CORS

Chebutalio commented 2 years ago

image На макеті рівненько

Chebutalio commented 2 years ago

image

не пофікшено. Ти юзаєш одночасно верхні та нижні марджини. Так робити не варто: https://developer.mozilla.org/ru/docs/Web/CSS/CSS_Box_Model/Mastering_margin_collapsing

Chebutalio commented 2 years ago

image ок, 375... Лого не по центру, текст кнопки переводиться на дві строчки, контент здвинутий, немає відступу від кнопки для кнопки зміни мов. Головний текст-заголовок дуже великий. Плесхолдер в інпуті не влізає і виглядає зломаним

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

На лекции показывали несколько способов, один из них был img. Я пробовал и с use, но тогда html становиться очень большой(тысячи строк) а так же линтер много, очень много ругается на все эти строки html. А если в use вставить линку на файл, то image и использовать без сервера не получиться из-за CORS

Юзай спрайт і передавай айдішку в юз. Просто треба розібратися. Вставляти свг в html - це взагалі не варіант, бо такий темплейт нереально буде підтримувати.

Chebutalio commented 2 years ago

В задачі описано, що на лінках і кнопках має бути ховер ефект (це стосується не лише курсора, а взагалі. Колір для ховер ефектів ти можеш обирати сам, який хочеш, головне. щоб це виглядало нормально.

не пофікшено