Skip to content

Commit effaf58

Browse files
Fuma419dhanz
andauthored
fix: confirm tx modal — address text black on yellow bg, hide zero fee (#74)
- Address text explicitly set to color black (was inheriting parent color, invisible on yellow background in dark mode). - Fee row hidden when fee is '0' or empty (before tx is built), so the modal doesn't show a misleading "0 fee" or "..." placeholder. Co-authored-by: dhanz <dhanz@local>
1 parent ff3c437 commit effaf58

1 file changed

Lines changed: 15 additions & 13 deletions

File tree

src/ui/app/pages/send.jsx

Lines changed: 15 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -1054,10 +1054,10 @@ const Send = () => {
10541054
<Box
10551055
position={'relative'}
10561056
background={background}
1057+
color="black"
10571058
rounded={'xl'}
10581059
p={2}
10591060
>
1060-
{' '}
10611061
<Copy label="Copied address" copy={address.result}>
10621062
<Box
10631063
width="180px"
@@ -1070,24 +1070,26 @@ const Send = () => {
10701070
flexDirection={'column'}
10711071
>
10721072
<MiddleEllipsis>
1073-
<span style={{ cursor: 'pointer' }}>{address.result}</span>
1073+
<span style={{ cursor: 'pointer', color: 'black' }}>{address.result}</span>
10741074
</MiddleEllipsis>
10751075
</Box>
10761076
</Copy>
10771077
</Box>
10781078
<Box h={4} />
1079-
<Box
1080-
width={'full'}
1081-
display={'flex'}
1082-
alignItems={'center'}
1083-
justifyContent={'center'}
1084-
fontSize={'sm'}
1085-
>
1086-
<UnitDisplay quantity={fee.fee} decimals={6} symbol={'₳'} />{' '}
1087-
<Box ml={1} fontWeight={'medium'}>
1088-
fee
1079+
{fee.fee && fee.fee !== '0' && (
1080+
<Box
1081+
width={'full'}
1082+
display={'flex'}
1083+
alignItems={'center'}
1084+
justifyContent={'center'}
1085+
fontSize={'sm'}
1086+
>
1087+
<UnitDisplay quantity={fee.fee} decimals={6} symbol={'₳'} />{' '}
1088+
<Box ml={1} fontWeight={'medium'}>
1089+
fee
1090+
</Box>
10891091
</Box>
1090-
</Box>
1092+
)}
10911093
{address.isM1 && (
10921094
<>
10931095
<Box h={4} />

0 commit comments

Comments
 (0)