Код

[kate-fsd]

На примере text-field

1. Все смешалось в кучу, логика, верстка. Лучше сначала по возможности получить все данные, а верстку оставить чистой
2. Копипаст
3. Нужно добавлять вертикальные отступы, код тяжело читается

4.   let buttonClassName = `${className}__button`;
     let buttonClassNames = `${buttonClassName} js-${buttonClassName}`;

Подобное использование переменной className тоже как то тяжело читается, хотя с точки зрения требовании к коду все верно.

Есть идеи как это упростить:

• вообще не вставлять className, не ${className}__button, а text-field__button
• поменять имя. className - не совсем подходит, не понятно что за имя класса. А это блок. Можно заменить на block-class или просто block
• написать или найти функцию для создания БЭМ классов (вроде classnames).

4 пункт - на твое усмотрение

5. Там где переменная не меняется использовать const

[kate-fsd]

Здесь тоже. Не принято так выносить имена классов. Да и смысла в этом нет, количество кода не сокращается. А читаемость ухудшается

[kate-fsd]

bullet можно не выносить в отдельный компонент, потому что отдельный пункт не может существовать в отрыве от списка

[kate-fsd]

Не стоит делать стрелку картинкой. Это просто стилизация

[kate-fsd]

Когда оборачиваешь input label-ом можно не добавлять for

[kate-fsd]

Не стоит делать стрелку картинкой. Это просто стилизация

в checkbox тоже

[kate-fsd]

Автатарки не должны лежать в папке comment, так как они не являются частью этого компонента

[kate-fsd]

dropdown__dropdown-summary второй дропдаун можно убрать

[kate-fsd]

Автатарки не должны лежать в папке comment, так как они не являются частью этого компонента

Фото комнат тоже