Open
Conversation
ghbd
reviewed
Dec 6, 2021
There was a problem hiding this comment.
Критерии
- Соответствие макету
- 1.1 Все элементы из макета сверстаны
- 1.2 Шрифты, размер, цвет и начертания текста соответствуют макету Комментарии
.main-screen__title,.gallery__title,press__title,.footer__subscription-titlefont-weight не по макету, http://joxi.ru/KAxWB4oIv7E8g2 - 1.3 Ширина элементов и отступы отличаются от макета не более, чем на
30pxна десктопной версии и на10pxна остальных размерах экрана, для которых подготовлен макет. Отличия в размерах не должны визуально противоречить макету и выбиваться из заложенной в макет сетки, в этом случае требуется более строгое соответствие размерам макета Комментарии http://joxi.ru/V2V8bZLi86zG3r http://joxi.ru/brRkzW5CLjoVnm http://joxi.ru/v29k36PCRePY3r http://joxi.ru/KAxWB4oIv7E8g2 - 1.4 Все блоки соответствуют макету в последних версиях Firefox, Google Chrome или Yandex Browser
- Адаптив
- 2.1 Отсутствует горизонтальный скролл на всех размерах экрана 320px и более
- 2.2 Установленная максимальная ширина контента соответствует макету
- 2.3 Верстка не ломается между брейкпоинтами
- 2.4 Брейкпоинты сгруппированы. Если два брейкпоинта имеют небольшую пиксельную разницу в медиа-правиле они объединены в один
- 2.5 Текст не выпадает из блоков на всех размерах экрана 320px и более
- 2.6 Изображения не искажаются и выглядят как в макете на всех размерах экрана
- 2.7 Одинаковые свойства в разных медиа-правилах не дублируются
- Верстка
- 3.1 Нет ошибок валидации https://validator.w3.org/nu/ (Erorrs)
- 3.2 Используются
<main>,<section>,<header>,<footer>,<ul>,<nav>,<h1-h6>и<p>там, где это необходимо - 3.3 Не используются элементы
<b>,<br>,<i> - 3.4 Отсутствуют лишние
divобертки - 3.5 Контентные картинки вставлены через
<img>, и в них присутствуетaltс описанием, которое соответствует языку страницы - 3.6 На странице есть заголовок первого уровня
- 3.7 Заголовки соответствуют иерархии страницы
- 3.8 Корректно задан
viewport, прописаныtitleиlang.
- CSS
- 4.1 Не используется
!important - 4.2 Абсолютное позиционирование не используется без непосредственной необходимости **Комментарии
.footer__copyright,.gallery__button-set,.header__close-menu-button,.header__menu-wrapper,.header__menu-wrapper,header__menu,.logo_place_footer,.main-screen__image,.press__article-link,.press__article,.press__button-set - 4.3 Использование
heightиmax-heightне приводит к выпадению текста или блоков при переполнении Комментарии.footer__subscription,.header__menu-wrapper,.header,.press__articles,.press__title - 4.4 Любой интерактивный элемент имеет состояние наведения, состояние наведения берется из макета или используется прозрачность Комментарии
.footer__submit-button - 4.5 Для каждого шрифта указаны альтернативные варианты из системных шрифтов и тип шрифта
- 4.1 Не используется
- Применение БЭМ
- 5.1 Нет нарушения методологии БЭМ: нет элементов без родительского блока, нет элементов элементов, модификаторы не используются без указания того, что они модифицируют
- 5.2 Не используются селекторы по тегу
- 5.3 Для одинаковых блоков в макете переиспользуется стили одного БЭМ-блока Комментарии
.gallery__title,press__title,.footer__subscription-titleможно объединить одним БЭМ блоком и не дублировать стили - 5.4 Числа в названиях классов не используются, за исключением классов для элементов, указанных в руководстве по стилю (Style Guide)
- 5.5 Модификатор не отражает позицию элемента в разметке (
paragraph_first) Комментарииgallery__button_type_left,.gallery__button_type_right - 5.6 Модификаторы не содержат дубли стилей модифицируемого элемента или блока, только изменяющиеся свойства
- 5.7 Файловая структура построена по БЭМ Nested или БЭМ Flat
- 6. Функциональность на JS работает корректно (слайдеры, бургерное меню или другие требования согласно заданию на проект)
- Структура проекта и оформление кода
- 7.1 Соблюдена единообразная файловая структура проекта, например, изображения в папке images, шрифты в папке fonts, стили в папке blocks, скрипты в папке scripts, а не разбросаны в разных папках и не хранятся в корне проекта
- 7.2 Код в проекте отформатирован единообразно
- 7.3 Нет нарушений иерархии отступов в разметке, стилях и файлах скриптов
- Использование SCSS
- 8.1 Файлы стилей имеют расширение SCSS, подключаемый к странице CSS файл получен генерацией из SCSS файлов
- 8.2 Повторяющеся значения вынесены в переменные, например, используемые цвета, шрифты, размеры экрана для медиазапросов Комментарии используемые цвета, шрифты, размеры экрана для медиазапросов нужно вынести в sass переменные
- 8.3 Корректно используются миксины, миксины не состоят из одного CSS свойства
Количество баллов: 78
- Страница внешне похожа на макет, при этом большое кол-во элементов спозициционировано абсолютно. При изменении контента страница с текущими стилями сломается
ghbd
reviewed
Dec 8, 2021
ghbd
left a comment
ghbd
left a comment
There was a problem hiding this comment.
Критерии
- Соответствие макету
- 1.1 Все элементы из макета сверстаны
- 1.2 Шрифты, размер, цвет и начертания текста соответствуют макету у Комментарии:
.main-screen__title,.titlefont-weight не по макету - 1.3 Ширина элементов и отступы отличаются от макета не более, чем на
30pxна десктопной версии и на10pxна остальных размерах экрана, для которых подготовлен макет. Отличия в размерах не должны визуально противоречить макету и выбиваться из заложенной в макет сетки, в этом случае требуется более строгое соответствие размерам макета Комментарии: http://joxi.ru/E2pNE41C4l7Y1r http://joxi.ru/v29k36PCR8ZL5r http://joxi.ru/DmBM8OLigRJ5dr - 1.4 Все блоки соответствуют макету в последних версиях Firefox, Google Chrome или Yandex Browser
- Адаптив
- 2.1 Отсутствует горизонтальный скролл на всех размерах экрана 320px и более
- 2.2 Установленная максимальная ширина контента соответствует макету
- 2.3 Верстка не ломается между брейкпоинтами
- 2.4 Брейкпоинты сгруппированы. Если два брейкпоинта имеют небольшую пиксельную разницу в медиа-правиле они объединены в один
- 2.5 Текст не выпадает из блоков на всех размерах экрана 320px и более
- 2.6 Изображения не искажаются и выглядят как в макете на всех размерах экрана
- 2.7 Одинаковые свойства в разных медиа-правилах не дублируются
- Верстка
- 3.1 Нет ошибок валидации https://validator.w3.org/nu/ (Erorrs)
- 3.2 Используются
<main>,<section>,<header>,<footer>,<ul>,<nav>,<h1-h6>и<p>там, где это необходимо - 3.3 Не используются элементы
<b>,<br>,<i> - 3.4 Отсутствуют лишние
divобертки - 3.5 Контентные картинки вставлены через
<img>, и в них присутствуетaltс описанием, которое соответствует языку страницы - 3.6 На странице есть заголовок первого уровня
- 3.7 Заголовки соответствуют иерархии страницы
- 3.8 Корректно задан
viewport, прописаныtitleиlang.
- CSS
- 4.1 Не используется
!important - 4.2 Абсолютное позиционирование не используется без непосредственной необходимости Комментарий:
.header__menu-button,.header__menu-wrapper,.main-screen__image,.press__article - 4.3 Использование
heightиmax-heightне приводит к выпадению текста или блоков при переполнении - 4.4 Любой интерактивный элемент имеет состояние наведения, состояние наведения берется из макета или используется прозрачность Комментарий:
.footer__submit-button - 4.5 Для каждого шрифта указаны альтернативные варианты из системных шрифтов и тип шрифта
- 4.1 Не используется
- Применение БЭМ
- 5.1 Нет нарушения методологии БЭМ: нет элементов без родительского блока, нет элементов элементов, модификаторы не используются без указания того, что они модифицируют
- 5.2 Не используются селекторы по тегу
- 5.3 Для одинаковых блоков в макете переиспользуется стили одного БЭМ-блока
- 5.4 Числа в названиях классов не используются, за исключением классов для элементов, указанных в руководстве по стилю (Style Guide)
- 5.5 Модификатор не отражает позицию элемента в разметке (
paragraph_first) - 5.6 Модификаторы не содержат дубли стилей модифицируемого элемента или блока, только изменяющиеся свойства
- 5.7 Файловая структура построена по БЭМ Nested или БЭМ Flat
- 6. Функциональность на JS работает корректно (слайдеры, бургерное меню или другие требования согласно заданию на проект)
- Структура проекта и оформление кода
- 7.1 Соблюдена единообразная файловая структура проекта, например, изображения в папке images, шрифты в папке fonts, стили в папке blocks, скрипты в папке scripts, а не разбросаны в разных папках и не хранятся в корне проекта
- 7.2 Код в проекте отформатирован единообразно
- 7.3 Нет нарушений иерархии отступов в разметке, стилях и файлах скриптов
- Использование SCSS
- 8.1 Файлы стилей имеют расширение SCSS, подключаемый к странице CSS файл получен генерацией из SCSS файлов
- 8.2 Повторяющеся значения вынесены в переменные, например, используемые цвета, шрифты, размеры экрана для медиазапросов Комментарии: используемые цвета, шрифты, размеры экрана для медиазапросов нужно вынести в sass переменные
- 8.3 Корректно используются миксины, миксины не состоят из одного CSS свойства
Количество баллов: 86
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Почти готовая страница.
Не сделан слайдер для изображений в галерее.