Skip to content

Landing#24

Open
grynkiv0x001 wants to merge 17 commits intomate-academy:masterfrom
grynkiv0x001:develop
Open

Landing#24
grynkiv0x001 wants to merge 17 commits intomate-academy:masterfrom
grynkiv0x001:develop

Conversation

@grynkiv0x001
Copy link
Copy Markdown

Copy link
Copy Markdown

@ArtemTopolskyi ArtemTopolskyi 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
Эти ссылки никуда не ведет

В остальном гуд, делай адаптив

src/index.html Outdated
</ul>
</div>
<div class="nav__contacts">
<a href="#" class="nav__contacts-phone">+00 123 456 789</a>
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Suggested change
<a href="#" class="nav__contacts-phone">+00 123 456 789</a>
<a hhref="tel:+00123456789" class="nav__contacts-phone">+00 123 456 789</a>

Copy link
Copy Markdown

@ArtemTopolskyi ArtemTopolskyi left a comment

Choose a reason for hiding this comment

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

На 650px футер слипается, ведет себя странно на меньшей ширине
image

На 550px нет отступов от краев экрана
image

сделай меньше шрифт, это не окей
image

Copy link
Copy Markdown

@ArtemTopolskyi ArtemTopolskyi 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

Добавь отступы справа и слева для этого текста
image

@grynkiv0x001
Copy link
Copy Markdown
Author

image
Добавь ограничение по ширине для футера

Добавь отступы сверху, кнопки прилипли к верхнему краю экрана
image

Добавь отступы справа и слева для этого текста
image

Я так понимаю ограничивать ширину надо для каждого блока, сейчас это сделаю

Copy link
Copy Markdown

@ArtemTopolskyi ArtemTopolskyi left a comment

Choose a reason for hiding this comment

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

Сделай лого кликабельным
image

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.

2 participants