forked from ViktoryiaYatskova/clean-code-s1e1
-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
clean-code-s1e1 #2
Open
AtlasShd
wants to merge
16
commits into
master
Choose a base branch
from
clean-code-s1e1
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains 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
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.
https://github.com/rolling-scopes-school/tasks/blob/master/stage1/modules/clean-code/clean-code-s1e1.md
https://atlasshd.github.io/clean-code-s1e1/
[ ] Правило '1.1 Отступы' из html-and-css выполняется во всем проекте: табуляция не используется, все отступы выполнены пробельным символом: (+2)
✓ Вместе с этим выполнен пункт 3.6, так как были добавлены правильные пробельные отступы во всём проекте.
[ ] Правило '1.2 Нижний регистр написания' из html-and-css выполняется во всем проекте: все HTML теги, атрибуты из значения, селекторы и их значения записаны в нижнем регистре. (+2)
[ ] Правило '1.3 Кавычки в HTML/CSS документе' из html-and-css выполняется во всем проекте: в HTML и СSS файлах используются только двойные кавычки. (+2)
[ ] Правило '2.1 Форматирование' из html-and-css выполняется во всем проекте: все блочные, табличные и списочные элементы перенесены на отдельную строку и выделены отступами в соответствии с лестничной иерархией вложенности: (+2)
✓ Не перенесено внутри: a, label, h2, button, так как внутри них обычный текст.
✓ Не перенесён элемент label внутри h1, потому что это строчный элемент.
✓ Не перенесены элементы img внутри button, так как img являются строчными элементами.
[ ] Правило '2.2 Тип документа / Document Type' из html-and-css выполняется во всем проекте: doctype присутствует первым тегом в html документе и соответствует типу html5: (+2)
✓ Пункт корректно выполнен в самом конце для удобства работы, так как всё смещается при его изменении.
[ ] Правило '2.3 Символы-мнемоники' из html-and-css выполняется во всем проекте: символы-мнемоники не используются в html файлах. (+2)
[ ] Правило '2.4 Атрибут 'type'' из html-and-css выполняется во всем проекте: CSS-стили и JS-скрипты подключены без использования данного атрибута. (+2)
[ ] Правило '3.1 Единый стиль именования селекторов (классов / id)' из html-and-css выполняется во всем проекте: все селекторы именованы в едином стиле, при этом или в соответствии с БЭМ, или все слова в нижнем регистре и разделены между собой дефисом: (+2)
✓ Именование произведено в соответствии с БЭМ, поэтому пункт 3.2 также одновременно выполнен.
[ ] Правило '3.2 Значимые названия идентификаторов и классов' из html-and-css выполняется во всем проекте: все имена классов и идентификаторов имеют или осмысленные имена в соответствии с их функциональным значением (!но НЕ ВНЕШНИМ ВИДОМ!), или шаблонное имя (также соответствующее назначению элемента). (+2)
[ ] Правило '3.3 Лаконичность названий идентификаторов и классов' из html-and-css выполняется во всем проекте: все имена id и классов понятны, достаточно длинные для понимания их назначения и при этом лаконичны. (+2)
[ ] Правило '3.4 Теговые селекторы' из html-and-css выполняется во всем проекте: теговые селекторы не используются (за исключением намеренного сброса дефолтных стилей): (+2)
✓ При рефакторинге стилей также был выполнен пункт 3.8.
[ ] Правило '3.5 Отступы в блоках' из html-and-css выполняется во всем проекте: содержимое всех блоков отделены отступами. (+2)
[ ] Правило '3.6 Пробел после названий свойств' из html-and-css выполняется во всем проекте: значения CSS-свойств отделены пробелами после двоеточия: (+2)
✓ Выполнен в пункте 1.1.
[ ] Правило '3.7 Точка с запятой после свойств' из html-and-css выполняется во всем проекте: после каждого CSS-правила стоит точка с запятой. (+2)
[ ] Правило '3.8 Разделение селекторов и свойств' из html-and-css выполняется во всем проекте: каждый селектор в групповом перечислении CSS-блока отделен переносом строки: (+2)
✓ Выполнен в пункте 3.4.
[ ] Правило '1.1 Семантика' из html-and-css-extended соблюдено: все html-теги используются в соответствии с их назначением. При этом использование div-элементов сведено к минимуму, где возможно они заменены на семантические html5 элементы: (+5)
✓ Добавлен header.
✓ Добавлен main.
✓ Добавлены section.
✓ Добавлен заголовок h1.
✓ Изменены заголовки h3 на h2.
[ ] Правило '1.2 Альтернатива для мультимедиа' из html-and-css-extended соблюдено: каждый мультимедиа элемент (в данном задании img) имеет alt-атрибут с осмысленным содержанием, соответствующим контенту элементы. При этом чисто декоративные картинки имеют alt с пустым значением: (+5)
✓ Добавлен alt к главному изображению в header со значением: "The Eisenhower Matrix".
✓ Добавлен alt к svg изображениям кнопок remove с пустым значением, потому что они носят декоративный характер.
[ ] Правило '2.1 БЭМ' из html-and-css-extended соблюдено: все классы именованы согласно БЭМ нотации: (+5)
✓ Доработан пункт 3.1.
[ ] Обоснованное подозрение на списывание: число коммитов, порядок, текст сообщения и их содержание полностью совпадает с чужой работой: (-0)
✓ Работу делал сам
[ ] Работа с невыполненным обязательным требованием задания "Весь функционал приложения должен исправно работать после ваших изменений" (исключение, если функциональность была изначально поломана в исходном репозитории до изменений студента): (-0)
✓ Весь функционал рабочий, js-файл адаптирован.
[ ] Формат коммит сообщений составлен вразрез указаным в задании требованиям: (-0)
✓ Формат коммит соответствует указанным в задании требованиям.
[ ] Все правки сделаны в ОДНОМ коммите. (-0)
[ ] Коммит содержит изменения по нескольким правилам, НЕ перекрывающим друг друга: (-0)
✓ Имеются два случая, когда коммит содержит изменения по нескольким правилам, но перекрывающие друг друга.
[ ] Использовано неправильное название ветки или все сделано в master ветке. (-0)
[ ] Pull Request сделан НЕ в свой репозиторий. (-0)