Skip to content

Conversation

@IsaacKlark
Copy link

@vkryvytskyi
Copy link

vkryvytskyi commented Jan 15, 2020

image
Делай чтобы всегда было максимум 4 элемента в строке, ибо на больших экранах их аж 5

Comment on lines +21 to +23
const getDataFromStore = state => ({

});

Choose a reason for hiding this comment

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

Это заготовка на будущее?

@@ -0,0 +1,134 @@
import PropTypes from 'prop-types';

Choose a reason for hiding this comment

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

Посмотри видео которое вам давал Миша, там нет таких больших файлов, разбивай их на несколько компонентов

@vpolets
Copy link

vpolets commented Jan 20, 2020

image
футер роз'їхався

@vpolets
Copy link

vpolets commented Jan 20, 2020

Шрифти не ті
В тебе ось такі
image
А мають бути без зарубок
image

@vpolets
Copy link

vpolets commented Jan 20, 2020

image
Два останні елементи мали б бути зліва на початку останньої строки, не треба їх центрувати

@vpolets
Copy link

vpolets commented Jan 20, 2020

image
текст занадто близько до Sign in, зроби відступ

@vpolets
Copy link

vpolets commented Jan 20, 2020

image
Це теж інпут

@vpolets
Copy link

vpolets commented Jan 20, 2020

image
Це зазвичай роблять посиланняь на index.html тобто на home сторінку

@vpolets
Copy link

vpolets commented Jan 20, 2020

Ящкщо елементів для відображення немає футер не має прилипати до хедера а бути розміщений внизу

@vpolets
Copy link

vpolets commented Jan 20, 2020

image
Не по дизайну

@vpolets
Copy link

vpolets commented Jan 20, 2020

Не по дизайну
карти мають займати 100% ширини на малих екранах, глянь дизайн
image
Те саме і при 2 колонках

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