Fix signature-request for new Storybook format (#12755)

* signature request storybook

* remove clearConfirmTransaction

* fix text

ui/components/app/signature-request/README.mdx

@@ -0,0 +1,15 @@
+import { Story, Canvas, ArgsTable } from '@storybook/addon-docs';
+
+import SignatureRequest from '.';
+
+# Signature Request
+
+dApp requesting a signature from the user.
+
+<Canvas>
+  <Story id="ui-components-app-signature-request-signature-request-stories-js--default-story" />
+</Canvas>
+
+## Component API
+
+<ArgsTable of={SignatureRequest} />

ui/components/app/signature-request/signature-request.component.js

@@ -8,15 +8,33 @@ import Message from './signature-request-message';
 
 export default class SignatureRequest extends PureComponent {
   static propTypes = {
+    /**
+     * The display content of transaction data
+     */
     txData: PropTypes.object.isRequired,
+    /**
+     * The display content of sender account
+     */
     fromAccount: PropTypes.shape({
       address: PropTypes.string.isRequired,
       balance: PropTypes.string,
       name: PropTypes.string,
     }).isRequired,
+    /**
+     * Check if the wallet is ledget wallet or not
+     */
     isLedgerWallet: PropTypes.bool,
+    /**
+     * Handler for cancel button
+     */
     cancel: PropTypes.func.isRequired,
+    /**
+     * Handler for sign button
+     */
     sign: PropTypes.func.isRequired,
+    /**
+     * Whether the hardware wallet requires a connection disables the sign button if true.
+     */
     hardwareWalletRequiresConnection: PropTypes.bool.isRequired,
   };
 

ui/components/app/signature-request/signature-request.stories.js

@@ -1,40 +1,60 @@
 import React from 'react';
 import testData from '../../../../.storybook/test-data';
+import README from './README.mdx';
 import SignatureRequest from './signature-request.component';
 
 const primaryIdentity = Object.values(testData.metamask.identities)[0];
 
-const containerStyle = {
-  width: '357px',
-};
-
 export default {
   title: 'Components/App/SignatureRequest',
   id: __filename,
+  component: SignatureRequest,
+  parameters: {
+    docs: {
+      page: README,
+    },
+  },
+  argTypes: {
+    txData: { control: 'object' },
+    fromAccount: {
+      table: {
+        address: { control: 'text' },
+        balance: { control: 'text' },
+        name: { control: 'text' },
+      },
+    },
+    isLedgerWallet: { control: 'boolean' },
+    clearConfirmTransaction: { action: 'Clean Confirm' },
+    cancel: { action: 'Cancel' },
+    sign: { action: 'Sign' },
+    hardwareWalletRequiresConnection: {
+      action: 'hardwareWalletRequiresConnection',
+    },
+  },
 };
 
-export const DefaultStory = () => {
+export const DefaultStory = (args) => {
-  return (
+  return <SignatureRequest {...args} />;
-    <div style={containerStyle}>
+};
-      <SignatureRequest
+
-        txData={{
+DefaultStory.storyName = 'Default';
-          msgParams: {
+
-            data: JSON.stringify({
+DefaultStory.args = {
-              domain: {
+  txData: {
-                name: 'happydapp.website',
+    msgParams: {
-              },
+      data: JSON.stringify({
-              message: {
+        domain: {
-                string: 'haay wuurl',
+          name: 'happydapp.website',
-                number: 42,
+        },
-              },
+        message: {
-            }),
+          string: 'haay wuurl',
-            origin: 'https://happydapp.website/governance?futarchy=true',
+          number: 42,
-          },
+        },
-        }}
+      }),
-        fromAccount={primaryIdentity}
+      origin: 'https://happydapp.website/governance?futarchy=true',
-      />
+    },
-    </div>
+  },
-  );
+  fromAccount: primaryIdentity,
 };
 
 DefaultStory.storyName = 'Default';