crowetic/protocol
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

chore: add clarification to _generateSelectAssetClickHandler

Browse Source
This commit is contained in:
fragosti
2018-11-01 13:37:43 -07:00
parent 7fc1a88680
commit 0a3af4eb22
1 changed files with 10 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

19
packages/instant/src/components/erc20_asset_amount_input.tsx
View File

@@ -122,18 +122,19 @@ export class ERC20AssetAmountInput extends React.Component<ERC20AssetAmountInput
currentFontSizePx: fontSizePx,
});
};
// We don't want to allow opening the token selection panel if there are no assets.
private readonly _generateSelectAssetClickHandler = (): (() => void) | undefined => {
const { numberOfAssetsAvailable } = this.props;
if (_.isUndefined(numberOfAssetsAvailable) || numberOfAssetsAvailable <= 1) {
// We don't want to allow opening the token selection panel if there are no assets.
// Since styles are inferred from the presence of a click handler, we want to return undefined
// instead of providing a noop.
const { numberOfAssetsAvailable, onSelectAssetClick } = this.props;
if (
_.isUndefined(numberOfAssetsAvailable) ||
numberOfAssetsAvailable <= 1 ||
_.isUndefined(onSelectAssetClick)
) {
return undefined;
}
return this._handleSelectAssetClick;
};
private readonly _handleSelectAssetClick = () => {
if (this.props.onSelectAssetClick) {
this.props.onSelectAssetClick(this.props.asset);
}
return () => onSelectAssetClick(this.props.asset);
};
// For assets with symbols of different length,
// start scaling the input at different character lengths