FIx double divider border in 'DeviceMenu'

pull/3/head
Vasek Mlejnsky 6 years ago
parent 27e3385b7d
commit fed2c3031a

@ -52,6 +52,8 @@ class CoinMenu extends Component {
<Divider <Divider
textLeft="Other coins" textLeft="Other coins"
textRight="(You will be redirected)" textRight="(You will be redirected)"
borderTop
borderBottom
/> />
{coins.map(coin => ( {coins.map(coin => (
<a key={coin.url} href={coin.url}> <a key={coin.url} href={coin.url}>

@ -6,7 +6,7 @@ import type { TrezorDevice } from 'flowtype';
import type { Props } from '../common'; import type { Props } from '../common';
import AsideDivider from '../Divider'; import Divider from '../Divider';
export const DeviceSelect = (props: Props) => { export const DeviceSelect = (props: Props) => {
const { devices } = props; const { devices } = props;
@ -214,7 +214,7 @@ export class DeviceDropdown extends Component<Props> {
return ( return (
<React.Fragment> <React.Fragment>
{currentDeviceMenu} {currentDeviceMenu}
{deviceList.length > 1 ? <AsideDivider textLeft="Other devices" /> : null} {deviceList.length > 1 ? <Divider textLeft="Other devices" borderBottom /> : null}
{deviceList} {deviceList}
{webUsbButton} {webUsbButton}
</React.Fragment> </React.Fragment>

@ -1,4 +1,4 @@
import styled from 'styled-components'; import styled, { css } from 'styled-components';
import React from 'react'; import React from 'react';
import PropTypes from 'prop-types'; import PropTypes from 'prop-types';
@ -12,12 +12,21 @@ const Wrapper = styled.div`
font-size: ${FONT_SIZE.SMALLER}; font-size: ${FONT_SIZE.SMALLER};
color: ${colors.TEXT_SECONDARY}; color: ${colors.TEXT_SECONDARY};
background: ${colors.GRAY_LIGHT}; background: ${colors.GRAY_LIGHT};
border-top: 1px solid ${colors.DIVIDER}; ${props => props.borderTop && css`
border-bottom: 1px solid ${colors.DIVIDER}; border-top: 1px solid ${colors.DIVIDER};
`}
${props => props.borderBottom && css`
border-bottom: 1px solid ${colors.DIVIDER};
`}
`; `;
const Divider = ({ textLeft, textRight }) => ( const Divider = ({
<Wrapper> textLeft, textRight, borderTop, borderBottom,
}) => (
<Wrapper
borderTop={borderTop}
borderBottom={borderBottom}
>
<p>{textLeft}</p> <p>{textLeft}</p>
<p>{textRight}</p> <p>{textRight}</p>
</Wrapper> </Wrapper>
@ -26,6 +35,8 @@ const Divider = ({ textLeft, textRight }) => (
Divider.propTypes = { Divider.propTypes = {
textLeft: PropTypes.string, textLeft: PropTypes.string,
textRight: PropTypes.string, textRight: PropTypes.string,
borderTop: PropTypes.bool.isRequired,
borderBottom: PropTypes.bool.isRequired,
}; };
export default Divider; export default Divider;

Loading…
Cancel
Save