Make sure that token list/settings tabs don't jump when switching

This commit is contained in:
Mikhail Mikheev 2019-09-06 17:46:06 +04:00
parent 0337d61ab8
commit 574b207ebd
5 changed files with 25 additions and 30 deletions

View File

@ -131,10 +131,12 @@ class Balances extends React.Component<Props, State> {
color="secondary" color="secondary"
disableRipple disableRipple
/> />
<Paragraph className={classes.zero}>Hide zero balances</Paragraph> <Paragraph size="lg">Hide zero balances</Paragraph>
</Col> </Col>
<Col xs={6} end="sm"> <Col xs={6} end="sm">
<ButtonLink onClick={this.onShow('Token')} testId="manage-tokens-btn">Manage Tokens</ButtonLink> <ButtonLink size="lg" onClick={this.onShow('Token')} testId="manage-tokens-btn">
Manage Tokens
</ButtonLink>
<Modal <Modal
title="Manage Tokens" title="Manage Tokens"
description="Enable and disable tokens to be listed" description="Enable and disable tokens to be listed"

View File

@ -6,9 +6,6 @@ export const styles = (theme: Object) => ({
width: '20px', width: '20px',
marginRight: sm, marginRight: sm,
}, },
zero: {
letterSpacing: '-0.5px',
},
message: { message: {
margin: `${sm} 0`, margin: `${sm} 0`,
}, },

View File

@ -44,7 +44,9 @@ export const sendReplaceOwner = async (
) => { ) => {
const gnosisSafe = await getGnosisSafeInstanceAt(safeAddress) const gnosisSafe = await getGnosisSafeInstanceAt(safeAddress)
const safeOwners = await gnosisSafe.getOwners() const safeOwners = await gnosisSafe.getOwners()
const index = safeOwners.findIndex(ownerAddress => ownerAddress.toLowerCase() === ownerAddressToRemove.toLowerCase()) const index = safeOwners.findIndex(
(ownerAddress) => ownerAddress.toLowerCase() === ownerAddressToRemove.toLowerCase(),
)
const prevAddress = index === 0 ? SENTINEL_ADDRESS : safeOwners[index - 1] const prevAddress = index === 0 ? SENTINEL_ADDRESS : safeOwners[index - 1]
const txData = gnosisSafe.contract.methods const txData = gnosisSafe.contract.methods
.swapOwner(prevAddress, ownerAddressToRemove, values.ownerAddress) .swapOwner(prevAddress, ownerAddressToRemove, values.ownerAddress)
@ -97,7 +99,7 @@ const ReplaceOwner = ({
} }
return ( return (
<React.Fragment> <>
<SharedSnackbarConsumer> <SharedSnackbarConsumer>
{({ openSnackbar }) => { {({ openSnackbar }) => {
const onReplaceOwner = () => { const onReplaceOwner = () => {
@ -127,7 +129,7 @@ const ReplaceOwner = ({
open={isOpen} open={isOpen}
paperClassName={classes.biggerModalWindow} paperClassName={classes.biggerModalWindow}
> >
<React.Fragment> <>
{activeScreen === 'checkOwner' && ( {activeScreen === 'checkOwner' && (
<OwnerForm <OwnerForm
onClose={onClose} onClose={onClose}
@ -152,12 +154,12 @@ const ReplaceOwner = ({
threshold={threshold} threshold={threshold}
/> />
)} )}
</React.Fragment> </>
</Modal> </Modal>
) )
}} }}
</SharedSnackbarConsumer> </SharedSnackbarConsumer>
</React.Fragment> </>
) )
} }

View File

@ -91,14 +91,8 @@ class Settings extends React.Component<Props, State> {
return ( return (
<> <>
<Row align="center" className={classes.message}> <Row className={classes.message}>
<Col xs={6}> <Paragraph size="lg" color="error" onClick={this.onShow('RemoveSafe')}>
<Paragraph className={classes.settings} size="lg" weight="bolder">
Settings
</Paragraph>
</Col>
<Col xs={6} end="sm">
<Paragraph noMargin size="lg" color="error" onClick={this.onShow('RemoveSafe')}>
<Span className={cn(classes.links, classes.removeSafeText)}>Remove Safe</Span> <Span className={cn(classes.links, classes.removeSafeText)}>Remove Safe</Span>
<Img alt="Trash Icon" className={classes.removeSafeIcon} src={RemoveSafeIcon} /> <Img alt="Trash Icon" className={classes.removeSafeIcon} src={RemoveSafeIcon} />
</Paragraph> </Paragraph>
@ -109,7 +103,6 @@ class Settings extends React.Component<Props, State> {
safeAddress={safeAddress} safeAddress={safeAddress}
safeName={safeName} safeName={safeName}
/> />
</Col>
</Row> </Row>
<Block className={classes.root}> <Block className={classes.root}>
<Col xs={3} layout="column"> <Col xs={3} layout="column">

View File

@ -38,6 +38,7 @@ export const styles = () => ({
}, },
message: { message: {
margin: `${sm} 0`, margin: `${sm} 0`,
justifyContent: 'flex-end',
}, },
links: { links: {
textDecoration: 'underline', textDecoration: 'underline',