Gui 74 deposit withdraw#86
Conversation
# Conflicts: # front/src/app/router/routes.tsx # front/src/app/stores/types.ts
| } | ||
|
|
||
| export interface ISendTransactionResult extends ISendTransaction { | ||
| action?: string; |
|
|
||
| &__withdraw-button { | ||
| margin-left: 10px; | ||
| font-size: 12px !important; |
There was a problem hiding this comment.
у кнопки есть props fontSize
| public render() { | ||
| const { autoFocus, className, allowAutoComplete, ...rest } = this.props; | ||
| const autoComplete = allowAutoComplete ? 'on' : 'off'; | ||
| const underline = this.props.readOnly ? ( |
There was a problem hiding this comment.
есть подозрения что лучше это сделать через css
типа добавить модификатор sonm-input--readonly к root element
| &--deposit, | ||
| &--withdraw { | ||
| -webkit-mask: url(./icon-arrow-right.svg) no-repeat center; | ||
| -webkit-mask-size: 70%; |
There was a problem hiding this comment.
это что-то нестандартное( как же firefox?) и выглядит непонятным. это надо как -то проще сделать
| } | ||
|
|
||
| &-dw-amount { | ||
| width: 100px; |
There was a problem hiding this comment.
этот класс у сразу нескольких колонок, которые не dw-amount
| className="sonm-tx-list__cell-amount-amount" | ||
| key="2" | ||
| symbol={currency.symbol} | ||
| balance={'0'} |
There was a problem hiding this comment.
Тут будет комиссия, пока хз как она планируется считаться
| title: 'Status', | ||
| className: 'sonm-tx-list__cell-status sonm-tx-list__cell', | ||
| render: (_, record) => { | ||
| const cls = `sonm-tx-list__cell-status--${record.status}`; |
There was a problem hiding this comment.
я бы добавил String(record.status).toLowerCase на всякий случай
| const cls = `sonm-tx-list__cell-status--${record.status}`; | ||
|
|
||
| return ( | ||
| <div className={cn('sonm-tx-list__cell-status', cls)}> |
There was a problem hiding this comment.
тут(и много где) одинаковый класс у внешнего td и его содержимого.
так быть не должно.
содержимое ячейки может быть либо отдельным блоком, либо как минимум отдельным элементом.
<td className="sonm-dw-list__cell-status sonm-dw-list__cell">
<div className"sonm-dw-status-cell"> // это изолированный блок
<div className"sonm-dw-status-cell__title">{title}</div> // это его элемент
</div>
</td>
sonm-dw-list__cell - это общий класс для всех ячеек. может задавать background падинги и что-то еще
sonm-dw-list__cell-status - это елемент sonm-dw-list для конкретного столбца. тут можно задать ширину столбца например
его кстати можно назвать sonm-dw-list__cell--status - тогда "status" будет модификатором ячейки. (и он тоже будет ширину столбца менять)
sonm-dw-status-cell - уже описывает содержимое ячейки
| query: '', | ||
| }; | ||
|
|
||
| public componentWillMount() { |
There was a problem hiding this comment.
- c 16.3 этот метод deprecated
- он ничего не делает здесь))
There was a problem hiding this comment.
Мигрировал с обычной history ))
| grid-template-rows: 30px 100px 1fr 80px; | ||
| grid-column-gap: 50px; | ||
|
|
||
| &__confirmation { |
There was a problem hiding this comment.
помоему этот элемент не этого блока. но надо проверить.
если все таки этого, то это модификатор. .sonm-deposit-withdraw--confirmation
# Conflicts: # front/src/app/components/common/index.less # front/src/app/components/layouts/profile/sub/details/index.tsx # front/src/app/router/routes.tsx
2 participants
No description provided.