From ea0067d99976a572dcd4a4d988f4d33ca083f230 Mon Sep 17 00:00:00 2001 From: fragosti Date: Wed, 16 May 2018 17:06:57 -0700 Subject: Show error messages in the wrapped ether item --- .../ts/components/inputs/balance_bounded_input.tsx | 32 ++++++++++---------- .../ts/components/inputs/eth_amount_input.tsx | 2 ++ .../ts/components/inputs/token_amount_input.tsx | 2 ++ .../ts/components/wallet/wrap_ether_item.tsx | 34 +++++++++++++++++----- 4 files changed, 48 insertions(+), 22 deletions(-) (limited to 'packages/website/ts/components') diff --git a/packages/website/ts/components/inputs/balance_bounded_input.tsx b/packages/website/ts/components/inputs/balance_bounded_input.tsx index 68b77cfc3..72a246280 100644 --- a/packages/website/ts/components/inputs/balance_bounded_input.tsx +++ b/packages/website/ts/components/inputs/balance_bounded_input.tsx @@ -14,6 +14,7 @@ interface BalanceBoundedInputProps { amount?: BigNumber; hintText?: string; onChange: ValidatedBigNumberCallback; + onErrorMsgChange?: (errorMsg: React.ReactNode) => void; shouldShowIncompleteErrs?: boolean; shouldCheckBalance: boolean; validate?: (amount: BigNumber) => InputErrMsg; @@ -36,6 +37,7 @@ export class BalanceBoundedInput extends React.Component { - const isValid = _.isUndefined(errMsg); + const isValid = _.isUndefined(this._validate(amountString, this.props.balance)); if (utils.isNumeric(amountString) && !_.includes(amountString, '-')) { this.props.onChange(isValid, new BigNumber(amountString)); } else { @@ -161,4 +154,13 @@ export class BalanceBoundedInput extends React.Component void = utils.noop): void { + const errorMsg = this._validate(amount, balance); + this.props.onErrorMsgChange(errorMsg); + this.setState({ + amountString: amount, + errMsg: this._validate(amount, balance), + }, callback); + } } diff --git a/packages/website/ts/components/inputs/eth_amount_input.tsx b/packages/website/ts/components/inputs/eth_amount_input.tsx index c3822a80b..862ba25f8 100644 --- a/packages/website/ts/components/inputs/eth_amount_input.tsx +++ b/packages/website/ts/components/inputs/eth_amount_input.tsx @@ -12,6 +12,7 @@ interface EthAmountInputProps { amount?: BigNumber; hintText?: string; onChange: ValidatedBigNumberCallback; + onErrorMsgChange?: (errorMsg: React.ReactNode) => void; shouldShowIncompleteErrs: boolean; onVisitBalancesPageClick?: () => void; shouldCheckBalance: boolean; @@ -40,6 +41,7 @@ export class EthAmountInput extends React.Component void; onVisitBalancesPageClick?: () => void; lastForceTokenStateRefetch: number; shouldShowErrs?: boolean; @@ -85,6 +86,7 @@ export class TokenAmountInput extends React.Component { @@ -60,8 +66,8 @@ export class WrapEtherItem extends React.Component ) : ( )} + {this._renderErrorMsg()} } secondaryTextLines={2} @@ -119,7 +128,11 @@ export class WrapEtherItem extends React.Component { @@ -144,6 +157,13 @@ export class WrapEtherItem extends React.Component ); } + private _renderErrorMsg(): React.ReactNode { + return ( +
+ {this.state.errorMsg} +
+ ); + } private async _wrapEtherConfirmationActionAsync(): Promise { this.setState({ isEthConversionHappening: true, -- cgit v1.2.3 From e7fd501200f37581fa0b97854fc0914246d3e95c Mon Sep 17 00:00:00 2001 From: fragosti Date: Wed, 16 May 2018 17:12:17 -0700 Subject: Remove duplicate function call to validate --- packages/website/ts/components/inputs/balance_bounded_input.tsx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'packages/website/ts/components') diff --git a/packages/website/ts/components/inputs/balance_bounded_input.tsx b/packages/website/ts/components/inputs/balance_bounded_input.tsx index 72a246280..9a6ff64a7 100644 --- a/packages/website/ts/components/inputs/balance_bounded_input.tsx +++ b/packages/website/ts/components/inputs/balance_bounded_input.tsx @@ -160,7 +160,7 @@ export class BalanceBoundedInput extends React.Component Date: Thu, 17 May 2018 10:54:32 -0700 Subject: Refactor code in response to CR feedback --- .../ts/components/inputs/balance_bounded_input.tsx | 15 ++++++++------- .../website/ts/components/inputs/token_amount_input.tsx | 4 ++-- packages/website/ts/components/wallet/wrap_ether_item.tsx | 2 +- 3 files changed, 11 insertions(+), 10 deletions(-) (limited to 'packages/website/ts/components') diff --git a/packages/website/ts/components/inputs/balance_bounded_input.tsx b/packages/website/ts/components/inputs/balance_bounded_input.tsx index 9a6ff64a7..b90232e05 100644 --- a/packages/website/ts/components/inputs/balance_bounded_input.tsx +++ b/packages/website/ts/components/inputs/balance_bounded_input.tsx @@ -5,7 +5,7 @@ import TextField from 'material-ui/TextField'; import * as React from 'react'; import { Link } from 'react-router-dom'; import { RequiredLabel } from 'ts/components/ui/required_label'; -import { InputErrMsg, ValidatedBigNumberCallback, WebsitePaths } from 'ts/types'; +import { ValidatedBigNumberCallback, WebsitePaths } from 'ts/types'; import { utils } from 'ts/utils/utils'; interface BalanceBoundedInputProps { @@ -17,7 +17,7 @@ interface BalanceBoundedInputProps { onErrorMsgChange?: (errorMsg: React.ReactNode) => void; shouldShowIncompleteErrs?: boolean; shouldCheckBalance: boolean; - validate?: (amount: BigNumber) => InputErrMsg; + validate?: (amount: BigNumber) => React.ReactNode; onVisitBalancesPageClick?: () => void; shouldHideVisitBalancesLink?: boolean; isDisabled?: boolean; @@ -26,7 +26,7 @@ interface BalanceBoundedInputProps { } interface BalanceBoundedInputState { - errMsg: InputErrMsg; + errMsg: React.ReactNode; amountString: string; } @@ -37,7 +37,7 @@ export class BalanceBoundedInput extends React.Component { const isValid = _.isUndefined(this._validate(amountString, this.props.balance)); - if (utils.isNumeric(amountString) && !_.includes(amountString, '-')) { + const isPositiveNumber = utils.isNumeric(amountString) && !_.includes(amountString, '-'); + if (isPositiveNumber) { this.props.onChange(isValid, new BigNumber(amountString)); } else { this.props.onChange(isValid); @@ -114,7 +115,7 @@ export class BalanceBoundedInput extends React.Component void = utils.noop): void { + private _setAmountState(amount: string, balance: BigNumber, callback: () => void = _.noop): void { const errorMsg = this._validate(amount, balance); this.props.onErrorMsgChange(errorMsg); this.setState({ diff --git a/packages/website/ts/components/inputs/token_amount_input.tsx b/packages/website/ts/components/inputs/token_amount_input.tsx index 3ff2c5440..a5fd86f6c 100644 --- a/packages/website/ts/components/inputs/token_amount_input.tsx +++ b/packages/website/ts/components/inputs/token_amount_input.tsx @@ -6,7 +6,7 @@ import * as React from 'react'; import { Link } from 'react-router-dom'; import { Blockchain } from 'ts/blockchain'; import { BalanceBoundedInput } from 'ts/components/inputs/balance_bounded_input'; -import { InputErrMsg, Token, ValidatedBigNumberCallback, WebsitePaths } from 'ts/types'; +import { Token, ValidatedBigNumberCallback, WebsitePaths } from 'ts/types'; interface TokenAmountInputProps { userAddress: string; @@ -107,7 +107,7 @@ export class TokenAmountInput extends React.Component diff --git a/packages/website/ts/components/wallet/wrap_ether_item.tsx b/packages/website/ts/components/wallet/wrap_ether_item.tsx index c8ced8d08..0dff22feb 100644 --- a/packages/website/ts/components/wallet/wrap_ether_item.tsx +++ b/packages/website/ts/components/wallet/wrap_ether_item.tsx @@ -56,7 +56,7 @@ const styles: Styles = { errorMsg: { fontSize: 12, marginTop: 4, - color: 'red', + color: colors.red, minHeight: 20, }, }; -- cgit v1.2.3 From e9e570db4f158119dc86141201b695f52b3a5ca2 Mon Sep 17 00:00:00 2001 From: fragosti Date: Thu, 17 May 2018 12:00:00 -0700 Subject: Center all the things --- .../ts/components/inputs/balance_bounded_input.tsx | 4 ++++ .../ts/components/inputs/eth_amount_input.tsx | 9 ++++++++- .../ts/components/inputs/token_amount_input.tsx | 22 ++++++++++++++++------ .../ts/components/wallet/wrap_ether_item.tsx | 22 ++++++++++++++++++---- 4 files changed, 46 insertions(+), 11 deletions(-) (limited to 'packages/website/ts/components') diff --git a/packages/website/ts/components/inputs/balance_bounded_input.tsx b/packages/website/ts/components/inputs/balance_bounded_input.tsx index b90232e05..f91a99355 100644 --- a/packages/website/ts/components/inputs/balance_bounded_input.tsx +++ b/packages/website/ts/components/inputs/balance_bounded_input.tsx @@ -23,6 +23,8 @@ interface BalanceBoundedInputProps { isDisabled?: boolean; shouldShowErrs?: boolean; shouldShowUnderline?: boolean; + inputStyle?: React.CSSProperties; + inputHintStyle?: React.CSSProperties; } interface BalanceBoundedInputState { @@ -95,6 +97,8 @@ export class BalanceBoundedInput extends React.Component{this.props.hintText}} onChange={this._onValueChange.bind(this)} underlineStyle={{ width: 'calc(100% + 50px)' }} + inputStyle={this.props.inputStyle} + hintStyle={this.props.inputHintStyle} underlineShow={this.props.shouldShowUnderline} disabled={this.props.isDisabled} /> diff --git a/packages/website/ts/components/inputs/eth_amount_input.tsx b/packages/website/ts/components/inputs/eth_amount_input.tsx index 862ba25f8..fa684d85c 100644 --- a/packages/website/ts/components/inputs/eth_amount_input.tsx +++ b/packages/website/ts/components/inputs/eth_amount_input.tsx @@ -20,6 +20,8 @@ interface EthAmountInputProps { shouldShowErrs?: boolean; shouldShowUnderline?: boolean; style?: React.CSSProperties; + labelStyle?: React.CSSProperties; + inputHintStyle?: React.CSSProperties; } interface EthAmountInputState {} @@ -49,8 +51,10 @@ export class EthAmountInput extends React.Component -
ETH
+
ETH
); } @@ -60,4 +64,7 @@ export class EthAmountInput extends React.Component +
-
{this.props.token.symbol}
+
{this.props.token.symbol}
); } @@ -141,4 +141,14 @@ export class TokenAmountInput extends React.Component ) : ( @@ -108,7 +120,9 @@ export class WrapEtherItem extends React.Component )} -- cgit v1.2.3