rolling-scopes-school / support

15 stars 3 forks source link

Cross-Check 'Shelter DOM' - ' vadimhmys' #887

Closed vadimhmys closed 1 year ago

vadimhmys commented 1 year ago
  1. A link to your deployed project
  2. A link to the project repository on Github.
  3. A link to the task
  4. A Link to the checklist for evaluating the task (if it exists)
  5. A screenshot of cross-checking marks Screenshot_1 Screenshot_2
  6. A final score after self-assessment, with comments Оцениваю свою работу на 110 баллов. Получил 0 т.к. не смог проверить работы других студентов, потому что во время кросс-чека находился на стационарном лечении в больнице. Медицинскую справку прилагаю. IMG_20230419_100502
  7. A cross-check score of your Score 0
haslie22 commented 1 year ago
Реализация burger menu на обеих страницах 26/26

В идеале css-правило, определяющее паддинг (выделила на скриншоте) применить к классу .header__nav, а не к .pull-out-menu, так как из-за этого ссылки "подскакивают", когда меню начинает закрываться. Хорошо, когда есть разделение ответственности: стили для одного класса отвечают за внешний вид, стили для другого — за анимацию/транзишн. Но на оценку в данном случае это не влияет.

https://i2.paste.pics/7e8a315859e695c108197e2bc1801939.png

При выборе темной цветовой схемы стоило бы перекрасить ссылки в меню на второй странице в светлый цвет, так как контрастность с фоном у них очень маленькая, текст читается плохо.

Реализация слайдера-карусели на странице Main 36/36
Реализация пагинации на странице Pets 36/36
Реализация попап на обеих страницах 12/12

Итоговый балл: 110/110

@helenakrasnova check please

helenakrasnova commented 1 year ago

@haslie22 thanks a lot!