From f57d10d8570cf9409d22b328276acd8a358dbce2 Mon Sep 17 00:00:00 2001 From: David Walsh Date: Wed, 22 Feb 2023 11:13:48 -0600 Subject: [PATCH] UX: Icons: Remove unused overview icons (#17671) * UX: Icons: Remove unused overview icons * Add args --- .storybook/2.DOCUMENTATION.stories.mdx | 6 +-- ui/components/ui/button/button.stories.js | 11 +++-- ui/components/ui/icon/README.mdx | 16 ------- .../ui/icon/overview-send-icon.component.js | 43 ------------------- 4 files changed, 8 insertions(+), 68 deletions(-) delete mode 100644 ui/components/ui/icon/overview-send-icon.component.js diff --git a/.storybook/2.DOCUMENTATION.stories.mdx b/.storybook/2.DOCUMENTATION.stories.mdx index 9552b2c5b..21f0d29d7 100644 --- a/.storybook/2.DOCUMENTATION.stories.mdx +++ b/.storybook/2.DOCUMENTATION.stories.mdx @@ -48,7 +48,7 @@ The example below displays the `Button` component and it explains how we should import React from 'react'; -import BuyIcon from '../icon/overview-buy-icon.component'; +import IconTokenSearch from '../icon/icon-token-search'; // The mdx file to document props and usage import README from './README.mdx'; @@ -92,9 +92,9 @@ export default { control: { type: 'select', }, - options: ['BuyIcon'], + options: ['IconTokenSearch'], mapping: { - BuyIcon: , + IconTokenSearch: , }, }, }, diff --git a/ui/components/ui/button/button.stories.js b/ui/components/ui/button/button.stories.js index 1bd4fa05b..9772b8ed3 100644 --- a/ui/components/ui/button/button.stories.js +++ b/ui/components/ui/button/button.stories.js @@ -1,6 +1,6 @@ import React from 'react'; -import BuyIcon from '../icon/overview-buy-icon.component'; +import IconTokenSearch from '../icon/icon-token-search'; import README from './README.mdx'; import Button from '.'; @@ -36,9 +36,9 @@ export default { control: { type: 'select', }, - options: ['BuyIcon'], + options: ['IconTokenSearch'], mapping: { - BuyIcon: , + IconTokenSearch: , }, }, submit: { control: 'boolean' }, @@ -132,11 +132,10 @@ TypeInline.args = { }; export const Icon = (args) => ; - Icon.args = { type: 'primary', - icon: , - children: 'Buy', + icon: , + children: 'Search', }; export const Submit = (args) => ( diff --git a/ui/components/ui/icon/README.mdx b/ui/components/ui/icon/README.mdx index 4eee1c29b..b9c654992 100644 --- a/ui/components/ui/icon/README.mdx +++ b/ui/components/ui/icon/README.mdx @@ -146,22 +146,6 @@ Use the `className` prop to add an additional class to the icon. This additional -## BuyIcon - - - - - - - -## SwapIcon - - - - - - - ## Send/SwapIcon diff --git a/ui/components/ui/icon/overview-send-icon.component.js b/ui/components/ui/icon/overview-send-icon.component.js deleted file mode 100644 index b86fac387..000000000 --- a/ui/components/ui/icon/overview-send-icon.component.js +++ /dev/null @@ -1,43 +0,0 @@ -import React from 'react'; -import PropTypes from 'prop-types'; - -/** - * @deprecated This has been deprecated in favour of the `` component in ./ui/components/component-library/icon/icon.js - * See storybook documentation for Icon here https://metamask.github.io/metamask-storybook/?path=/docs/components-componentlibrary-icon--default-story#icon - */ - -export default function SwapIcon({ - width = '15', - height = '15', - fill = 'white', -}) { - return ( - - - - ); -} - -SwapIcon.propTypes = { - /** - * Width of the icon - */ - width: PropTypes.string, - /** - * Height of the icon - */ - height: PropTypes.string, - /** - * Fill of the icon should be a valid design system color - */ - fill: PropTypes.string, -};