diff options
Diffstat (limited to 'vnext/src/components/Icon.js')
-rw-r--r-- | vnext/src/components/Icon.js | 46 |
1 files changed, 21 insertions, 25 deletions
diff --git a/vnext/src/components/Icon.js b/vnext/src/components/Icon.js index 0a4c7dbc..faf1a704 100644 --- a/vnext/src/components/Icon.js +++ b/vnext/src/components/Icon.js @@ -1,39 +1,35 @@ import React from 'react'; import PropTypes from 'prop-types'; -export default class Icon extends React.Component { - constructor(props) { - super(props); - } +const Icon = React.memo(props => { + var size = props.size ? ' icon--' + props.size : ''; + var className = props.className ? ' ' + props.className : ''; + var klass = 'icon' + (!props.noFill ? ' icon--' + props.name : '') + size + className; - render() { - var size = this.props.size ? ' icon--' + this.props.size : ''; - var className = this.props.className ? ' ' + this.props.className : ''; - var klass = 'icon' + (!this.props.noFill ? ' icon--' + this.props.name : '') + size + className; + var name = '#' + props.name + '-icon'; + var useTag = '<use xlink:href=' + name + ' />'; + var Icon = React.createElement('svg', { className: 'icon__cnt', dangerouslySetInnerHTML: { __html: useTag } }); + return React.createElement( + 'div', + { className: klass }, + wrapSpinner(Icon, klass) + ); +}); - var name = '#' + this.props.name + '-icon'; - var useTag = '<use xlink:href=' + name + ' />'; - var Icon = React.createElement('svg', { className: 'icon__cnt', dangerouslySetInnerHTML: { __html: useTag } }); +function wrapSpinner(Html, klass) { + if (klass.indexOf('spinner') > -1) { return React.createElement( 'div', - { className: klass }, - this.wrapSpinner(Icon, klass) + { className: 'icon__spinner' }, + Html ); - } - - wrapSpinner(Html, klass) { - if (klass.indexOf('spinner') > -1) { - return React.createElement( - 'div', - { className: 'icon__spinner' }, - Html - ); - } else { - return Html; - } + } else { + return Html; } } +export default Icon; + Icon.propTypes = { size: PropTypes.string.isRequired, name: PropTypes.string.isRequired, |