Skip to content

feat: completed task thrive_talk#56

Open
VolodymyrKlepykov wants to merge 8 commits into
mate-academy:masterfrom
VolodymyrKlepykov:talk
Open

feat: completed task thrive_talk#56
VolodymyrKlepykov wants to merge 8 commits into
mate-academy:masterfrom
VolodymyrKlepykov:talk

Conversation

@VolodymyrKlepykov
Copy link
Copy Markdown

Copy link
Copy Markdown

@danylo-onishchuk danylo-onishchuk left a comment

Choose a reason for hiding this comment

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

Сделай часы работы в одну строчку

Сделай кликабельным номер телефона и имейл

Добавь на разноцветные блоки анимашку, что было повеселее

Навигация не работает

для мобилки лучше сделать бургер меню

Copy link
Copy Markdown

@danylo-onishchuk danylo-onishchuk left a comment

Choose a reason for hiding this comment

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

для телефона лучше что то типо этого
https://snipp.ru/html-css/site-phone

было бы круто, чтоб по клику на адрес открывались гугл карты с этим адресом. есть готовое решение, почитай про него.

бургер меню лучше сделать на весь экран и с беграундом

тут поехало

Screenshot 2021-06-09 at 10 46 17

придерживайся одних размеров на всей странице

Screenshot 2021-06-09 at 10 46 17

Copy link
Copy Markdown

@danylo-onishchuk danylo-onishchuk 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-06-09 at 17 59 53

@VolodymyrKlepykov
Copy link
Copy Markdown
Author

давай все же бургер сделаем на весь экран. так будет удобнее для использования на телефоне.

и давай тут фон растянем на весь контейнер

Screenshot 2021-06-09 at 17 59 53

Преход по адресу и окрытие карт, так нужно было реализовать ?

Copy link
Copy Markdown

@danylo-onishchuk danylo-onishchuk left a comment

Choose a reason for hiding this comment

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

адрес отлично, бургер отлично

фон хидера ты не растянул

Comment thread src/styles/blocks/_info.scss Outdated
}

&__link {
// margin-top: 63px;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

убери комментарии

Comment thread src/styles/main.scss Outdated
@import "./blocks/info.scss";
@import "./blocks/header.scss";

@font-face {
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

это лучше вынести в утилиты

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