mpreterer / Hotel-Toxin

0 stars 0 forks source link

SCSS #10

Closed metanonum closed 2 years ago

metanonum commented 2 years ago
  1. К js- именам мы обращаемся только из .js и только для поиска элемента
    Скриншот 22-08-2022 010318
  2. Использование селекторов допускается, обращение к тегам - нет. Там же есть класс .nav-burger__link
    Скриншот 22-08-2022 010455
  3. Вероятно это должно быть модификаторами, но не отдельными блоками внутри button-like
    Скриншот 22-08-2022 011919
  4. Медиа-запрос лучше размешать внутри скобок блока или элемента к которому относится запрос, после всех прочих правил этого блока или элемента.
    Скриншот 22-08-2022 012514
  5. Лучше использовать сокращения только для элементов и модификаторов (&__element, &_modifier) Скриншот 22-08-2022 012758
metanonum commented 2 years ago
  1. Все отрицательные margin нужно убрать Скриншот 24-08-2022 191118
  2. Лучше не добавлять внешнее позиционирование блокам. Скриншот 24-08-2022 191657 Здесь нужно убрать margin-right и позиционировать блок в том месте где он используется
  3. Отображение других блоков нужно настраивать через опции pug-миксина Скриншот 31-08-2022 221042
metanonum commented 2 years ago

10.7

Скриншот 12-09-2022 170115 Внешнее позиционирование блока

metanonum commented 2 years ago
  1. Нельзя просто так использовать !important Скриншот 12-09-2022 180725 И еще, я не обнаружил где используется элемент __room-order. Там где он ожидается, используется __room-order-container. Проверь чтобы нигде не было мертвого кода.
metanonum commented 2 years ago
  1. Относительно какого родителя идет позиционирование? Лучше не использовать абсолютное позиционирование. Скриншот 16-09-2022 164034