Convert QrCodeView to use JSX (#7504)

MetaMask · Nov 23, 2019 · c775a84 · c775a84
1 parent 7d097de
commit c775a84
Showing 1 changed file with 52 additions and 40 deletions.
diff --git a/ui/app/components/ui/qr-code.js b/ui/app/components/ui/qr-code.js
@@ -1,7 +1,6 @@
-const Component = require('react').Component
-const h = require('react-hyperscript')
+import PropTypes from 'prop-types'
+import React from 'react'
 const qrCode = require('qrcode-generator')
-const inherits = require('util').inherits
 const connect = require('react-redux').connect
 const { isHexPrefixed } = require('ethereumjs-util')
 const ReadOnlyInput = require('./readonly-input')
@@ -17,47 +16,60 @@ function mapStateToProps (state) {
   }
 }
 
-inherits(QrCodeView, Component)
-
-function QrCodeView () {
-  Component.call(this)
-}
-
-QrCodeView.prototype.render = function () {
-  const props = this.props
-  const { message, data, network } = props.Qr
-  const address = `${isHexPrefixed(data) ? 'ethereum:' : ''}${checksumAddress(data, network)}`
+function QrCodeView (props) {
+  const { message, data } = props.Qr
+  const address = `${isHexPrefixed(data) ? 'ethereum:' : ''}${checksumAddress(data)}`
   const qrImage = qrCode(4, 'M')
   qrImage.addData(address)
   qrImage.make()
 
-  return h('.div.flex-column.flex-center', [
-    Array.isArray(message)
-      ? h('.message-container', this.renderMultiMessage())
-      : message && h('.qr-header', message),
-
-    this.props.warning ? this.props.warning && h('span.error.flex-center', {
-      style: {
-      },
-    },
-    this.props.warning) : null,
-
-    h('.div.qr-wrapper', {
-      style: {},
-      dangerouslySetInnerHTML: {
-        __html: qrImage.createTableTag(4),
-      },
-    }),
-    h(ReadOnlyInput, {
-      wrapperClass: 'ellip-address-wrapper',
-      inputClass: 'qr-ellip-address',
-      value: checksumAddress(data, network),
-    }),
-  ])
+  return (
+    <div className="div flex-column flex-center">
+      {
+        Array.isArray(message)
+          ? (
+            <div className="message-container">
+              {props.Qr.message.map((message, index) => (
+                <div className="qr-message" key={index}>
+                  {message}
+                </div>
+              ))}
+            </div>
+          )
+          : message && (
+            <div className="qr-header">
+              {message}
+            </div>
+          )
+      }
+      {
+        props.warning
+          ? (props.warning && (
+            <span className="error flex-center">
+              {props.warning}
+            </span>
+          ))
+          : null
+      }
+      <div
+        className="div qr-wrapper"
+        dangerouslySetInnerHTML={{
+          __html: qrImage.createTableTag(4),
+        }}
+      />
+      <ReadOnlyInput
+        wrapperClass="ellip-address-wrapper"
+        inputClass="qr-ellip-address"
+        value={checksumAddress(data)}
+      />
+    </div>
+  )
 }
 
-QrCodeView.prototype.renderMultiMessage = function () {
-  var Qr = this.props.Qr
-  var multiMessage = Qr.message.map((message) => h('.qr-message', message))
-  return multiMessage
+QrCodeView.propTypes = {
+  warning: PropTypes.node,
+  Qr: PropTypes.shape({
+    message: PropTypes.array,
+    data: PropTypes.string.isRequired,
+  }).isRequired,
 }