Page MenuHomePhabricator

[Cashtab] Signature Copied text color patch
ClosedPublic

Authored by kieran709 on May 4 2022, 16:10.

Details

Reviewers
bytesofman
Group Reviewers
Restricted Project
Commits
rABC7d6f9672f27a: [Cashtab] Signature Copied text color patch
Summary

Rekated to task T2434. Patched font color of "Signature copied to clipboard" text in Send.js to lightGrey.

Test Plan

cd web/cashtab && npm start
navigate to send tab
expand the Sign Message collapse and generate a signature
click on the 'Signature:' text area
observe that the text "Signature copied to clipboard" is lightgrey

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Failed tests logs:

====== CashTab Unit Tests:  Wallet without BCH balance ======
Error: expect(received).toMatchSnapshot()

Snapshot name: `Wallet without BCH balance 1`

- Snapshot  - 1
+ Received  + 1

@@ -28,11 +28,11 @@
        <div
          className="ant-col ant-col-24"
          style={Object {}}
        >
          <div
-           className="sc-kgoBCf ivtXmS"
+           className="sc-kgoBCf jGSjNz"
          >
            <div
              className="ant-collapse ant-collapse-icon-position-left sc-kpOJdX juWwfY"
              role={null}
              style={
    at Object.<anonymous> (/work/web/cashtab/src/components/Airdrop/__tests__/Airdrop.test.js:46:18)
    at Promise.then.completed (/work/web/cashtab/node_modules/jest-circus/build/utils.js:391:28)
    at new Promise (<anonymous>)
    at callAsyncCircusFn (/work/web/cashtab/node_modules/jest-circus/build/utils.js:316:10)
    at _callCircusTest (/work/web/cashtab/node_modules/jest-circus/build/run.js:218:40)
    at processTicksAndRejections (node:internal/process/task_queues:94:5)
    at _runTest (/work/web/cashtab/node_modules/jest-circus/build/run.js:155:3)
    at _runTestsForDescribeBlock (/work/web/cashtab/node_modules/jest-circus/build/run.js:66:9)
    at run (/work/web/cashtab/node_modules/jest-circus/build/run.js:25:3)
    at runAndTransformResultsToJestFormat (/work/web/cashtab/node_modules/jest-circus/build/legacy-code-todo-rewrite/jestAdapterInit.js:170:21)
    at jestAdapter (/work/web/cashtab/node_modules/jest-circus/build/legacy-code-todo-rewrite/jestAdapter.js:82:19)
    at runTestInternal (/work/web/cashtab/node_modules/jest-runner/build/runTest.js:389:16)
    at runTest (/work/web/cashtab/node_modules/jest-runner/build/runTest.js:475:34)
    at Object.worker (/work/web/cashtab/node_modules/jest-runner/build/testWorker.js:133:12)Error: expect(received).toMatchSnapshot()

Snapshot name: `Wallet without BCH balance 1`

- Snapshot  - 3
+ Received  + 3

@@ -377,11 +377,11 @@
              >
                Send
              </button>
            </div>
            <div
-             className="sc-kgoBCf ivtXmS"
+             className="sc-kgoBCf jGSjNz"
            >
              <div
                className="ant-collapse ant-collapse-icon-position-left sc-kpOJdX juWwfY"
                role={null}
                style={
@@ -433,11 +433,11 @@
            </div>
          </form>
        </div>
      </div>
      <div
-       className="sc-kgoBCf ivtXmS"
+       className="sc-kgoBCf jGSjNz"
      >
        <div
          className="ant-collapse ant-collapse-icon-position-left sc-kpOJdX juWwfY"
          role={null}
          style={
@@ -486,11 +486,11 @@
            </div>
          </div>
        </div>
      </div>
      <div
-       className="sc-kgoBCf ivtXmS"
+       className="sc-kgoBCf jGSjNz"
      >
        <div
          className="ant-collapse ant-collapse-icon-position-left sc-kpOJdX juWwfY"
          role={null}
          style={
    at Object.<anonymous> (/work/web/cashtab/src/components/Send/__tests__/Send.test.js:46:18)
    at Promise.then.completed (/work/web/cashtab/node_modules/jest-circus/build/utils.js:391:28)
    at new Promise (<anonymous>)
    at callAsyncCircusFn (/work/web/cashtab/node_modules/jest-circus/build/utils.js:316:10)
    at _callCircusTest (/work/web/cashtab/node_modules/jest-circus/build/run.js:218:40)
    at processTicksAndRejections (node:internal/process/task_queues:94:5)
    at _runTest (/work/web/cashtab/node_modules/jest-circus/build/run.js:155:3)
    at _runTestsForDescribeBlock (/work/web/cashtab/node_modules/jest-circus/build/run.js:66:9)
    at run (/work/web/cashtab/node_modules/jest-circus/build/run.js:25:3)
    at runAndTransformResultsToJestFormat (/work/web/cashtab/node_modules/jest-circus/build/legacy-code-todo-rewrite/jestAdapterInit.js:170:21)
    at jestAdapter (/work/web/cashtab/node_modules/jest-circus/build/legacy-code-todo-rewrite/jestAdapter.js:82:19)
    at runTestInternal (/work/web/cashtab/node_modules/jest-runner/build/runTest.js:389:16)
    at runTest (/work/web/cashtab/node_modules/jest-runner/build/runTest.js:475:34)
    at Object.worker (/work/web/cashtab/node_modules/jest-runner/build/testWorker.js:133:12)
====== CashTab Unit Tests:  Wallet with BCH balances ======
Error: expect(received).toMatchSnapshot()

Snapshot name: `Wallet with BCH balances 1`

- Snapshot  - 1
+ Received  + 1

@@ -28,11 +28,11 @@
        <div
          className="ant-col ant-col-24"
          style={Object {}}
        >
          <div
-           className="sc-kgoBCf ivtXmS"
+           className="sc-kgoBCf jGSjNz"
          >
            <div
              className="ant-collapse ant-collapse-icon-position-left sc-kpOJdX juWwfY"
              role={null}
              style={
    at Object.<anonymous> (/work/web/cashtab/src/components/Airdrop/__tests__/Airdrop.test.js:61:18)
    at Promise.then.completed (/work/web/cashtab/node_modules/jest-circus/build/utils.js:391:28)
    at new Promise (<anonymous>)
    at callAsyncCircusFn (/work/web/cashtab/node_modules/jest-circus/build/utils.js:316:10)
    at _callCircusTest (/work/web/cashtab/node_modules/jest-circus/build/run.js:218:40)
    at processTicksAndRejections (node:internal/process/task_queues:94:5)
    at _runTest (/work/web/cashtab/node_modules/jest-circus/build/run.js:155:3)
    at _runTestsForDescribeBlock (/work/web/cashtab/node_modules/jest-circus/build/run.js:66:9)
    at run (/work/web/cashtab/node_modules/jest-circus/build/run.js:25:3)
    at runAndTransformResultsToJestFormat (/work/web/cashtab/node_modules/jest-circus/build/legacy-code-todo-rewrite/jestAdapterInit.js:170:21)
    at jestAdapter (/work/web/cashtab/node_modules/jest-circus/build/legacy-code-todo-rewrite/jestAdapter.js:82:19)
    at runTestInternal (/work/web/cashtab/node_modules/jest-runner/build/runTest.js:389:16)
    at runTest (/work/web/cashtab/node_modules/jest-runner/build/runTest.js:475:34)
    at Object.worker (/work/web/cashtab/node_modules/jest-runner/build/testWorker.js:133:12)Error: expect(received).toMatchSnapshot()

Snapshot name: `Wallet with BCH balances 1`

- Snapshot  - 3
+ Received  + 3

@@ -377,11 +377,11 @@
              >
                Send
              </button>
            </div>
            <div
-             className="sc-kgoBCf ivtXmS"
+             className="sc-kgoBCf jGSjNz"
            >
              <div
                className="ant-collapse ant-collapse-icon-position-left sc-kpOJdX juWwfY"
                role={null}
                style={
@@ -433,11 +433,11 @@
            </div>
          </form>
        </div>
      </div>
      <div
-       className="sc-kgoBCf ivtXmS"
+       className="sc-kgoBCf jGSjNz"
      >
        <div
          className="ant-collapse ant-collapse-icon-position-left sc-kpOJdX juWwfY"
          role={null}
          style={
@@ -486,11 +486,11 @@
            </div>
          </div>
        </div>
      </div>
      <div
-       className="sc-kgoBCf ivtXmS"
+       className="sc-kgoBCf jGSjNz"
      >
        <div
          className="ant-collapse ant-collapse-icon-position-left sc-kpOJdX juWwfY"
          role={null}
          style={
    at Object.<anonymous> (/work/web/cashtab/src/components/Send/__tests__/Send.test.js:61:18)
    at Promise.then.completed (/work/web/cashtab/node_modules/jest-circus/build/utils.js:391:28)
    at new Promise (<anonymous>)
    at callAsyncCircusFn (/work/web/cashtab/node_modules/jest-circus/build/utils.js:316:10)
    at _callCircusTest (/work/web/cashtab/node_modules/jest-circus/build/run.js:218:40)
    at processTicksAndRejections (node:internal/process/task_queues:94:5)
    at _runTest (/work/web/cashtab/node_modules/jest-circus/build/run.js:155:3)
    at _runTestsForDescribeBlock (/work/web/cashtab/node_modules/jest-circus/build/run.js:66:9)
    at run (/work/web/cashtab/node_modules/jest-circus/build/run.js:25:3)
    at runAndTransformResultsToJestFormat (/work/web/cashtab/node_modules/jest-circus/build/legacy-code-todo-rewrite/jestAdapterInit.js:170:21)
    at jestAdapter (/work/web/cashtab/node_modules/jest-circus/build/legacy-code-todo-rewrite/jestAdapter.js:82:19)
    at runTestInternal (/work/web/cashtab/node_modules/jest-runner/build/runTest.js:389:16)
    at runTest (/work/web/cashtab/node_modules/jest-runner/build/runTest.js:475:34)
    at Object.worker (/work/web/cashtab/node_modules/jest-runner/build/testWorker.js:133:12)
====== CashTab Unit Tests:  Wallet with BCH balances and tokens ======
Error: expect(received).toMatchSnapshot()

Snapshot name: `Wallet with BCH balances and tokens 1`

- Snapshot  - 1
+ Received  + 1

@@ -28,11 +28,11 @@
        <div
          className="ant-col ant-col-24"
          style={Object {}}
        >
          <div
-           className="sc-kgoBCf ivtXmS"
+           className="sc-kgoBCf jGSjNz"
          >
            <div
              className="ant-collapse ant-collapse-icon-position-left sc-kpOJdX juWwfY"
              role={null}
              style={
    at Object.<anonymous> (/work/web/cashtab/src/components/Airdrop/__tests__/Airdrop.test.js:76:18)
    at Promise.then.completed (/work/web/cashtab/node_modules/jest-circus/build/utils.js:391:28)
    at new Promise (<anonymous>)
    at callAsyncCircusFn (/work/web/cashtab/node_modules/jest-circus/build/utils.js:316:10)
    at _callCircusTest (/work/web/cashtab/node_modules/jest-circus/build/run.js:218:40)
    at processTicksAndRejections (node:internal/process/task_queues:94:5)
    at _runTest (/work/web/cashtab/node_modules/jest-circus/build/run.js:155:3)
    at _runTestsForDescribeBlock (/work/web/cashtab/node_modules/jest-circus/build/run.js:66:9)
    at run (/work/web/cashtab/node_modules/jest-circus/build/run.js:25:3)
    at runAndTransformResultsToJestFormat (/work/web/cashtab/node_modules/jest-circus/build/legacy-code-todo-rewrite/jestAdapterInit.js:170:21)
    at jestAdapter (/work/web/cashtab/node_modules/jest-circus/build/legacy-code-todo-rewrite/jestAdapter.js:82:19)
    at runTestInternal (/work/web/cashtab/node_modules/jest-runner/build/runTest.js:389:16)
    at runTest (/work/web/cashtab/node_modules/jest-runner/build/runTest.js:475:34)
    at Object.worker (/work/web/cashtab/node_modules/jest-runner/build/testWorker.js:133:12)Error: expect(received).toMatchSnapshot()

Snapshot name: `Wallet with BCH balances and tokens 1`

- Snapshot  - 3
+ Received  + 3

@@ -377,11 +377,11 @@
              >
                Send
              </button>
            </div>
            <div
-             className="sc-kgoBCf ivtXmS"
+             className="sc-kgoBCf jGSjNz"
            >
              <div
                className="ant-collapse ant-collapse-icon-position-left sc-kpOJdX juWwfY"
                role={null}
                style={
@@ -433,11 +433,11 @@
            </div>
          </form>
        </div>
      </div>
      <div
-       className="sc-kgoBCf ivtXmS"
+       className="sc-kgoBCf jGSjNz"
      >
        <div
          className="ant-collapse ant-collapse-icon-position-left sc-kpOJdX juWwfY"
          role={null}
          style={
@@ -486,11 +486,11 @@
            </div>
          </div>
        </div>
      </div>
      <div
-       className="sc-kgoBCf ivtXmS"
+       className="sc-kgoBCf jGSjNz"
      >
        <div
          className="ant-collapse ant-collapse-icon-position-left sc-kpOJdX juWwfY"
          role={null}
          style={
    at Object.<anonymous> (/work/web/cashtab/src/components/Send/__tests__/Send.test.js:76:18)
    at Promise.then.completed (/work/web/cashtab/node_modules/jest-circus/build/utils.js:391:28)
    at new Promise (<anonymous>)
    at callAsyncCircusFn (/work/web/cashtab/node_modules/jest-circus/build/utils.js:316:10)
    at _callCircusTest (/work/web/cashtab/node_modules/jest-circus/build/run.js:218:40)
    at processTicksAndRejections (node:internal/process/task_queues:94:5)
    at _runTest (/work/web/cashtab/node_modules/jest-circus/build/run.js:155:3)
    at _runTestsForDescribeBlock (/work/web/cashtab/node_modules/jest-circus/build/run.js:66:9)
    at run (/work/web/cashtab/node_modules/jest-circus/build/run.js:25:3)
    at runAndTransformResultsToJestFormat (/work/web/cashtab/node_modules/jest-circus/build/legacy-code-todo-rewrite/jestAdapterInit.js:170:21)
    at jestAdapter (/work/web/cashtab/node_modules/jest-circus/build/legacy-code-todo-rewrite/jestAdapter.js:82:19)
    at runTestInternal (/work/web/cashtab/node_modules/jest-runner/build/runTest.js:389:16)
    at runTest (/work/web/cashtab/node_modules/jest-runner/build/runTest.js:475:34)
    at Object.worker (/work/web/cashtab/node_modules/jest-runner/build/testWorker.js:133:12)
====== CashTab Unit Tests:  Wallet with BCH balances and tokens and state field ======
Error: expect(received).toMatchSnapshot()

Snapshot name: `Wallet with BCH balances and tokens and state field 1`

- Snapshot  - 1
+ Received  + 1

@@ -27,11 +27,11 @@
        <div
          className="ant-col ant-col-24"
          style={Object {}}
        >
          <div
-           className="sc-kgoBCf ivtXmS"
+           className="sc-kgoBCf jGSjNz"
          >
            <div
              className="ant-collapse ant-collapse-icon-position-left sc-kpOJdX juWwfY"
              role={null}
              style={
    at Object.<anonymous> (/work/web/cashtab/src/components/Airdrop/__tests__/Airdrop.test.js:93:18)
    at Promise.then.completed (/work/web/cashtab/node_modules/jest-circus/build/utils.js:391:28)
    at new Promise (<anonymous>)
    at callAsyncCircusFn (/work/web/cashtab/node_modules/jest-circus/build/utils.js:316:10)
    at _callCircusTest (/work/web/cashtab/node_modules/jest-circus/build/run.js:218:40)
    at processTicksAndRejections (node:internal/process/task_queues:94:5)
    at _runTest (/work/web/cashtab/node_modules/jest-circus/build/run.js:155:3)
    at _runTestsForDescribeBlock (/work/web/cashtab/node_modules/jest-circus/build/run.js:66:9)
    at run (/work/web/cashtab/node_modules/jest-circus/build/run.js:25:3)
    at runAndTransformResultsToJestFormat (/work/web/cashtab/node_modules/jest-circus/build/legacy-code-todo-rewrite/jestAdapterInit.js:170:21)
    at jestAdapter (/work/web/cashtab/node_modules/jest-circus/build/legacy-code-todo-rewrite/jestAdapter.js:82:19)
    at runTestInternal (/work/web/cashtab/node_modules/jest-runner/build/runTest.js:389:16)
    at runTest (/work/web/cashtab/node_modules/jest-runner/build/runTest.js:475:34)
    at Object.worker (/work/web/cashtab/node_modules/jest-runner/build/testWorker.js:133:12)Error: expect(received).toMatchSnapshot()

Snapshot name: `Wallet with BCH balances and tokens and state field 1`

- Snapshot  - 3
+ Received  + 3

@@ -377,11 +377,11 @@
              >
                Send
              </button>
            </div>
            <div
-             className="sc-kgoBCf ivtXmS"
+             className="sc-kgoBCf jGSjNz"
            >
              <div
                className="ant-collapse ant-collapse-icon-position-left sc-kpOJdX juWwfY"
                role={null}
                style={
@@ -433,11 +433,11 @@
            </div>
          </form>
        </div>
      </div>
      <div
-       className="sc-kgoBCf ivtXmS"
+       className="sc-kgoBCf jGSjNz"
      >
        <div
          className="ant-collapse ant-collapse-icon-position-left sc-kpOJdX juWwfY"
          role={null}
          style={
@@ -486,11 +486,11 @@
            </div>
          </div>
        </div>
      </div>
      <div
-       className="sc-kgoBCf ivtXmS"
+       className="sc-kgoBCf jGSjNz"
      >
        <div
          className="ant-collapse ant-collapse-icon-position-left sc-kpOJdX juWwfY"
          role={null}
          style={
    at Object.<anonymous> (/work/web/cashtab/src/components/Send/__tests__/Send.test.js:93:18)
    at Promise.then.completed (/work/web/cashtab/node_modules/jest-circus/build/utils.js:391:28)
    at new Promise (<anonymous>)
    at callAsyncCircusFn (/work/web/cashtab/node_modules/jest-circus/build/utils.js:316:10)
    at _callCircusTest (/work/web/cashtab/node_modules/jest-circus/build/run.js:218:40)
    at processTicksAndRejections (node:internal/process/task_queues:94:5)
    at _runTest (/work/web/cashtab/node_modules/jest-circus/build/run.js:155:3)
    at _runTestsForDescribeBlock (/work/web/cashtab/node_modules/jest-circus/build/run.js:66:9)
    at run (/work/web/cashtab/node_modules/jest-circus/build/run.js:25:3)
    at runAndTransformResultsToJestFormat (/work/web/cashtab/node_modules/jest-circus/build/legacy-code-todo-rewrite/jestAdapterInit.js:170:21)
    at jestAdapter (/work/web/cashtab/node_modules/jest-circus/build/legacy-code-todo-rewrite/jestAdapter.js:82:19)
    at runTestInternal (/work/web/cashtab/node_modules/jest-runner/build/runTest.js:389:16)
    at runTest (/work/web/cashtab/node_modules/jest-runner/build/runTest.js:475:34)
    at Object.worker (/work/web/cashtab/node_modules/jest-runner/build/testWorker.js:133:12)
====== CashTab Unit Tests:  Without wallet defined ======
Error: expect(received).toMatchSnapshot()

Snapshot name: `Without wallet defined 1`

- Snapshot  - 1
+ Received  + 1

@@ -23,11 +23,11 @@
        <div
          className="ant-col ant-col-24"
          style={Object {}}
        >
          <div
-           className="sc-kgoBCf ivtXmS"
+           className="sc-kgoBCf jGSjNz"
          >
            <div
              className="ant-collapse ant-collapse-icon-position-left sc-kpOJdX juWwfY"
              role={null}
              style={
    at Object.<anonymous> (/work/web/cashtab/src/components/Airdrop/__tests__/Airdrop.test.js:113:18)
    at Promise.then.completed (/work/web/cashtab/node_modules/jest-circus/build/utils.js:391:28)
    at new Promise (<anonymous>)
    at callAsyncCircusFn (/work/web/cashtab/node_modules/jest-circus/build/utils.js:316:10)
    at _callCircusTest (/work/web/cashtab/node_modules/jest-circus/build/run.js:218:40)
    at processTicksAndRejections (node:internal/process/task_queues:94:5)
    at _runTest (/work/web/cashtab/node_modules/jest-circus/build/run.js:155:3)
    at _runTestsForDescribeBlock (/work/web/cashtab/node_modules/jest-circus/build/run.js:66:9)
    at run (/work/web/cashtab/node_modules/jest-circus/build/run.js:25:3)
    at runAndTransformResultsToJestFormat (/work/web/cashtab/node_modules/jest-circus/build/legacy-code-todo-rewrite/jestAdapterInit.js:170:21)
    at jestAdapter (/work/web/cashtab/node_modules/jest-circus/build/legacy-code-todo-rewrite/jestAdapter.js:82:19)
    at runTestInternal (/work/web/cashtab/node_modules/jest-runner/build/runTest.js:389:16)
    at runTest (/work/web/cashtab/node_modules/jest-runner/build/runTest.js:475:34)
    at Object.worker (/work/web/cashtab/node_modules/jest-runner/build/testWorker.js:133:12)Error: expect(received).toMatchSnapshot()

Snapshot name: `Without wallet defined 1`

- Snapshot  - 3
+ Received  + 3

@@ -372,11 +372,11 @@
              >
                Send
              </button>
            </div>
            <div
-             className="sc-kgoBCf ivtXmS"
+             className="sc-kgoBCf jGSjNz"
            >
              <div
                className="ant-collapse ant-collapse-icon-position-left sc-kpOJdX juWwfY"
                role={null}
                style={
@@ -428,11 +428,11 @@
            </div>
          </form>
        </div>
      </div>
      <div
-       className="sc-kgoBCf ivtXmS"
+       className="sc-kgoBCf jGSjNz"
      >
        <div
          className="ant-collapse ant-collapse-icon-position-left sc-kpOJdX juWwfY"
          role={null}
          style={
@@ -481,11 +481,11 @@
            </div>
          </div>
        </div>
      </div>
      <div
-       className="sc-kgoBCf ivtXmS"
+       className="sc-kgoBCf jGSjNz"
      >
        <div
          className="ant-collapse ant-collapse-icon-position-left sc-kpOJdX juWwfY"
          role={null}
          style={
    at Object.<anonymous> (/work/web/cashtab/src/components/Send/__tests__/Send.test.js:113:18)
    at Promise.then.completed (/work/web/cashtab/node_modules/jest-circus/build/utils.js:391:28)
    at new Promise (<anonymous>)
    at callAsyncCircusFn (/work/web/cashtab/node_modules/jest-circus/build/utils.js:316:10)
    at _callCircusTest (/work/web/cashtab/node_modules/jest-circus/build/run.js:218:40)
    at processTicksAndRejections (node:internal/process/task_queues:94:5)
    at _runTest (/work/web/cashtab/node_modules/jest-circus/build/run.js:155:3)
    at _runTestsForDescribeBlock (/work/web/cashtab/node_modules/jest-circus/build/run.js:66:9)
    at run (/work/web/cashtab/node_modules/jest-circus/build/run.js:25:3)
    at runAndTransformResultsToJestFormat (/work/web/cashtab/node_modules/jest-circus/build/legacy-code-todo-rewrite/jestAdapterInit.js:170:21)
    at jestAdapter (/work/web/cashtab/node_modules/jest-circus/build/legacy-code-todo-rewrite/jestAdapter.js:82:19)
    at runTestInternal (/work/web/cashtab/node_modules/jest-runner/build/runTest.js:389:16)
    at runTest (/work/web/cashtab/node_modules/jest-runner/build/runTest.js:475:34)
    at Object.worker (/work/web/cashtab/node_modules/jest-runner/build/testWorker.js:133:12)

Each failure log is accessible here:
CashTab Unit Tests: Wallet without BCH balance
CashTab Unit Tests: Wallet with BCH balances
CashTab Unit Tests: Wallet with BCH balances and tokens
CashTab Unit Tests: Wallet with BCH balances and tokens and state field
CashTab Unit Tests: Without wallet defined

bytesofman requested changes to this revision.May 4 2022, 23:21

Update snapshots

Bonus: we should have the snapshots update before the commit is made. It's clear now that this automation is necessary. It's okay to do with with a new module if it's installed as a dev dependency. Please make a task and look into this.

This revision now requires changes to proceed.May 4 2022, 23:21
This revision is now accepted and ready to land.May 5 2022, 16:56