search

xenialugovaya / Hotel-booking-service-template

MIT License
0 stars 0 forks source link

БЭМ #6

Closed iatsdotfatr closed 4 years ago

iatsdotfatr commented 4 years ago

Безымянный Один блок - один препроцессорный файл. Если number-sign используется не только в booking-header, создается соответствующий отдельный блок со своими шаблонами-стилями-поведением, если нет, тогда number-sign является элементом booking-header и именуется соответственно бэм-элементу. Исправить по всему проекту.

iatsdotfatr commented 4 years ago
  1. К блокам не применяются свойства внешнего позиционирования Безымянный1
  2. Некорректное наименование модификатора типа "ключ-значение" Безымянный2
  3. Зачем нужен данный класс? Безымянный3 Классы с общими стилями нельзя использовать, вся разметка должна соответствовать БЭМ, то есть класс может являться только блоком/элементом/модификатором, исключением условно (в зависимости от проверяющего :) ) могут быть классы из подключаемых модулей, типа material-icons.

  4. Безымянный4 @-импорты убрать, переопределяемые стили объединить в отдельном файле и переместить (один блок - один препроцессорный файл) в папку с соответствующим названием, jquery, кажется, подключен глобально, зачем тогда импорт.

iatsdotfatr commented 4 years ago
  1. Раздел calendar лучше переименовать в datepicker в соответствии с классом в стилях.
  2. Important! в своих стилях не использовать, фиксированную высоту не использовать (см. также UI issue). Комментарии или удалить, или раскомментировать. Безымянный
  3. Подобные комментарии также удалить: Безымянный Комментарий "//forms" лучше так же удалить, объем кода небольшой и без него все понятно.
iatsdotfatr commented 4 years ago
  1. Фиксированную высоту убрать, для блоков также не используется фиксированная ширина, проверить по всему проекту. outer geometry
  2. Использовать модификаторы. modifiers
  3. Файл с шаблоном также должен быть один, или создать новые блоки, если надо, или объединить. one-template
iatsdotfatr commented 4 years ago

Фиксированная геометрия блока: Бэм fixed

iatsdotfatr commented 4 years ago
  1. Переписать в соответствии с БЭМ. В svg также использовать БЭМ, chart-text лучше тоже сделать частью svg. БЭМ
  2. Использовать модификатор. use modifier
  3. Здесь также использовать модификатор, шрифт задать в px, и символ № рядом сделать жирным. use modifiers1
  4. Не использовать селекторы тегов. tag selector
  5. Для внешнего позиционирования использовать элемент-обретку. Посмотреть вложенность селекторов по всему проекту. tag selectors
  6. Название шаблона не совпадает с названием блока two bem blocks template block-template name blk tmplt nm и т.д.
iatsdotfatr commented 4 years ago
  1. Если модифицируется элемент, то модификатор добавляется к элементу. modifier on element
  2. Несколько блоков. Не только в этом файле. two bem blocks
iatsdotfatr commented 4 years ago
  1. Использовать модификатор. modifier
  2. Много блоков. blocks
iatsdotfatr commented 4 years ago

Несоответствие имен naming