pug about hotel HOT 7 CLOSED

В JavaScript для случаев, когда названия ключа объекта совпадает и передаваемой переменной совпадают, следует использовать краткую запись. Вместо:

будет:

+button-check({ labels })

from hotel.

3. То же самое в button-like, от условия можно избавиться:

mixin button-like({ isLiked, likeNumber })
  .button.button-like.js-button-like(class = isLiked && 'button-like_is-liked')
    .button-like__heart.js-button-like__heart= isLiked ? 'favorite' : 'favorite_border'
    .button-like__number.js-button-like__number= likeNumber

В примере выше я заодно скомбинировал синтаксис задания классов (через точку и атрибут) и заменил ternary, где третий операнд - пустая строка логческим И.

И ещё один трик: в pug не обязательно начинать каждую строчку JavaScript-кода с -, можно добавить после первого "минуса" перенос строки и отступ:

-
  const one = 1;
  const two = 2;

from hotel.

Нашёл ещё var-ы:

from hotel.

Тут наверное button должен быть вместо a:

if variant==='arrow'
  a(class=buttonClassName type=purpose)

from hotel.

Тут явно можно убрать лишние обёртки (обе), тем более для display: flex можно задать gap и margin уже не нужен:

.header-bar__buttons
  .buttons-header-bar
    .buttons-header-bar__first-button
      +button
    +button

from hotel.

Интересное решение, #{tagHTML}, я таким не пользовался.

Только зачем задавать промежуточную переменную tagHTML=tag==='a'?'a':'button'?

И ещё такая проблемка: атрибут href не валиден для кнопок, а type - для ссылок. Следует либо разделить на кнопку и ссылку, либо в зависимости от тэга и атрибуты задавать.

Также было бы неплохо добавить пробелов в некоторые места, плюс список деструктурируемых параметров аргумента длинноват, не очень удобно читать - можно деструктурирование перенести в тело миксина:

mixin button(options)
  - const { ... } = options;

from hotel.

span.button-icon__empty-elem - а можно этот заменить на padding, margin или псевдоэлемент?

from hotel.