rolling-scopes-school / support

15 stars 3 forks source link

Cross-Check 'museum-adaptive' - 'ereticor' #302

Closed ereticor closed 2 years ago

ereticor commented 2 years ago
Ссылка на деплой вашего проекта - https://rolling-scopes-school.github.io/ereticor-JSFE2021Q3/museum/
Ссылка на репозиторий с проектом на GitHub - https://github.com/rolling-scopes-school/ereticor-JSFE2021Q3.git
Ссылка на задание - https://github.com/rolling-scopes-school/tasks/blob/master/tasks/museum/museum-adaptive.md
Ссылка на форму для проверки проекта (если такая есть) - https://museum-adaptive-cross-check.netlify.app/
Скриншот с оценками за cross-check (контакты проверяющих должны быть скрыты) - одна оценка

изображение

Оценка в результате самопроверки с ее аргументацией - 154 балла (152 если учитывать отхождение ссылок в footer на разрешениях менее 350px)
Вёрстка соответствует макету. Ширина экрана 1024px +40

Блок header +4 Секция Welcome +4 Секция Visiting +4 Секция Explore +4 Секция Video +4 Секция Gallery +4 Секция Tickets +4 Форма покупки билетов +4 Секция Contacts +4 Блок footer +4

Вёрстка соответствует макету. Ширина экрана 768px +40 Блок header +4 Секция Welcome +4 Секция Visiting +4 Секция Explore +4 Секция Video +4 Секция Gallery +4 Секция Tickets +4 Форма покупки билетов +4 Секция Contacts +4 Блок footer +4

Вёрстка соответствует макету. Ширина экрана 420px +40 Блок header +4 Секция Welcome +4 Секция Visiting +4 Секция Explore +4 Секция Video +4 Секция Gallery +4 Секция Tickets +4 Форма покупки билетов +4 Секция Contacts +4 Блок footer +4 Ни на одном из разрешений до 320px включительно не появляется горизонтальная полоса прокрутки +6 Совмещается адаптивная и респонсивная (резиновая) вёрстка +12 При изменении ширины экрана плавно изменяются размеры: слайдера в секции Welcome +2 слайдера сравнения изображений в секции Explore +2 кастомного видеоплеера в секции Video +0 слайдера в секции Video +2 YouTube-видео в плейлисте в секции Video, маленькие видео выровнены по краям большого +2 галереи изображений и изображений в ней +2 карты +2

На ширине экрана 1024рх и меньше реализовано адаптивное меню +12 при нажатии на бургер-иконку меню появляется, плавно выдвигаясь слева, бургер-иконка изменяется на крестик. При нажатии на крестик меню исчезает, плавно возвращаясь назад, иконка крестика превращается в бургер-иконку +2 ссылки в меню работают, обеспечивая плавную прокрутку по якорям +2 при клике по ссылке в адаптивном меню, или при клике по любому месту сайта, кроме самого адаптивного меню, меню закрывается +2 вёрстка меню соответствует макету на всех проверяемых разрешениях +6 Оптимизация скорости загрузки страницы +4 Результат проверки скорости сайта для мобильных устройств: 0 to 49 (red): Poor - не выполнено, 0 баллов 50 to 89 (orange): Needs Improvement - частично выполнено +4 90 to 100 (green): Good - выполнено полностью +0

Балл за cross-check из вашего Score - 118
ereticor commented 2 years ago

Вторая оценка за крос-чек (через два или три дня после дедлайна) При проверке прошу скачать файлы с ветки museum и проверять их, т.к. gh-pages уже отредактирован под museum-dom изображение

valerydluski commented 2 years ago

https://frosty-lumiere-4e352b.netlify.app/ это версия с gh-pages на момент дедлайна, проверять по этой ссылке

Ilya758 commented 2 years ago

Ваша оценка - 153 балла Отзыв по пунктам ТЗ: Частично выполненные пункты: 1) вёрстка меню соответствует макету на всех проверяемых разрешениях feedback: при разрешении 768px отсутствуют три картины во всплывающем меню в соответствии с макетом, на разрешении 420px они отображаются корректно +3 2) Результат проверки скорости сайта для мобильных устройств: 0 to 49 (red): Poor - не выполнено 0 ,баллов; 50 to 89 (orange): Needs Improvement - частично выполнено - 4 баллов; 90 to 100 (green): Good - выполнено полностью - 8 баллов feedback: результаты проверки: 85 / 87 / 83 + 4 Выполненные пункты: 1) Блок header

2) Секция Welcome

3) Секция Visiting

4) Секция Explore

5) Секция Video

6) Секция Gallery

7) Секция Tickets

8) Форма покупки билетов

9) Секция Contacts

10) Блок footer

11) Блок header

12) Секция Welcome

13) Секция Visiting

14) Секция Explore

15) Секция Video

16) Секция Gallery

17) Секция Tickets

18) Форма покупки билетов

19) Секция Contacts

20) Блок footer

21) Блок header

22) Секция Welcome

23) Секция Visiting

24) Секция Explore

25) Секция Video

26) Секция Gallery

27) Секция Tickets

28) Форма покупки билетов

29) Секция Contacts

30) Блок footer

31) Ни на одном из разрешений до 320px включительно не появляется горизонтальная полоса прокрутки, элементы не должны скрываться, обрезаться, наезжать друг на друга, если это не предусмотрено макетом.

32) слайдера в секции Welcome

33) слайдера сравнения изображений в секции Explore

34) кастомного видеоплеера в секции Video

35) слайдера в секции Video

36) YouTube-видео в плейлисте в секции Video, маленькие видео выровнены по краям большого

37) галереи изображений и изображений в ней

38) карты

39) при нажатии на бургер-иконку меню появляется, плавно выдвигаясь слева, бургер-иконка изменяется на крестик. При нажатии на крестик меню исчезает, плавно возвращаясь назад, иконка крестика превращается в бургер-иконку

40) ссылки в меню работают, обеспечивая плавную прокрутку по якорям

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

Послесловие: размеры блоков подстраиваются под изменение размеров экрана корректно. Однако, верно подчеркнули, что высота параллакс блока выше требуемой при разрешении менее 1920px. За ошибку не считаю, т.к. форма проверки не предусматривает этого. Успехов!

viGITory commented 2 years ago

Ваша оценка - 110 баллов Отзыв по пунктам ТЗ:

Не выполненные/не засчитанные пункты: 1) Секция Explore (768px) feedback: Слайдер не соответствует макету 2) Секция Explore (420px) feedback: Слайдер не соответствует макету 3) Ни на одном из разрешений до 320px включительно не появляется горизонтальная полоса прокрутки, элементы не должны скрываться, обрезаться, наезжать друг на друга, если это не предусмотрено макетом. feedback: Есть полоса прокрутки скрытая overflow-x: hidden 4) слайдера в секции Welcome (респонсив) feedback: Слайдер обрезается 5) кастомного видеоплеера в секции Video (респонсив) feedback: Видео обрезается 6) слайдера в секции Video (респонсив) feedback: Слайды обрезаются, пропорции нарушаются 7) карты (респонсив) feedback: Не сохраняются пропорции

Частично выполненные пункты: 1) Секция Welcome (1024px) feedback: Нет тени на слайдере 2) Секция Explore (1024px) feedback: Слайдер не соответствует макету 3) Секция Video (1024px) feedback: Видеоплеер не соответствуют макету 4) Секция Contacts (1024px) feedback: Карта не соответствует макету 5) Секция Video (768px) feedback: Плеер не соответствуют макету 6) Секция Tickets (768px) feedback: Масштаб картины не соответствует макету 7) Секция Contacts (768px) feedback: Карта не соответствует макету 8) Секция Visiting (420px) feedback: Расхождения по PP больше 10px 9) Секция Video (420px) feedback: Плеер не соответствуют макету 10) Секция Contacts (420px) feedback: Карта не соответствует макету 11) слайдера сравнения изображений в секции Explore (респонсив) feedback: 810-825px слайдер вываливается за границы секции 12) вёрстка меню соответствует макету на всех проверяемых разрешениях feedback: На 1024 текста под меню быть не должно. На 768 меню всегда растягивается на всю высоту, проверить соцсети на PP невозможно. На 420 не соответствует макету 13) Результат проверки скорости сайта для мобильных устройств: 0 to 49 (red): Poor - не выполнено 0 ,баллов; 50 to 89 (orange): Needs Improvement - частично выполнено - 4 баллов; 90 to 100 (green): Good - выполнено полностью - 8 баллов feedback: 69-75-82

Выполненные пункты: 1) Блок header 2) Секция Visiting 3) Секция Gallery 4) Секция Tickets 5) Форма покупки билетов 6) Блок footer
7) Блок header 8) Секция Welcome 9) Секция Visiting 10) Секция Gallery 11) Форма покупки билетов 12) Блок footer
13) Блок header 14) Секция Welcome 15) Секция Gallery 16) Секция Tickets 17) Форма покупки билетов 18) Блок footer
19) YouTube-видео в плейлисте в секции Video, маленькие видео выровнены по краям большого 20) галереи изображений и изображений в ней 21) при нажатии на бургер-иконку меню появляется, плавно выдвигаясь слева, бургер-иконка изменяется на крестик. При нажатии на крестик меню исчезает, плавно возвращаясь назад, иконка крестика превращается в бургер-иконку 22) ссылки в меню работают, обеспечивая плавную прокрутку по якорям 23) при клике по ссылке в адаптивном меню, или при клике по любому месту сайта, кроме самого адаптивного меню, меню закрывается

ereticor commented 2 years ago
  1. Ни на одном из разрешений до 320px включительно не появляется горизонтальная полоса прокрутки, элементы не должны скрываться, обрезаться, наезжать друг на друга, если это не предусмотрено макетом. feedback: Есть полоса прокрутки скрытая overflow-x: hidden

Серьезно? Видимо людям с 500+ позиций виднее

valerydluski commented 2 years ago

Ну учитывая, что я тоже смотрел работу, видимо да