#1 Review module

Open
opened 4 years ago by OlgaMaslova · 1 comments
  1. https://screencast.com/t/XT6QREuL2 - так как мы находимся на странице About Us, то лого должно быть ссылкой и вести на хоум-пейдж

  2. https://screencast.com/t/tzzQoNSJik - такого не бывает. Зачем 2 меню??

  3. https://screencast.com/t/NjFBW6uxyccw - текст внутри поля не выровнен

  4. https://screencast.com/t/A5crlr2kgmEu - зачем все эти стили? Ты ограничил контент со всех сторон (например добавила текст, а блок не тянется https://screencast.com/t/v4J3zttE)

  5. https://screencast.com/t/i4m2xU9Skdl - Это должны быть отступы от елементов, а не br

  6. https://screencast.com/t/8vvHLjl2 - p это больше текст. В твоем варианте это похоже на заголовок

  7. https://screencast.com/t/PS09SA2oO - ты КАЖДОМУ елементу внутри дал марджины. Почему вместо этого не дать паддинг на родителя? И весь блок в отступах лишних. Почему бы не сделать так: https://screencast.com/t/YMQjnKMHoCV

  8. https://screencast.com/t/l29c9Aa3Zw8 - это называется "костыли". А если добавится контент, что будет?

  9. https://screencast.com/t/vpcHf9dt6O5E - не может быть 10 h1 в одном блоке. h1 - это главный заголовок страницы (ну максисум тега section или article) и од должен быть ОДИН.

  10. https://screencast.com/t/HQfqnCFb2p - Это не цитата, обычный текст.

  11. https://screencast.com/t/UTr8RuiqNV - при добавлении текста нет отступа

  12. Очень много лишних стилей https://screencast.com/t/L5VjIn9b

  13. https://screencast.com/t/3IxKXh3d0DG - добавила еще 4 блока, и их обрезало (к этому привело ограничение высоты)

  14. чтобы у картинок не было отступа снизу, им надо прописать

  15. https://screencast.com/t/oSgcHPzlPEXm - можно было бы хоть в цвет иконку сделать (фиолетовую)

  16. при добавлении текста галлерея не должна тянуться - https://screencast.com/t/F6jR2NV9DC

  17. не может сколько быть заголовков h1 - https://screencast.com/t/uxrCOcneKY

18.не отцентрировано - https://screencast.com/t/EoED7q8M

  1. отцентрировать елементы https://screencast.com/t/s8nbUkoFomgy

  2. структуру не доделал - https://screencast.com/t/qC7gwU7O

  3. при респонсиве разные отступы - https://screencast.com/t/yS7wXsPxft

  4. на мобилке уменьшить отступ - https://screencast.com/t/4mgriRhN

  5. https://screencast.com/t/dhpY0Mx2Njt - Почему стили лежат отдельно?

  6. https://screencast.com/t/LFEKQ0otzXFK - не надо прописывать и *, атк как прописав эти стили на body, они будут унаследованы всеми чаилдами

  7. https://screencast.com/t/EtQ0fLs4 - не надо ширину

  8. https://screencast.com/t/vtxOQyGF - на p стили старайся не прописывать (это глобальное правило). Их на родителя надо писать + альтернативный шрифт нужно писать

Думала, что сделаешь работу лучше ...

Оценка: 7

1. https://screencast.com/t/XT6QREuL2 - так как мы находимся на странице About Us, то лого должно быть ссылкой и вести на хоум-пейдж 2. https://screencast.com/t/tzzQoNSJik - такого не бывает. Зачем 2 меню?? 3. https://screencast.com/t/NjFBW6uxyccw - текст внутри поля не выровнен 4. https://screencast.com/t/A5crlr2kgmEu - зачем все эти стили? Ты ограничил контент со всех сторон (например добавила текст, а блок не тянется https://screencast.com/t/v4J3zttE) 5. https://screencast.com/t/i4m2xU9Skdl - Это должны быть отступы от елементов, а не `br` 6. https://screencast.com/t/8vvHLjl2 - `p` это больше текст. В твоем варианте это похоже на заголовок 7. https://screencast.com/t/PS09SA2oO - ты КАЖДОМУ елементу внутри дал марджины. Почему вместо этого не дать паддинг на родителя? И весь блок в отступах лишних. Почему бы не сделать так: https://screencast.com/t/YMQjnKMHoCV 8. https://screencast.com/t/l29c9Aa3Zw8 - это называется "костыли". А если добавится контент, что будет? 9. https://screencast.com/t/vpcHf9dt6O5E - не может быть 10 `h1` в одном блоке. `h1` - это главный заголовок страницы (ну максисум тега `section` или `article`) и од должен быть ОДИН. 10. https://screencast.com/t/HQfqnCFb2p - Это не цитата, обычный текст. 11. https://screencast.com/t/UTr8RuiqNV - при добавлении текста нет отступа 12. Очень много лишних стилей https://screencast.com/t/L5VjIn9b 13. https://screencast.com/t/3IxKXh3d0DG - добавила еще 4 блока, и их обрезало (к этому привело ограничение высоты) 14. чтобы у картинок не было отступа снизу, им надо прописать 15. https://screencast.com/t/oSgcHPzlPEXm - можно было бы хоть в цвет иконку сделать (фиолетовую) 16. при добавлении текста галлерея не должна тянуться - https://screencast.com/t/F6jR2NV9DC 17. не может сколько быть заголовков `h1` - https://screencast.com/t/uxrCOcneKY 18.не отцентрировано - https://screencast.com/t/EoED7q8M 19. отцентрировать елементы https://screencast.com/t/s8nbUkoFomgy 20. структуру не доделал - https://screencast.com/t/qC7gwU7O 21. при респонсиве разные отступы - https://screencast.com/t/yS7wXsPxft 22. на мобилке уменьшить отступ - https://screencast.com/t/4mgriRhN 23. https://screencast.com/t/dhpY0Mx2Njt - Почему стили лежат отдельно? 24. https://screencast.com/t/LFEKQ0otzXFK - не надо прописывать и `*`, атк как прописав эти стили на `body`, они будут унаследованы всеми чаилдами 25. https://screencast.com/t/EtQ0fLs4 - не надо ширину 26. https://screencast.com/t/vtxOQyGF - на `p` стили старайся не прописывать (это глобальное правило). Их на родителя надо писать + альтернативный шрифт нужно писать Думала, что сделаешь работу лучше ... Оценка: 7
OlgaMaslova commented 4 years ago
Poster
  1. https://screencast.com/t/Y873Xjfp - при респонсиве "дырка"

  2. форма не выровняна https://screencast.com/t/yfZehj4OaRtm

  3. бургер-меню не видно https://screencast.com/t/blLoPrlE

А вообще-то могу сказать, что ты сделал большую работу над ошибками. Молодец!

Совет на будущее: рассматривай блоки как пользователь: удобно ли тебе будет пользоваться таким видом продукта. Как сайт может измениться, если добавить или удалить контент.

Повысила оценку до 8,5 ;)

1. https://screencast.com/t/Y873Xjfp - при респонсиве "дырка" 2. форма не выровняна https://screencast.com/t/yfZehj4OaRtm 3. бургер-меню не видно https://screencast.com/t/blLoPrlE А вообще-то могу сказать, что ты сделал большую работу над ошибками. Молодец! Совет на будущее: рассматривай блоки как пользователь: удобно ли тебе будет пользоваться таким видом продукта. Как сайт может измениться, если добавить или удалить контент. Повысила оценку до 8,5 ;)
Sign in to join this conversation.
No Label
No Milestone
No assignee
1 Participants
Loading...
Cancel
Save
There is no content yet.