Burize / FULLSTACK-Template-2

Набор компонентов для 2го макета
0 stars 0 forks source link

Message form #3

Open NikitaRzm opened 7 years ago

NikitaRzm commented 7 years ago

Этот компонент немного неудачно выделен. Имеет смысл либо вынести drop-down, hint, inputs в отдельные компоненты и в index.pug собрать нужную форму из простейших элементов, или же сразу собрать конкретную имплементацию MessageForm внутри выделенного Вами компонента. Сейчас подразумевается, что эти компоненты могут быть использованы для созания MessageForm, но где, как и когда ее создавать - абсолютно не ясно. При описанном мной подходе конкретная имплементация уже будет готова к использованию с учетом всех специфик. Если же потребуется дополнительное представление для другого контекста, можно сздать внутри этого же компонента другое представление из тех же или измененных с учетом специфики простейших компонентов. Вам выбирать.

Если вы группируете простейшие компоненты по признаку того, что из них будет построено, будет бардак. Если у вас сильное желание это осуществить, можно вынести это поближе к index.pug и рядом положить все компоненты, которые будут собираться конкретно для страницы index.pug из других.

Burize commented 7 years ago

"Имеет смысл либо вынести drop-down, hint, inputs в отдельные компоненты и в index.pug собрать нужную форму из простейших элементов" - Изначально так и было, но потом все же решил объединить все в один элемент. У всех элементов формы в макете есть общий стиль + так у всех элементов(инпутов) будет общий класс form__control. А если разделять то после изменений в одном элементе придется тестировать все другие.

"или же сразу собрать конкретную имплементацию MessageForm внутри выделенного Вами компонента." - Надеюсь правильно понял, что имелось ввиду. Добавил внутрь компонента папку Forms, в которой находятся уже конкретные формы: MessageForm и SearchForm, собранные из сделанных компонентов; и в index подключаются уже не миксины, а сами формы.