Team work review

[haiarha]

HTML CSS Review

visual

✅ is good

• у вебсайта очень продуманная адаптивность
• в целом все аккуратно, и хорошо смотрится на всех экранах
• классно смотрится мобильная навигация, при открытии смещающая контент страницы вниз и вся навигация в целом работает (подскроливается страница к секции по клику)

❗️can be even better

Можно по клику на стрелку вниз подскролить на следующую секцию


Немного скачет контент под табами при наведении на табы (можно поправить, если манипулировать цветом рамки, вместо того, чтобы добавлять и убирать ее)


HTML validation

• ✅ В целом все гуд, все картинки содержат не пустой атрибут alt итд.
• ❗️ Валидатор, вижу, посоветовал присвоить value для option

CSS validation

✅ Тут все отлично)

DevTools Coverage

✅ Ненужных правил (стили, которые не примениются ни к одному элементу страницы) нет, или их совсем мало

HTML semantics

✅ Семантика использована правильно (есть header, footer, nav, main, section)

BEM methodology

✅ На мой взгляд, методология БЕМ применена правильно

other recomendations

✅ В остальном все ОК 👍

[haiarha]

@Beetroot-FE-20210415/y 👆

[Kainos-SU]

Круто. Не думав шо тут так можна. По поводу проскрола - я цим займусь. Титульна секція ща мною.

[Kainos-SU]

Питання. А як можна добитись плавного прскрола? Куди копати? Де дивитись?

[haiarha]

Вот, пожалуйста, нужно указать элементу, содержимое которого скроллится

https://developer.mozilla.org/ru/docs/Web/CSS/scroll-behavior

[Kainos-SU]

Перевірив адаптив на своєму телефоні - прийдеться допрацьовувати.

[SerhiiHlukhov]

Я також перевірив, в мене правильно відображає, але пару інших елементів по інакшому( браузер Сафарі)

Пн, 7 июня 2021 г. в 10:15, Kainos-SU @.***>:

[image: Screenshot_20210607-101409.png] https://user-images.githubusercontent.com/62204744/120974730-26bfb400-c779-11eb-939b-cd2d538b94a8.png

Перевірив адаптив на своєму телефоні - прийдеться допрацьовувати.

— You are receiving this because you are on a team that was mentioned. Reply to this email directly, view it on GitHub https://github.com/Beetroot-FE-20210415/team-y/issues/22#issuecomment-855660726, or unsubscribe https://github.com/notifications/unsubscribe-auth/ATWTBPQXJWOGJK3XJARNVH3TRRWZRANCNFSM46E7UGBA .