search

inconsumable1812 / Toxin

MetaLamp, task 2. Сайт Toxin
0 stars 0 forks source link

PUG #4

Closed ArsenGadjiusmanov closed 2 years ago

ArsenGadjiusmanov commented 2 years ago

  1. Лучше использовать семантические теги. Для кнопок есть тег button. image

  2. Для html используем двойные кавычки тык image

  3. Отступы делать пробелами, на проекте во всех файлах html, pug, etc должен быть одинаковый отступ — либо 4, либо 2 пробела. (Рекомендуемое значение 2).

  4. Длинна строки в 80 символов считается одной из удобнейших для чтения, надо стараться не отходить далеко от этой цифры. image

  5. image

  6. Должен быть единый стиль для всех компонентов. В некоторых миксинах у тебя есть перевод строки в некоторых нет. По Airbnb добавлять отступы в начале и конце блока нельзя тык image

  7. Теги div можно опустить в данных случаях image

  8. Имя блока/элемента должно соответствовать имени существительному, из названия должно быть понятно, чем данный блок является. Мне не особо понятно что за "Тип" image

  9. Про запятые. Про табуляцию. А еще лучше если ты это вынесешь в отдельный json и будешь брать данные оттуда и передавать в миксин image

P.s. наши стандарты основаны на стандартах airbnb и лишь уточняют и дополняют некоторые моменты.

ArsenGadjiusmanov commented 2 years ago

Там где используется Js 'кавычки' должны быть одинарными в html "двойные". Js в pug не является исключением

https://github.com/inconsumable1812/Toxin/blob/50e2fd3b0901cd8a823d2c95d4d8162ac62a290f/src/layout/main.pug#L19-L23

а здесь в атрибутах двойные https://github.com/inconsumable1812/Toxin/blob/50e2fd3b0901cd8a823d2c95d4d8162ac62a290f/src/layout/main.pug#L6-L16

ArsenGadjiusmanov commented 2 years ago

Тк в паге возможно использование js кода, то нужно придерживаться всех стандартов для js, наших и airbnb. image

Вот тут про отступы между curly brackets

ArsenGadjiusmanov commented 2 years ago

По дефолту button имеет type submit, окажись данный компонент в форме то начнет вызывать перезапуск страницы при клике, желательно явно проставлять type="button" когда мы его подразумеваем. image

Зачем тут tabindex=0 у button? image

ArsenGadjiusmanov commented 2 years ago

Ссылка она либо ссылка либо кнопка) Тег a не может содержать в себе кнопку image

Надо пройтись валидатором и проверить все страницы w3

ArsenGadjiusmanov commented 2 years ago

Отсутствует <!DOCTYPE html> на страницах

ArsenGadjiusmanov commented 2 years ago

image

ArsenGadjiusmanov commented 2 years ago

Надо будет к единому стилю привести. После знака - ты на этом уровне вложенности можешь писать js код. image

ArsenGadjiusmanov commented 2 years ago

image Тут надо поправить форматирование тык

ArsenGadjiusmanov commented 2 years ago

Должно быть 2 пробела image Не правильное форматирование объекта . Нельзя перенести половину пропсов, либо все либо ничего) image

ArsenGadjiusmanov commented 2 years ago

image Строка длиннее 80 символов

ArsenGadjiusmanov commented 2 years ago

image Пробелы curly brackets

ArsenGadjiusmanov commented 2 years ago

image

ArsenGadjiusmanov commented 2 years ago

image Тут можно применить destructuring

ArsenGadjiusmanov commented 2 years ago

image image шо за классы такие?)

ArsenGadjiusmanov commented 2 years ago

image Теги без классов нельзя)

ArsenGadjiusmanov commented 2 years ago

image link можно деструктурировать.

Может я не писал но в issue я указываю на одну ошибку, а остальные схожие по проекту ты сам должен найти и исправить)

ArsenGadjiusmanov commented 2 years ago

image

ArsenGadjiusmanov commented 2 years ago

image Для конкатенации строк мы предпочитаем использовать обратные кавычки ``

ArsenGadjiusmanov commented 2 years ago

image тут у тебя вроде бы и кавычки есть, но почему-то все еще складываешь) Масло масленое. ` ```${pageClass + '__header'}``` =>${pageClass}__header`

ArsenGadjiusmanov commented 2 years ago

image Надо исправить