Feature/header by serhii-mykalo · Pull Request #10 · FrontKid/Sladoled

serhii-mykalo · 2022-12-24T12:46:42Z

Добавив навігацію хедера.
Також добавив до спрайту хреста, так як перед тим десь загубився.
Також готове мобільне меню.

FrontKid · 2022-12-24T13:05:41Z

partials/header.html

@@ -0,0 +1,34 @@
+<div class="container header-flex__container">


общий container не должен бытв флекс контейнером

Добавити ще один div для флексу, чи container накласти на сам header?

Дивна поведінка в хромі, при ширині 768 мобільне меню частково видно

У хромі взагалі не відповідає дійсності без f12, в Edge наче все працює.

Виправив!

FrontKid · 2022-12-24T13:06:42Z

src/sass/2-components/_header.scss

+.header-flex__container {
+   display: flex;
+   justify-content: baseline;
+   align-items: flex-end;


Должно идти по центру и без марджинов, у флексов полно свойств для выравнивания элементов как угодно

Виправив!

FrontKid · 2022-12-24T13:06:57Z

src/sass/2-components/_header.scss

+
+   background-color: $dark-green;
+
+   @media screen and (min-width: 768px) {


Все медиа внизу файла

Виправив!

FrontKid · 2022-12-24T13:09:53Z

src/sass/2-components/_header.scss

+   font-style: normal;
+   font-weight: 600;
+   font-size: 14px;
+   line-height: calc(17 / 14);


17/14 то еств 1.2, зачем лишнии мат операции?

Виправив!

FrontKid · 2022-12-24T13:10:08Z

src/sass/2-components/_header.scss

+.button__buy {
+   padding: 12px 35px;
+   font-family: inherit;
+   font-style: normal;


Не дублируй свойства

font-family: inherit; це для кнопок, аби наслідували шрифт
font-style: normal; у загальних не було задано тому рішив написати, чи не потрібно?

Виправив!

FrontKid · 2022-12-24T13:10:25Z

src/sass/2-components/_header.scss

+
+.button__buy {
+   padding: 12px 35px;
+   font-family: inherit;


Это лишнее

Виправив!

FrontKid · 2022-12-24T13:17:09Z

src/sass/2-components/_header.scss

+   font-size: 14px;
+   line-height: calc(17 / 14);
+   display: flex;
+   justify-content: baseline;


Для чего тут это свойство

Видалив display: flex; та justify-content: baseline;

Також ось це line-height: calc(17 / 14);

Виправив!

FrontKid · 2022-12-24T13:18:57Z

src/sass/2-components/_header.scss

+   }
+}
+
+.buy-arrow-in-button {


иконка через псевдоклас должна быть

Випрвив!

Виправив!

FrontKid · 2022-12-24T13:20:26Z

src/sass/2-components/_menu-container.scss

+   font-style: normal;
+   font-weight: 700;
+   font-size: 32px;
+   line-height: calc(39 / 32);


Тоже самое

Виправив!

src/sass/2-components/_header.scss

illia-dobro · 2022-12-24T14:37:23Z

partials/header.html

@@ -0,0 +1,34 @@
+<div class="container header-flex__container">


Дивна поведінка в хромі, при ширині 768 мобільне меню частково видно

serhii-mykalo added 3 commits December 24, 2022 00:51

Add header, and menu

c2db377

add header

2aed2bb

fix-bag header

0423d71

serhii-mykalo requested review from a user, FrontKid and illia-dobro December 24, 2022 12:46

FrontKid requested changes Dec 24, 2022

View reviewed changes

illia-dobro reviewed Dec 24, 2022

View reviewed changes

serhii-mykalo added 2 commits December 24, 2022 21:25

error correction

6aa0428

add menu display:none

37efa73

FrontKid merged commit a3954b1 into main Dec 26, 2022

FrontKid deleted the feature/header branch December 26, 2022 08:57

		@@ -0,0 +1,34 @@
		<div class="container header-flex__container">


		background-color: $dark-green;

		@media screen and (min-width: 768px) {

Conversation

serhii-mykalo commented Dec 24, 2022

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants