Skip to content

Conversation

@inixxes
Copy link

@inixxes inixxes commented Oct 26, 2021

DEMO LINK
Вигружаю, але буду далі правити, треба додати перемінні, порівняти деякі блоки.

Copy link

@varseniuk varseniuk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ось деякі зауваження, які вже можна помітити на даний момент:

На цій кнопці дизайн не по макету

Screenshot 2021-10-28 at 11 00 50

Тут слово cinemax майже не видно

Screenshot 2021-10-28 at 11 01 11

Має бути відступ над заголовком секції
Screenshot 2021-10-28 at 11 01 25

Текст у кнопці не відцентрований
Screenshot 2021-10-28 at 11 01 34

Під заголовком також має бути відступ
Screenshot 2021-10-28 at 11 01 51

Знизу в цих картках відступ не такий великий
Screenshot 2021-10-28 at 11 02 01

Вся секція з футером поки не виглядає, як на макеті
Screenshot 2021-10-28 at 11 02 16

На іконки потрібно додати cursor:pointer
Screenshot 2021-10-28 at 11 02 30

Потрібно додати заголовок

Screenshot 2021-10-28 at 11 05 42

Адаптив поки що відсутній

@inixxes
Copy link
Author

inixxes commented Oct 29, 2021

я, доречі, так і не зрозумів чому не було видно картинку з сінемаксом. декілька разів її перезавантажував, вийшло тільки джпг. щось, мабуть, з прозорістю. в цілому все попідтягував, додав ховери, тіні, стрілочку в верхноьму меню, адаптивність. тільки чи треба мені ще додати 640, бо в завданні не було, тому я 320, а потім одразу 1024 було з роздільної здатності.

@inixxes inixxes requested a review from varseniuk October 29, 2021 10:35
Copy link

@varseniuk varseniuk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Зверни увагу, що при адаптиві певні секції вилазять і збоку дуже багато місця

Screen.Recording.2021-11-01.at.13.52.18.mov

Зроби лінки соціальних мереж клікабельними
Screenshot 2021-11-01 at 13 52 46

Іконка бургер-меню виглядає не ок. Потрібно пофіксити
Screenshot 2021-11-01 at 13 53 03

Додай scroll-behavior: smooth на сторінку, щоб скрол до секцій по кліках відбувався плавно

@inixxes inixxes requested a review from varseniuk November 2, 2021 13:15
@inixxes
Copy link
Author

inixxes commented Nov 2, 2021

я просто робив по завданню, де вказані тільки ті розміри. але, як я розумію, треба щоб була адапвтність в тому плані, щоб не тільки було так, коли чітко розмір вказаний, наприклад 1024, а коли звужуєш/розширюеш вікно, і може бути 1127.

Copy link

@vkryvytskyi vkryvytskyi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image
Почему у тебя фон везде одного, серого цвета?
image
В дизайне секции вылелены разными цветами, и ни один из них не является тем, который на фоне у тебя

Screen.Recording.2021-11-04.at.13.43.55.mov

Кнопка странно обрезается справа при наведении

image

Почему эти иконки не svg?

image

Добавь какой-то отступ между кнопками

image

Тут текст залезает под карточку снизу

@inixxes inixxes requested a review from vkryvytskyi November 4, 2021 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants