karina-doskuch / grid-project

0 stars 0 forks source link

Работа над ошибками #1

Open Alecto opened 4 years ago

Alecto commented 4 years ago

1) неправильное положение меню image

2) неправильный цвет разделителя. неправильный размер текстового блока image

3) некорректные отступы image

4) неправильное оформление элементов формы image

5) некорректное оформление и расположение текстовых блоков image

6) именование классов не соответствует БЭМ и принято только в PHP-переменных image

Alecto commented 4 years ago

7) здесь лучше элемент. а второе выделение - вообще не понял, что это, но 6 - не исправлено. проверьте подобные классы во всех документе image

здесь зачем блоки? это разве независимые элементы? нет image

8) не соответствует макету image

9) горизонтальный скрол. нет адаптации image

10) размер шрифта не соответствует макету для всех разрешений image

11) отступы не соответствуют image

12) отступы не соответствуют image

Alecto commented 4 years ago

13) очень много ошибок валидации image

Alecto commented 4 years ago

14) неправильные отступы для различных разрешений image

15) разделитель является элементом оформления, а не структурной частью страницы, вы считаете хорошей идеей делать для него разметку? image

16) требования к высоте не выполнены image image

17) не соответствует макету image

18) проблемы с внешним видом image image

karina-doskuch commented 4 years ago

Здравствуйте! Спасибо за замечания. Провела работу над ошибками. Единственное с 18 пунктом у меня не получилось добиться такого результата как у вас, у меня отступы меньше (возможно необходим специфический размер экрана), но тоже были большие отступы, я поправила.

репозиторий: https://github.com/karina-doskuch/grid-project github pages: https://karina-doskuch.github.io/grid-project/

пт, 16 окт. 2020 г. в 20:05, Andrey notifications@github.com:

1.

неправильные отступы для различных разрешений [image: image] https://user-images.githubusercontent.com/8464662/96287007-f0ff6c80-0fe9-11eb-9cc0-286f01b2956d.png

1.

разделитель является элементом оформления, а не структурной частью страницы, вы считаете хорошей идеей делать для него разметку? [image: image] https://user-images.githubusercontent.com/8464662/96287177-32901780-0fea-11eb-9d9f-c73c369ea4c5.png

1.

требования к высоте не выполнены [image: image] https://user-images.githubusercontent.com/8464662/96287281-5f442f00-0fea-11eb-96cb-a0e669f62bde.png [image: image] https://user-images.githubusercontent.com/8464662/96287599-ded1fe00-0fea-11eb-8a49-239572a3ed31.png

1.

не соответствует макету [image: image] https://user-images.githubusercontent.com/8464662/96287361-7c78fd80-0fea-11eb-92b3-f77119c0512c.png

1.

проблемы с внешним видом [image: image] https://user-images.githubusercontent.com/8464662/96287486-b0ecb980-0fea-11eb-978a-eaafbdff30e2.png [image: image] https://user-images.githubusercontent.com/8464662/96287511-bfd36c00-0fea-11eb-8d9e-8fd4c3434f32.png

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-710233628, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOQBYQ4P7BTGFXDVBM6RRVDSLB4NNANCNFSM4SNE5WDQ .

Alecto commented 4 years ago

16) не исправлено

17) не исправлено image

karina-doskuch commented 4 years ago

По поводу 16 я так понимаю что там только для размера от 768px до 576px не соответветствует верно? Потому что все остальные размеры у меня совпадают с шаблоном.

А вот по поводу 17 пункта не совсем пойму что нужно сделать? Кнопки должны становиться еще меньше? Просто раньше я их сделала как было на уроке и когда доходило до 576px кнопки становились одна под другой, вы сказали что это не соответствует шаблону и я сделала как на шаблоне, скажите пожалуйста что там не так? Я вижу что они не вписываются когда размер экрана становиться меньше 380px по ширине, но как должно быть когда он становиться еще меньше не ясно, шаблона для такого размера нет...

сб, 17 окт. 2020 г. в 18:22, Andrey notifications@github.com:

1.

не исправлено

1.

не исправлено [image: image] https://user-images.githubusercontent.com/8464662/96346621-6f6d1480-10a5-11eb-8c1a-32ddf4486a5e.png

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711028907, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOQBYQ25OGXV37WZKQLOEXLSLGZCHANCNFSM4SNE5WDQ .

Alecto commented 4 years ago

сб, 17 жовт. 2020 о 18:44 karina-doskuch notifications@github.com пише:

По поводу 16 я так понимаю что там только для размера от 768px до 576px не соответветствует верно?

везде не соответствует. даже если требования, описанные словами в макете не были поняты - выглядит секция плохо и неправильно на всех разрешениях. посмотрите мастер-шаблон, тогда станет понятно.

Потому что все остальные размеры у меня совпадают с шаблоном.

А вот по поводу 17 пункта не совсем пойму что нужно сделать? Кнопки должны становиться еще меньше?

в вашем шаблоне при отображении - ошибка, я показал ее на скрине. если по прежнему не понятно, сравните с мастер-шаблоном

Просто раньше я их сделала как было на уроке и когда доходило до 576px кнопки становились одна под другой, вы сказали что это не соответствует шаблону

размер не соответствует, вы исправили одну ошибку, сделали другую

и я сделала как на шаблоне, скажите пожалуйста что там не так? Я вижу что они не вписываются когда размер экрана становиться меньше 380px по ширине, но как должно быть когда он становиться еще меньше не ясно, шаблона для такого размера нет...

сб, 17 окт. 2020 г. в 18:22, Andrey notifications@github.com:

1.

не исправлено

1.

не исправлено [image: image] < https://user-images.githubusercontent.com/8464662/96346621-6f6d1480-10a5-11eb-8c1a-32ddf4486a5e.png>

— You are receiving this because you commented. Reply to this email directly, view it on GitHub < https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711028907>,

or unsubscribe < https://github.com/notifications/unsubscribe-auth/AOQBYQ25OGXV37WZKQLOEXLSLGZCHANCNFSM4SNE5WDQ>

.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711032498, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACASSFVZQ2E53BBZ6UXF24TSLG3W3ANCNFSM4SNE5WDQ .

karina-doskuch commented 4 years ago

Здравствуйте! Что вы имеете ввиду под мастер шаблоном? Шаблон в Abobe XD? У меня по 16 пункту блоки совпадают как по высоте так и по ширине для всех имеющихся в шаблоне разрешений, не совпадало только по высоте для размера от 576px до 768px, это я тоже только что поправила, может быть у вас страница закешировалась? По 17 пункту размер чего не совпадает? Опять же я сверяла размеры кнопок и расстояние между ними с имеющимися шаблонами в Adobe XD и у меня размеры и положение кнопок совпадают для всех разрешений? Все сравнения делала с помощью PerfectPixel плагина. Вы можете более детально описать что вы хотите видеть?

вс, 18 окт. 2020 г. в 08:05, Andrey notifications@github.com:

сб, 17 жовт. 2020 о 18:44 karina-doskuch notifications@github.com пише:

По поводу 16 я так понимаю что там только для размера от 768px до 576px не соответветствует верно?

везде не соответствует. даже если требования, описанные словами в макете не были поняты - выглядит секция плохо и неправильно на всех разрешениях. посмотрите мастер-шаблон, тогда станет понятно.

Потому что все остальные размеры у меня совпадают с шаблоном.

А вот по поводу 17 пункта не совсем пойму что нужно сделать? Кнопки должны становиться еще меньше?

в вашем шаблоне при отображении - ошибка, я показал ее на скрине. если по прежнему не понятно, сравните с мастер-шаблоном

Просто раньше я их сделала как было на уроке и когда доходило до 576px кнопки становились одна под другой, вы сказали что это не соответствует шаблону

размер не соответствует, вы исправили одну ошибку, сделали другую

и я сделала как на шаблоне, скажите пожалуйста что там не так? Я вижу что они не вписываются когда размер экрана становиться меньше 380px по ширине, но как должно быть когда он становиться еще меньше не ясно, шаблона для такого размера нет...

сб, 17 окт. 2020 г. в 18:22, Andrey notifications@github.com:

1.

не исправлено

1.

не исправлено [image: image] <

https://user-images.githubusercontent.com/8464662/96346621-6f6d1480-10a5-11eb-8c1a-32ddf4486a5e.png>

— You are receiving this because you commented. Reply to this email directly, view it on GitHub <

https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711028907>,

or unsubscribe <

https://github.com/notifications/unsubscribe-auth/AOQBYQ25OGXV37WZKQLOEXLSLGZCHANCNFSM4SNE5WDQ>

.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub < https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711032498>,

or unsubscribe < https://github.com/notifications/unsubscribe-auth/ACASSFVZQ2E53BBZ6UXF24TSLG3W3ANCNFSM4SNE5WDQ>

.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711117326, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOQBYQ6LKTP5ZJ3KUCNPMNLSLJZRFANCNFSM4SNE5WDQ .

Alecto commented 4 years ago

нд, 18 жовт. 2020 о 11:02 karina-doskuch notifications@github.com пише:

Здравствуйте! Что вы имеете ввиду под мастер шаблоном?

https://alecto.github.io/grid-template/

Шаблон в Abobe XD? У меня по 16 пункту блоки совпадают как по высоте так и по ширине для всех имеющихся в шаблоне разрешений, не совпадало только по высоте для размера от 576px до 768px, это я тоже только что поправила, может быть у вас страница закешировалась?

не думаю. проверяйте не только ширину, но и высоту.

По 17 пункту размер чего не совпадает? Опять же я сверяла размеры кнопок и расстояние между ними с имеющимися шаблонами в Adobe XD и у меня размеры и положение кнопок совпадают для всех разрешений? Все сравнения делала с помощью PerfectPixel плагина.

pixelperfect проверяет статику, вы делаете динамику под ЛЮБЫЕ разрешения. проверяйте с мастер-шаблоно, ссылка выше.

Вы можете более детально описать что вы хотите видеть?

полное совпадение.

вс, 18 окт. 2020 г. в 08:05, Andrey notifications@github.com:

сб, 17 жовт. 2020 о 18:44 karina-doskuch notifications@github.com пише:

По поводу 16 я так понимаю что там только для размера от 768px до 576px не соответветствует верно?

везде не соответствует. даже если требования, описанные словами в макете не были поняты - выглядит секция плохо и неправильно на всех разрешениях. посмотрите мастер-шаблон, тогда станет понятно.

Потому что все остальные размеры у меня совпадают с шаблоном.

А вот по поводу 17 пункта не совсем пойму что нужно сделать? Кнопки должны становиться еще меньше?

в вашем шаблоне при отображении - ошибка, я показал ее на скрине. если по прежнему не понятно, сравните с мастер-шаблоном

Просто раньше я их сделала как было на уроке и когда доходило до 576px кнопки становились одна под другой, вы сказали что это не соответствует шаблону

размер не соответствует, вы исправили одну ошибку, сделали другую

и я сделала как на шаблоне, скажите пожалуйста что там не так? Я вижу что они не вписываются когда размер экрана становиться меньше 380px по ширине, но как должно быть когда он становиться еще меньше не ясно, шаблона для такого размера нет...

сб, 17 окт. 2020 г. в 18:22, Andrey notifications@github.com:

1.

не исправлено

1.

не исправлено [image: image] <

https://user-images.githubusercontent.com/8464662/96346621-6f6d1480-10a5-11eb-8c1a-32ddf4486a5e.png>

— You are receiving this because you commented. Reply to this email directly, view it on GitHub <

https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711028907>,

or unsubscribe <

https://github.com/notifications/unsubscribe-auth/AOQBYQ25OGXV37WZKQLOEXLSLGZCHANCNFSM4SNE5WDQ>

.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <

https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711032498>,

or unsubscribe <

https://github.com/notifications/unsubscribe-auth/ACASSFVZQ2E53BBZ6UXF24TSLG3W3ANCNFSM4SNE5WDQ>

.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub < https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711117326>,

or unsubscribe < https://github.com/notifications/unsubscribe-auth/AOQBYQ6LKTP5ZJ3KUCNPMNLSLJZRFANCNFSM4SNE5WDQ>

.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711132034, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACASSFT6R2IHEODSAJ37I4LSLKOKBANCNFSM4SNE5WDQ .

karina-doskuch commented 4 years ago

В том то и дело что я проверяла и высоту и ширину для всех разрешений, и у меня совпадение очень высокое. А вот здесь https://alecto.github.io/grid-template/ высота у блоков skills меньше чем в шаблонах Abobe XD. После урока сброшу скриншоты.

вс, 18 окт. 2020 г. в 11:56, Andrey notifications@github.com:

нд, 18 жовт. 2020 о 11:02 karina-doskuch notifications@github.com пише:

Здравствуйте! Что вы имеете ввиду под мастер шаблоном?

https://alecto.github.io/grid-template/

Шаблон в Abobe XD? У меня по 16 пункту блоки совпадают как по высоте так и по ширине для всех имеющихся в шаблоне разрешений, не совпадало только по высоте для размера от 576px до 768px, это я тоже только что поправила, может быть у вас страница закешировалась?

не думаю. проверяйте не только ширину, но и высоту.

По 17 пункту размер чего не совпадает? Опять же я сверяла размеры кнопок и расстояние между ними с имеющимися шаблонами в Adobe XD и у меня размеры и положение кнопок совпадают для всех разрешений? Все сравнения делала с помощью PerfectPixel плагина.

pixelperfect проверяет статику, вы делаете динамику под ЛЮБЫЕ разрешения. проверяйте с мастер-шаблоно, ссылка выше.

Вы можете более детально описать что вы хотите видеть?

полное совпадение.

вс, 18 окт. 2020 г. в 08:05, Andrey notifications@github.com:

сб, 17 жовт. 2020 о 18:44 karina-doskuch notifications@github.com пише:

По поводу 16 я так понимаю что там только для размера от 768px до 576px не соответветствует верно?

везде не соответствует. даже если требования, описанные словами в макете не были поняты - выглядит секция плохо и неправильно на всех разрешениях. посмотрите мастер-шаблон, тогда станет понятно.

Потому что все остальные размеры у меня совпадают с шаблоном.

А вот по поводу 17 пункта не совсем пойму что нужно сделать? Кнопки должны становиться еще меньше?

в вашем шаблоне при отображении - ошибка, я показал ее на скрине. если по прежнему не понятно, сравните с мастер-шаблоном

Просто раньше я их сделала как было на уроке и когда доходило до 576px кнопки становились одна под другой, вы сказали что это не соответствует шаблону

размер не соответствует, вы исправили одну ошибку, сделали другую

и я сделала как на шаблоне, скажите пожалуйста что там не так? Я вижу что они не вписываются когда размер экрана становиться меньше 380px по ширине, но как должно быть когда он становиться еще меньше не ясно, шаблона для такого размера нет...

сб, 17 окт. 2020 г. в 18:22, Andrey notifications@github.com:

1.

не исправлено

1.

не исправлено [image: image] <

https://user-images.githubusercontent.com/8464662/96346621-6f6d1480-10a5-11eb-8c1a-32ddf4486a5e.png>

— You are receiving this because you commented. Reply to this email directly, view it on GitHub <

https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711028907>,

or unsubscribe <

https://github.com/notifications/unsubscribe-auth/AOQBYQ25OGXV37WZKQLOEXLSLGZCHANCNFSM4SNE5WDQ>

.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <

https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711032498>,

or unsubscribe <

https://github.com/notifications/unsubscribe-auth/ACASSFVZQ2E53BBZ6UXF24TSLG3W3ANCNFSM4SNE5WDQ>

.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub <

https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711117326>,

or unsubscribe <

https://github.com/notifications/unsubscribe-auth/AOQBYQ6LKTP5ZJ3KUCNPMNLSLJZRFANCNFSM4SNE5WDQ>

.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub < https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711132034>,

or unsubscribe < https://github.com/notifications/unsubscribe-auth/ACASSFT6R2IHEODSAJ37I4LSLKOKBANCNFSM4SNE5WDQ>

.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711137673, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOQBYQZ7SLZYHXLU7NPYCO3SLKUUXANCNFSM4SNE5WDQ .

karina-doskuch commented 4 years ago

Прилагаю скриншоты для каждого разрешения, да есть небольшое несоответствие, но не по высоте точно.

[image: chrome_qhFLQBJbrO.jpg]

[image: chrome_I21OeiKwEY.png]

[image: chrome_1qEsKUqG2D.png]

[image: chrome_12wyDfDW4b.png]

вс, 18 окт. 2020 г. в 12:04, Карина Доскуч karina.doskuch@gmail.com:

В том то и дело что я проверяла и высоту и ширину для всех разрешений, и у меня совпадение очень высокое. А вот здесь https://alecto.github.io/grid-template/ высота у блоков skills меньше чем в шаблонах Abobe XD. После урока сброшу скриншоты.

вс, 18 окт. 2020 г. в 11:56, Andrey notifications@github.com:

нд, 18 жовт. 2020 о 11:02 karina-doskuch notifications@github.com пише:

Здравствуйте! Что вы имеете ввиду под мастер шаблоном?

https://alecto.github.io/grid-template/

Шаблон в Abobe XD? У меня по 16 пункту блоки совпадают как по высоте так и по ширине для всех имеющихся в шаблоне разрешений, не совпадало только по высоте для размера от 576px до 768px, это я тоже только что поправила, может быть у вас страница закешировалась?

не думаю. проверяйте не только ширину, но и высоту.

По 17 пункту размер чего не совпадает? Опять же я сверяла размеры кнопок и расстояние между ними с имеющимися шаблонами в Adobe XD и у меня размеры и положение кнопок совпадают для всех разрешений? Все сравнения делала с помощью PerfectPixel плагина.

pixelperfect проверяет статику, вы делаете динамику под ЛЮБЫЕ разрешения. проверяйте с мастер-шаблоно, ссылка выше.

Вы можете более детально описать что вы хотите видеть?

полное совпадение.

вс, 18 окт. 2020 г. в 08:05, Andrey notifications@github.com:

сб, 17 жовт. 2020 о 18:44 karina-doskuch notifications@github.com пише:

По поводу 16 я так понимаю что там только для размера от 768px до 576px не соответветствует верно?

везде не соответствует. даже если требования, описанные словами в макете не были поняты - выглядит секция плохо и неправильно на всех разрешениях. посмотрите мастер-шаблон, тогда станет понятно.

Потому что все остальные размеры у меня совпадают с шаблоном.

А вот по поводу 17 пункта не совсем пойму что нужно сделать? Кнопки должны становиться еще меньше?

в вашем шаблоне при отображении - ошибка, я показал ее на скрине. если по прежнему не понятно, сравните с мастер-шаблоном

Просто раньше я их сделала как было на уроке и когда доходило до 576px кнопки становились одна под другой, вы сказали что это не соответствует шаблону

размер не соответствует, вы исправили одну ошибку, сделали другую

и я сделала как на шаблоне, скажите пожалуйста что там не так? Я вижу что они не вписываются когда размер экрана становиться меньше 380px по ширине, но как должно быть когда он становиться еще меньше не ясно, шаблона для такого размера нет...

сб, 17 окт. 2020 г. в 18:22, Andrey notifications@github.com:

1.

не исправлено

1.

не исправлено [image: image] <

https://user-images.githubusercontent.com/8464662/96346621-6f6d1480-10a5-11eb-8c1a-32ddf4486a5e.png>

— You are receiving this because you commented. Reply to this email directly, view it on GitHub <

https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711028907>,

or unsubscribe <

https://github.com/notifications/unsubscribe-auth/AOQBYQ25OGXV37WZKQLOEXLSLGZCHANCNFSM4SNE5WDQ>

.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <

https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711032498>,

or unsubscribe <

https://github.com/notifications/unsubscribe-auth/ACASSFVZQ2E53BBZ6UXF24TSLG3W3ANCNFSM4SNE5WDQ>

.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub <

https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711117326>,

or unsubscribe <

https://github.com/notifications/unsubscribe-auth/AOQBYQ6LKTP5ZJ3KUCNPMNLSLJZRFANCNFSM4SNE5WDQ>

.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub < https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711132034>,

or unsubscribe < https://github.com/notifications/unsubscribe-auth/ACASSFT6R2IHEODSAJ37I4LSLKOKBANCNFSM4SNE5WDQ>

.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711137673, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOQBYQZ7SLZYHXLU7NPYCO3SLKUUXANCNFSM4SNE5WDQ .

Alecto commented 4 years ago

прочитайте задачу, она даже словами расписана прямо в макете

нд, 18 жовт. 2020 о 12:04 karina-doskuch notifications@github.com пише:

В том то и дело что я проверяла и высоту и ширину для всех разрешений, и у меня совпадение очень высокое. А вот здесь https://alecto.github.io/grid-template/ высота у блоков skills меньше чем в шаблонах Abobe XD. После урока сброшу скриншоты.

вс, 18 окт. 2020 г. в 11:56, Andrey notifications@github.com:

нд, 18 жовт. 2020 о 11:02 karina-doskuch notifications@github.com пише:

Здравствуйте! Что вы имеете ввиду под мастер шаблоном?

https://alecto.github.io/grid-template/

Шаблон в Abobe XD? У меня по 16 пункту блоки совпадают как по высоте так и по ширине для всех имеющихся в шаблоне разрешений, не совпадало только по высоте для размера от 576px до 768px, это я тоже только что поправила, может быть у вас страница закешировалась?

не думаю. проверяйте не только ширину, но и высоту.

По 17 пункту размер чего не совпадает? Опять же я сверяла размеры кнопок и расстояние между ними с имеющимися шаблонами в Adobe XD и у меня размеры и положение кнопок совпадают для всех разрешений? Все сравнения делала с помощью PerfectPixel плагина.

pixelperfect проверяет статику, вы делаете динамику под ЛЮБЫЕ разрешения. проверяйте с мастер-шаблоно, ссылка выше.

Вы можете более детально описать что вы хотите видеть?

полное совпадение.

вс, 18 окт. 2020 г. в 08:05, Andrey notifications@github.com:

сб, 17 жовт. 2020 о 18:44 karina-doskuch notifications@github.com пише:

По поводу 16 я так понимаю что там только для размера от 768px до 576px не соответветствует верно?

везде не соответствует. даже если требования, описанные словами в макете не были поняты - выглядит секция плохо и неправильно на всех разрешениях. посмотрите мастер-шаблон, тогда станет понятно.

Потому что все остальные размеры у меня совпадают с шаблоном.

А вот по поводу 17 пункта не совсем пойму что нужно сделать? Кнопки должны становиться еще меньше?

в вашем шаблоне при отображении - ошибка, я показал ее на скрине. если по прежнему не понятно, сравните с мастер-шаблоном

Просто раньше я их сделала как было на уроке и когда доходило до 576px кнопки становились одна под другой, вы сказали что это не соответствует шаблону

размер не соответствует, вы исправили одну ошибку, сделали другую

и я сделала как на шаблоне, скажите пожалуйста что там не так? Я вижу что они не вписываются когда размер экрана становиться меньше 380px по ширине, но как должно быть когда он становиться еще меньше не ясно, шаблона для такого размера нет...

сб, 17 окт. 2020 г. в 18:22, Andrey notifications@github.com:

1.

не исправлено

1.

не исправлено [image: image] <

https://user-images.githubusercontent.com/8464662/96346621-6f6d1480-10a5-11eb-8c1a-32ddf4486a5e.png>

— You are receiving this because you commented. Reply to this email directly, view it on GitHub <

https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711028907>,

or unsubscribe <

https://github.com/notifications/unsubscribe-auth/AOQBYQ25OGXV37WZKQLOEXLSLGZCHANCNFSM4SNE5WDQ>

.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <

https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711032498>,

or unsubscribe <

https://github.com/notifications/unsubscribe-auth/ACASSFVZQ2E53BBZ6UXF24TSLG3W3ANCNFSM4SNE5WDQ>

.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub <

https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711117326>,

or unsubscribe <

https://github.com/notifications/unsubscribe-auth/AOQBYQ6LKTP5ZJ3KUCNPMNLSLJZRFANCNFSM4SNE5WDQ>

.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <

https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711132034>,

or unsubscribe <

https://github.com/notifications/unsubscribe-auth/ACASSFT6R2IHEODSAJ37I4LSLKOKBANCNFSM4SNE5WDQ>

.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub < https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711137673>,

or unsubscribe < https://github.com/notifications/unsubscribe-auth/AOQBYQZ7SLZYHXLU7NPYCO3SLKUUXANCNFSM4SNE5WDQ>

.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711138584, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACASSFVXPCUEAMTX7KGEFLTSLKVTBANCNFSM4SNE5WDQ .

karina-doskuch commented 4 years ago

Спасибо! Да прочитала, увидела в чем проблема, исправила, посмотрите пожалуйста.

ссылка на репозиторий: https://github.com/karina-doskuch/grid-project ссылка на github pages: https://karina-doskuch.github.io/grid-project/

вс, 18 окт. 2020 г. в 15:56, Andrey notifications@github.com:

прочитайте задачу, она даже словами расписана прямо в макете

нд, 18 жовт. 2020 о 12:04 karina-doskuch notifications@github.com пише:

В том то и дело что я проверяла и высоту и ширину для всех разрешений, и у меня совпадение очень высокое. А вот здесь https://alecto.github.io/grid-template/ высота у блоков skills меньше чем в шаблонах Abobe XD. После урока сброшу скриншоты.

вс, 18 окт. 2020 г. в 11:56, Andrey notifications@github.com:

нд, 18 жовт. 2020 о 11:02 karina-doskuch notifications@github.com пише:

Здравствуйте! Что вы имеете ввиду под мастер шаблоном?

https://alecto.github.io/grid-template/

Шаблон в Abobe XD? У меня по 16 пункту блоки совпадают как по высоте так и по ширине для всех имеющихся в шаблоне разрешений, не совпадало только по высоте для размера от 576px до 768px, это я тоже только что поправила, может быть у вас страница закешировалась?

не думаю. проверяйте не только ширину, но и высоту.

По 17 пункту размер чего не совпадает? Опять же я сверяла размеры кнопок и расстояние между ними с имеющимися шаблонами в Adobe XD и у меня размеры и положение кнопок совпадают для всех разрешений? Все сравнения делала с помощью PerfectPixel плагина.

pixelperfect проверяет статику, вы делаете динамику под ЛЮБЫЕ разрешения. проверяйте с мастер-шаблоно, ссылка выше.

Вы можете более детально описать что вы хотите видеть?

полное совпадение.

вс, 18 окт. 2020 г. в 08:05, Andrey notifications@github.com:

сб, 17 жовт. 2020 о 18:44 karina-doskuch notifications@github.com

пише:

По поводу 16 я так понимаю что там только для размера от 768px до 576px не соответветствует верно?

везде не соответствует. даже если требования, описанные словами в макете не были поняты - выглядит секция плохо и неправильно на всех разрешениях. посмотрите мастер-шаблон, тогда станет понятно.

Потому что все остальные размеры у меня совпадают с шаблоном.

А вот по поводу 17 пункта не совсем пойму что нужно сделать? Кнопки должны становиться еще меньше?

в вашем шаблоне при отображении - ошибка, я показал ее на скрине. если по прежнему не понятно, сравните с мастер-шаблоном

Просто раньше я их сделала как было на уроке и когда доходило до 576px кнопки становились одна под другой, вы сказали что это не соответствует шаблону

размер не соответствует, вы исправили одну ошибку, сделали другую

и я сделала как на шаблоне, скажите пожалуйста что там не так? Я вижу что они не вписываются когда размер экрана становиться меньше 380px по ширине, но как должно быть когда он становиться еще меньше не ясно, шаблона для такого размера нет...

сб, 17 окт. 2020 г. в 18:22, Andrey notifications@github.com:

1.

не исправлено

1.

не исправлено [image: image] <

https://user-images.githubusercontent.com/8464662/96346621-6f6d1480-10a5-11eb-8c1a-32ddf4486a5e.png>

— You are receiving this because you commented. Reply to this email directly, view it on GitHub <

https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711028907>,

or unsubscribe <

https://github.com/notifications/unsubscribe-auth/AOQBYQ25OGXV37WZKQLOEXLSLGZCHANCNFSM4SNE5WDQ>

.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <

https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711032498>,

or unsubscribe <

https://github.com/notifications/unsubscribe-auth/ACASSFVZQ2E53BBZ6UXF24TSLG3W3ANCNFSM4SNE5WDQ>

.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub <

https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711117326>,

or unsubscribe <

https://github.com/notifications/unsubscribe-auth/AOQBYQ6LKTP5ZJ3KUCNPMNLSLJZRFANCNFSM4SNE5WDQ>

.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <

https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711132034>,

or unsubscribe <

https://github.com/notifications/unsubscribe-auth/ACASSFT6R2IHEODSAJ37I4LSLKOKBANCNFSM4SNE5WDQ>

.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub <

https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711137673>,

or unsubscribe <

https://github.com/notifications/unsubscribe-auth/AOQBYQZ7SLZYHXLU7NPYCO3SLKUUXANCNFSM4SNE5WDQ>

.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub < https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711138584>,

or unsubscribe < https://github.com/notifications/unsubscribe-auth/ACASSFVXPCUEAMTX7KGEFLTSLKVTBANCNFSM4SNE5WDQ>

.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711164140, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOQBYQ4WTYED42UDSGCPQS3SLLQXTANCNFSM4SNE5WDQ .

Alecto commented 4 years ago

проверил. без изменений, высота рассчитана неправильно.

нд, 18 жовт. 2020 о 19:27 karina-doskuch notifications@github.com пише:

Спасибо! Да прочитала, увидела в чем проблема, исправила, посмотрите пожалуйста.

ссылка на репозиторий: https://github.com/karina-doskuch/grid-project ссылка на github pages: https://karina-doskuch.github.io/grid-project/

вс, 18 окт. 2020 г. в 15:56, Andrey notifications@github.com:

прочитайте задачу, она даже словами расписана прямо в макете

нд, 18 жовт. 2020 о 12:04 karina-doskuch notifications@github.com пише:

В том то и дело что я проверяла и высоту и ширину для всех разрешений, и у меня совпадение очень высокое. А вот здесь https://alecto.github.io/grid-template/ высота у блоков skills меньше чем в шаблонах Abobe XD. После урока сброшу скриншоты.

вс, 18 окт. 2020 г. в 11:56, Andrey notifications@github.com:

нд, 18 жовт. 2020 о 11:02 karina-doskuch notifications@github.com пише:

Здравствуйте! Что вы имеете ввиду под мастер шаблоном?

https://alecto.github.io/grid-template/

Шаблон в Abobe XD? У меня по 16 пункту блоки совпадают как по высоте так и по ширине для всех имеющихся в шаблоне разрешений, не совпадало только по высоте для размера от 576px до 768px, это я тоже только что поправила, может быть у вас страница закешировалась?

не думаю. проверяйте не только ширину, но и высоту.

По 17 пункту размер чего не совпадает? Опять же я сверяла размеры кнопок и расстояние между ними с имеющимися шаблонами в Adobe XD и у меня размеры и положение кнопок совпадают для всех разрешений? Все сравнения делала с помощью PerfectPixel плагина.

pixelperfect проверяет статику, вы делаете динамику под ЛЮБЫЕ разрешения. проверяйте с мастер-шаблоно, ссылка выше.

Вы можете более детально описать что вы хотите видеть?

полное совпадение.

вс, 18 окт. 2020 г. в 08:05, Andrey notifications@github.com:

сб, 17 жовт. 2020 о 18:44 karina-doskuch < notifications@github.com>

пише:

По поводу 16 я так понимаю что там только для размера от 768px до 576px не соответветствует верно?

везде не соответствует. даже если требования, описанные словами в макете не были поняты

выглядит секция плохо и неправильно на всех разрешениях. посмотрите мастер-шаблон, тогда станет понятно.

Потому что все остальные размеры у меня совпадают с шаблоном.

А вот по поводу 17 пункта не совсем пойму что нужно сделать? Кнопки должны становиться еще меньше?

в вашем шаблоне при отображении - ошибка, я показал ее на скрине. если по прежнему не понятно, сравните с мастер-шаблоном

Просто раньше я их сделала как было на уроке и когда доходило до 576px кнопки становились одна под другой, вы сказали что это не соответствует шаблону

размер не соответствует, вы исправили одну ошибку, сделали другую

и я сделала как на шаблоне, скажите пожалуйста что там не так? Я вижу что они не вписываются когда размер экрана становиться меньше 380px по ширине, но как должно быть когда он становиться еще меньше не ясно, шаблона для такого размера нет...

сб, 17 окт. 2020 г. в 18:22, Andrey notifications@github.com:

1.

не исправлено

1.

не исправлено [image: image] <

https://user-images.githubusercontent.com/8464662/96346621-6f6d1480-10a5-11eb-8c1a-32ddf4486a5e.png>

— You are receiving this because you commented. Reply to this email directly, view it on GitHub <

https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711028907>,

or unsubscribe <

https://github.com/notifications/unsubscribe-auth/AOQBYQ25OGXV37WZKQLOEXLSLGZCHANCNFSM4SNE5WDQ>

.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <

https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711032498>,

or unsubscribe <

https://github.com/notifications/unsubscribe-auth/ACASSFVZQ2E53BBZ6UXF24TSLG3W3ANCNFSM4SNE5WDQ>

.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub <

https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711117326>,

or unsubscribe <

https://github.com/notifications/unsubscribe-auth/AOQBYQ6LKTP5ZJ3KUCNPMNLSLJZRFANCNFSM4SNE5WDQ>

.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <

https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711132034>,

or unsubscribe <

https://github.com/notifications/unsubscribe-auth/ACASSFT6R2IHEODSAJ37I4LSLKOKBANCNFSM4SNE5WDQ>

.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub <

https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711137673>,

or unsubscribe <

https://github.com/notifications/unsubscribe-auth/AOQBYQZ7SLZYHXLU7NPYCO3SLKUUXANCNFSM4SNE5WDQ>

.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <

https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711138584>,

or unsubscribe <

https://github.com/notifications/unsubscribe-auth/ACASSFVXPCUEAMTX7KGEFLTSLKVTBANCNFSM4SNE5WDQ>

.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub < https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711164140>,

or unsubscribe < https://github.com/notifications/unsubscribe-auth/AOQBYQ4WTYED42UDSGCPQS3SLLQXTANCNFSM4SNE5WDQ>

.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711275562, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACASSFXBGKQFQM7EMXCHR6TSLMJQPANCNFSM4SNE5WDQ .

karina-doskuch commented 4 years ago

Добрый день! Как не правильно, если вот условие

[image: image.png]

вот результат

[image: image.png]

Секция занимает ровно один экран браузера с учетом отступов сверху и снизу. Кнопка не участвует в расчете. Минимальная высота задана. Что не так и как без изменений?

пн, 19 окт. 2020 г. в 14:42, Andrey notifications@github.com:

проверил. без изменений, высота рассчитана неправильно.

нд, 18 жовт. 2020 о 19:27 karina-doskuch notifications@github.com пише:

Спасибо! Да прочитала, увидела в чем проблема, исправила, посмотрите пожалуйста.

ссылка на репозиторий: https://github.com/karina-doskuch/grid-project ссылка на github pages: https://karina-doskuch.github.io/grid-project/

вс, 18 окт. 2020 г. в 15:56, Andrey notifications@github.com:

прочитайте задачу, она даже словами расписана прямо в макете

нд, 18 жовт. 2020 о 12:04 karina-doskuch notifications@github.com пише:

В том то и дело что я проверяла и высоту и ширину для всех разрешений, и у меня совпадение очень высокое. А вот здесь https://alecto.github.io/grid-template/ высота у блоков skills меньше чем в шаблонах Abobe XD. После урока сброшу скриншоты.

вс, 18 окт. 2020 г. в 11:56, Andrey notifications@github.com:

нд, 18 жовт. 2020 о 11:02 karina-doskuch notifications@github.com

пише:

Здравствуйте! Что вы имеете ввиду под мастер шаблоном?

https://alecto.github.io/grid-template/

Шаблон в Abobe XD? У меня по 16 пункту блоки совпадают как по высоте так и по ширине для всех имеющихся в шаблоне разрешений, не совпадало только по высоте для размера от 576px до 768px, это я тоже только что поправила, может быть у вас страница закешировалась?

не думаю. проверяйте не только ширину, но и высоту.

По 17 пункту размер чего не совпадает? Опять же я сверяла размеры кнопок и расстояние между ними с имеющимися шаблонами в Adobe XD и у меня размеры и положение кнопок совпадают для всех разрешений? Все сравнения делала с помощью PerfectPixel плагина.

pixelperfect проверяет статику, вы делаете динамику под ЛЮБЫЕ разрешения. проверяйте с мастер-шаблоно, ссылка выше.

Вы можете более детально описать что вы хотите видеть?

полное совпадение.

вс, 18 окт. 2020 г. в 08:05, Andrey notifications@github.com:

сб, 17 жовт. 2020 о 18:44 karina-doskuch < notifications@github.com>

пише:

По поводу 16 я так понимаю что там только для размера от 768px до 576px не соответветствует верно?

везде не соответствует. даже если требования, описанные словами в макете не были поняты

выглядит секция плохо и неправильно на всех разрешениях. посмотрите мастер-шаблон, тогда станет понятно.

Потому что все остальные размеры у меня совпадают с шаблоном.

А вот по поводу 17 пункта не совсем пойму что нужно сделать? Кнопки должны становиться еще меньше?

в вашем шаблоне при отображении - ошибка, я показал ее на скрине. если по прежнему не понятно, сравните с мастер-шаблоном

Просто раньше я их сделала как было на уроке и когда доходило до 576px кнопки становились одна под другой, вы сказали что это не соответствует шаблону

размер не соответствует, вы исправили одну ошибку, сделали другую

и я сделала как на шаблоне, скажите пожалуйста что там не так? Я вижу что они не вписываются когда размер экрана становиться меньше 380px по ширине, но как должно быть когда он становиться еще меньше не ясно, шаблона для такого размера нет...

сб, 17 окт. 2020 г. в 18:22, Andrey < notifications@github.com>:

1.

не исправлено

1.

не исправлено [image: image] <

https://user-images.githubusercontent.com/8464662/96346621-6f6d1480-10a5-11eb-8c1a-32ddf4486a5e.png>

— You are receiving this because you commented. Reply to this email directly, view it on GitHub <

https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711028907>,

or unsubscribe <

https://github.com/notifications/unsubscribe-auth/AOQBYQ25OGXV37WZKQLOEXLSLGZCHANCNFSM4SNE5WDQ>

.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <

https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711032498>,

or unsubscribe <

https://github.com/notifications/unsubscribe-auth/ACASSFVZQ2E53BBZ6UXF24TSLG3W3ANCNFSM4SNE5WDQ>

.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub <

https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711117326>,

or unsubscribe <

https://github.com/notifications/unsubscribe-auth/AOQBYQ6LKTP5ZJ3KUCNPMNLSLJZRFANCNFSM4SNE5WDQ>

.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <

https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711132034>,

or unsubscribe <

https://github.com/notifications/unsubscribe-auth/ACASSFT6R2IHEODSAJ37I4LSLKOKBANCNFSM4SNE5WDQ>

.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub <

https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711137673>,

or unsubscribe <

https://github.com/notifications/unsubscribe-auth/AOQBYQZ7SLZYHXLU7NPYCO3SLKUUXANCNFSM4SNE5WDQ>

.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <

https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711138584>,

or unsubscribe <

https://github.com/notifications/unsubscribe-auth/ACASSFVXPCUEAMTX7KGEFLTSLKVTBANCNFSM4SNE5WDQ>

.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub <

https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711164140>,

or unsubscribe <

https://github.com/notifications/unsubscribe-auth/AOQBYQ4WTYED42UDSGCPQS3SLLQXTANCNFSM4SNE5WDQ>

.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub < https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-711275562>,

or unsubscribe < https://github.com/notifications/unsubscribe-auth/ACASSFXBGKQFQM7EMXCHR6TSLMJQPANCNFSM4SNE5WDQ>

.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-712100029, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOQBYQ5YOE3ERIV4OJQS5ITSLQQ37ANCNFSM4SNE5WDQ .

Alecto commented 4 years ago

В вашем письме отсутствуют какие либо изображения, но я понимаю что вы хотели показать, потому что сам писал это условие. Вы не выполнили поставленную задачу.

karina-doskuch commented 4 years ago

я не пойму что вы хотите? Я вижу условие и вижу что я его выполнила. Если вы говорите что я его не выполнила, тогда пожалуйста объясните что именно я сделала не так?

пн, 19 окт. 2020 г. в 15:50, Andrey notifications@github.com:

В вашем письме отсутствуют какие либо изображения, но я понимаю что вы хотели показать, потому что сам писал это условие. Вы не выполнили поставленную задачу.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/karina-doskuch/grid-project/issues/1#issuecomment-712133981, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOQBYQ56AXIUSE7LILBCNITSLQYZ3ANCNFSM4SNE5WDQ .

Alecto commented 4 years ago

Карина, уже наверное с десяток писем я пытаюсь вам объяснить, что не так. Понимаю, что многие не смотрят уроки, а если смотрят, то не видят решений или не обращают внимание на детали. Поэтому в самом макете я расписал, что требуется сделать. Вам этого оказалось недостаточно, поэтому на скринах тикета я несколько раз показал, где именно вы ошиблись и даже указал в чем ошибка. Также, я предложил вам сравнить с мастер-шаблоном в надежде, что вы все таки увидите решение и поймете свою ошибку. Все это не привело к правильному результату. Повторять одно и тоже далее не имеет абсолютно никакого смысла. Вы либо разберетесь и решите задачу, либо нет. Решать за вас - считаю неправильным. На данный момент контрольная работа не выполнена и не принята.