Check

Check

dp

Б4. Грубые ошибки в разметке отсутствуют.

http://prntscr.com/iy49cj , http://prntscr.com/iy49qk - в проекте можно обойтись без br.

У Адрес главного офиса<br> и офлайн-магазина: я оставила чтобы не задавать фикс. ширину для блоков с текстом. Что скажешь? Подгонять один элемент, разумно ли? И не понимаю почему в данном случае br ошибка.


http://prntscr.com/iy4dgs - какое действие будет происходит по нажатию кнопки? Текст с названием мороженого лишь имитирует ссылку, а на самом деле это span. Получается, что мы никак не можем попасть на страницу продукта.

В ТЗ про поведение кнопки и ссылки ничего не написано.
Заменить название на ссылку? И кнопку переделать в ссылку?
Вообще все элементы страницы переделать в ссылку @_@


Б9. Подключены правильные шрифты, их размеры, высота строк, цвет и толщина равны соответствующим параметрам в макетах и техническом задании.

*высота строки теперь везде задана коэффициентом

http://prntscr.com/iy4ly6 - исправлено .sub-nav { line-height: 1.14; }

http://prntscr.com/iy4mud - исправлено .form-field {line-height: 1.5;}

http://prntscr.com/iy4nk5 - исправлено .form-field {line-height: 1.5;}

http://prntscr.com/iy4o6b - исправлено на line-height: 1.61;
line-height: 1.84; или line-height: 24px; неверный размер. Высота строки в макете пересекается http://prntscr.com/iznju4 заменено на line-height: 1.61;

http://prntscr.com/iy4p04 - - исправлено. для телефона задан line-height: 1.09; white-space: nowrap;

? http://prntscr.com/iy4phw - не буду менять, потому что задан общий стиль для кнопок и white-space: nowrap;Но переделала немного свойства для большой кнопки. (Предполагалось задать для большой кнопки line-height: 1.38; для всех кнопок line-height: 1.33;)

http://prntscr.com/iy4pvz - исправлено .promo__title { line-height: 1.17;}

http://prntscr.com/iy4r6d - исправлено product__price {line-height: 1.5;}

https://prnt.sc/iy4rmc - исправлено product__price {line-height: 1.5;}

http://prntscr.com/iy4s4j - исправлено font-weight: 500; но стало меньше похоже на макет

http://prntscr.com/iy4svi - исправлено .features__content { color: #323232;}

?! http://prntscr.com/iy4tb5 - высота строки задана коэффициентом. Разница с макетом 0,08 px

http://prntscr.com/iy4tm4 - исправлено .features__content { color: #323232;}

http://prntscr.com/iy4u4q - исправлено .blog-post__text { line-height: 1.38;}

http://prntscr.com/iy4ui7 - исправлено .blog-post__link {line-height: 1.25;}

http://prntscr.com/iy4ux2 - исправлено .subscribe__text { line-height: 1.38; color: #5a5a5a;}

http://prntscr.com/iy5nhu - исправлено .form-field {line-height: 1.5;}

http://prntscr.com/iy5o3m - разница с макетом 1px. высота исправлена на line-height: 1.43;

?! http://prntscr.com/iy5okv - высота строки задана коэффициентом. Разница с макетом 0,06px. line-height: 1.33;

http://prntscr.com/iy5ovn - исправлено .location__info color: #323232;

http://prntscr.com/iy5p46 - исправлено .location__info color: #323232;

http://prntscr.com/iy5q0l - исправлено .copyright { line-height: 1.5; }

http://prntscr.com/iy5rlg - исправлено .breadcrumbs { line-height: 1.13; }

?! http://prntscr.com/iy5rxe - высота строки задана коэффициентом. Разница с макетом 0,05 px

http://prntscr.com/iy5tat - исправлено pagination__page line-height: 1.13;



Б12. Вёрстка идентично отображается в последних версиях браузеров Chrome, Opera, Firefox, Safari, Edge, а также в Internet Explorer 11+.

Firefox: http://prntscr.com/iy5w59 -высота textarea больше.

Исправлено.

задала min-height, уменьшила row

весь макет на firefox уезжает примерно на 3-4px (мне кажется из-за текста)

? Edge: http://prntscr.com/iy5xn2, http://prntscr.com/iy5yjy, http://prntscr.com/iy5yuo

С телефонами так и не понятно. Для текста который уехал я сделала меньше padding.

IE 11: http://prntscr.com/iy603w - попытка открыть форму обратной связи.

Исправлено.

Проблематично попасть в выпадающие элементы: меню, форму поиска, логина, корзину. Продемонстрировать в скриншоте это невозможно.

Исправлено.

Добавила псевдоэлемент при hover для .nav__link как мостик. Должно лучше работать.



✓ Б11. При наполнении контентом (как в макете) элементы каждой страницы соответствуют макету.

http://prntscr.com/iy5u3s

Исправлено.

текст был не жирным.
Размер кнопок принудительно увеличен за счет увеличения padding padding: 10px 20px; Остальные кнопки чувствую себя нормально.

http://prntscr.com/iy5v0w

Исправлено. Уменьшение margin на 3px .catalog__item {... margin-bottom: 37px; ...}


Б13. Сайт нормально смотрится на минимальной для макета ширине.

http://prntscr.com/iy62n2

Исправлено


Дополнительные критерии


✓ Д5. Использован normalize.css.

normalize.css есть, но не подключен.

✓ Д23. Проведена базовая оптимизация: минифицированы стили и скрипты (при наличии).

normalize.css не минимизирован

тут вопросов нет.


Д9. Для CSS-свойств с префиксом указан его вариант без префикса, и это указание идёт последним.

Префиксов практически нет нигде.

Тут не согласна. Префиксы есть для свойств рекомендованных по ссылке http://shouldiprefix.com/.

Автопрефиксы вставлены через https://autoprefixer.github.io/ru/ с фильтром defaults, которые покрывает все браузеры указанные в ТЗ.

У меня нет префиксов только для свойств display: flex; которые для IE вроде и не нужны.

Я добавляю их после стилизации.


✓ Д11. Все состояния элементов (смотрите styleguide.psd) прописаны в стилевом файле.

У вас: http://prntscr.com/iy6efi, в макете: http://prntscr.com/iy6epb

Исправлено


✓ Д14. Собственные скрипты собраны в отдельном JavaScript-файле, который подключён в HTML перед закрывающим тегом body.

http://prntscr.com/iy6g2d - подключено более 1 скрипта.

Исправлено


Д20. Критическая функциональность сайта работоспособна без JavaScript (использовано прогрессивное улучшение).

http://prntscr.com/iy6j0q - без JS невозможно будет попасть на страницу формы.

Button заменен ссылкой с href
Исправлено?

<a class="location__btn page-btn" href="writeus.html" type="button">Форма обратной связи</a>




Report Page