Skip to content
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion packages/examples/packages/send-flow/snap.manifest.json
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
"url": "https://github.com/MetaMask/snaps.git"
},
"source": {
"shasum": "FFP7+q2qlyOMqmtm01zXewHHczyLyTsgbFYpkoSkw7s=",
"shasum": "9YAT2soLTWY6yInzFfNkwCY5aD6+OTv9flf5fflyJyw=",
"location": {
"npm": {
"filePath": "dist/bundle.js",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,6 @@ import type { Account, Currency } from '../types';
* @property selectedCurrency - The selected currency to display.
* @property total - The total cost of the transaction.
* @property fees - The fees for the transaction.
* @property displayClearIcon - Whether to display the clear icon or not.
* @property flushToAddress - Whether to flush the address field or not.
* @property errors - The form errors.
*/
Expand All @@ -25,7 +24,6 @@ export type SendFlowProps = {
selectedCurrency: 'BTC' | '$';
total: Currency;
fees: Currency;
displayClearIcon: boolean;
flushToAddress?: boolean;
errors?: {
amount?: string;
Expand All @@ -43,7 +41,6 @@ export type SendFlowProps = {
* @param props.total - The total cost of the transaction.
* @param props.errors - The form errors.
* @param props.fees - The fees for the transaction.
* @param props.displayClearIcon - Whether to display the clear icon or not.
* @param props.flushToAddress - Whether to flush the address field or not.
* @returns The SendFlow component.
*/
Expand All @@ -53,7 +50,6 @@ export const SendFlow: SnapComponent<SendFlowProps> = ({
selectedCurrency,
total,
fees,
displayClearIcon,
flushToAddress,
errors,
}) => {
Expand All @@ -66,7 +62,6 @@ export const SendFlow: SnapComponent<SendFlowProps> = ({
accounts={accounts}
selectedCurrency={selectedCurrency}
flushToAddress={flushToAddress}
displayClearIcon={displayClearIcon}
errors={errors}
/>
<TransactionSummary fees={fees} total={total} />
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,13 +6,13 @@ import {
Icon,
Image,
Input,
AddressInput,
Text,
type SnapComponent,
} from '@metamask/snaps-sdk/jsx';

import { AccountSelector } from './AccountSelector';
import btcIcon from '../images/btc.svg';
import jazzicon3 from '../images/jazzicon3.svg';
import type { Account, SendFormErrors } from '../types';

/**
Expand All @@ -22,15 +22,13 @@ import type { Account, SendFormErrors } from '../types';
* @property accounts - The available accounts.
* @property errors - The form errors.
* @property selectedCurrency - The selected currency to display.
* @property displayClearIcon - Whether to display the clear icon or not.
* @property flushToAddress - Whether to flush the address field or not.
*/
export type SendFormProps = {
selectedAccount: string;
accounts: Account[];
errors?: SendFormErrors;
selectedCurrency: 'BTC' | '$';
displayClearIcon: boolean;
flushToAddress?: boolean;
};

Expand All @@ -42,7 +40,6 @@ export type SendFormProps = {
* @param props.accounts - The available accounts.
* @param props.errors - The form errors.
* @param props.selectedCurrency - The selected currency to display.
* @param props.displayClearIcon - Whether to display the clear icon or not.
* @param props.flushToAddress - Whether to flush the address field or not.
* @returns The SendForm component.
*/
Expand All @@ -51,7 +48,6 @@ export const SendForm: SnapComponent<SendFormProps> = ({
accounts,
errors,
selectedCurrency,
displayClearIcon,
flushToAddress,
}) => (
<Form name="sendForm">
Expand All @@ -69,21 +65,12 @@ export const SendForm: SnapComponent<SendFormProps> = ({
</Box>
</Field>
<Field label="To account" error={errors?.to}>
<Box>
<Image src={jazzicon3} />
</Box>
<Input
<AddressInput
name="to"
chainId="eip155:0"
placeholder="Enter receiving address"
value={flushToAddress ? '' : undefined}
/>
{displayClearIcon && (
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why did you remove this?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The AddressInput component has a clear icon built-in.

<Box>
<Button name="clear">
<Icon name="close" color="primary" />
</Button>
</Box>
)}
</Field>
</Form>
);
11 changes: 0 additions & 11 deletions packages/examples/packages/send-flow/src/images/jazzicon3.svg

This file was deleted.

2 changes: 0 additions & 2 deletions packages/examples/packages/send-flow/src/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -111,7 +111,6 @@ export const onUserInput: OnUserInputHandler = async ({
selectedCurrency={selectedCurrency}
total={total}
fees={fees}
displayClearIcon={Boolean(sendForm.to) && sendForm.to !== ''}
errors={formErrors}
/>
),
Expand All @@ -138,7 +137,6 @@ export const onUserInput: OnUserInputHandler = async ({
total={total}
fees={fees}
flushToAddress={true}
displayClearIcon={false}
errors={formErrors}
/>
),
Expand Down
1 change: 0 additions & 1 deletion packages/examples/packages/send-flow/src/utils.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,6 @@ export async function generateSendFlow({
selectedCurrency="BTC"
total={{ amount: 0, fiat: 0 }}
fees={fees}
displayClearIcon={false}
/>
),
context: {
Expand Down
Loading