Skip to content

Develop#70

Open
7dimnik wants to merge 10 commits into
mate-academy:masterfrom
7dimnik:develop
Open

Develop#70
7dimnik wants to merge 10 commits into
mate-academy:masterfrom
7dimnik:develop

Conversation

@7dimnik
Copy link
Copy Markdown

@7dimnik 7dimnik commented Jan 16, 2022

Comment thread README.md
12. Create a Pull Request (PR) from your branch `develop` to branch `master` of original repo.
13. Replace `<your_account>` with your Github username in the
[DEMO LINK](https://<your_account>.github.io/Thrive_talk/).
[DEMO LINK](https://7dimnik.github.io/Thrive_talk/).
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

image

Тут лого кнопка и бургер не выровнены по вертикали

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

image

На мобильных лучше делать фон однотонным

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

image

Слишком большие отступы между блоками, и возможно текст лучше через justify располагать, потому что выглядит вместо с заголовком слева не очень ровно

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

image

Меню немного странно открывается, сливается с кнопкой

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Screen.Recording.2022-01-21.at.18.49.24.mov

И еще scroll-behavior: smooth не помешал бы, для плавного перехода по ссылкам

@7dimnik
Copy link
Copy Markdown
Author

7dimnik commented Jan 22, 2022

#70 (comment)
Так було в макеті, мені також це не подобалось...

@7dimnik 7dimnik requested a review from vkryvytskyi January 22, 2022 11:14
@7dimnik
Copy link
Copy Markdown
Author

7dimnik commented Jan 22, 2022

#70 (comment)
прописав, але не працює
де я роблю помилку?
image

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.

попробуй через тег html

html {
scroll-behavior: smooth;
}

у тебя на телефонной версии разные отступы в разных блоках

Screenshot 2022-01-26 at 11 01 55

Screenshot 2022-01-26 at 11 02 02

@7dimnik
Copy link
Copy Markdown
Author

7dimnik commented Jan 27, 2022

#70 (review)
Так є в дизайні. Розмір блоку "Nutritional Facts..." дозволяє використовувати його без змін на всіх екранах, тому вирфшмв його не чипати.

@7dimnik
Copy link
Copy Markdown
Author

7dimnik commented Jan 29, 2022

#70 (review) fixed

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