-
Notifications
You must be signed in to change notification settings - Fork 109
feat(popup-header): add popup-header component #174
Conversation
@@ -0,0 +1,10 @@ | |||
``` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Я думаю нужно показывать в сборке с popup-ом.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Заменить или дополнить?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Я бы добавил.
src/popup-header/popup-header.jsx
Outdated
className={ cn('closer') } | ||
size={ this.props.size } | ||
icon='close' | ||
onClick={ this.props.onCloseClick } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ноу. через handler и проверку есть или нет внешнего пропса, иначе будет ошибка, если обработчик непередан.
src/popup-header/README.md
Outdated
) } | ||
visible={ this.state.popup } | ||
> | ||
<div>Lorem ipsum dolor sit amet, consectetur adipiscing elit,</div> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Сделай пожалуйста осмысленную демку на русском языке. Запрос от дизайнера
75a3a50
to
e50edce
Compare
Компонент заголовка для попапа.
Мотивация и контекст
Выносим типовой хедер на уровень компонента для переиспользования в типичных полноэкранных попапах — select, calendar-input и т.п.