Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

UX: Icon: Remove fa-exclamation-triangle usages #17691

Merged
merged 19 commits into from Feb 23, 2023

Conversation

darkwing
Copy link
Contributor

@darkwing darkwing commented Feb 9, 2023

Explanation

Removes usages of fa-exclamation-triangle throughout the app.

Screenshots/Screencaps

Add Network

Danger

Confirmation Warning Modal

DangerWarningModal

Custom Spending Cap Tooltip

CustomSpendingCap

Signature Request Origin Warning

WarnWarn

Skip SRP Popover

DangerBig

Pre-merge author checklist

  • I've clearly explained:
    • What problem this PR is solving
    • How this problem was solved
    • How reviewers can test my changes
  • Sufficient automated test coverage has been added

Pre-merge reviewer checklist

  • Manual testing (e.g. pull and build branch, run in browser, test code being changed)
  • PR is linked to the appropriate GitHub issue
  • IF this PR fixes a bug in the release milestone, add this PR to the release milestone

If further QA is required (e.g. new feature, complex testing steps, large refactor), add the Extension QA Board label.

In this case, a QA Engineer approval will be be required.

@darkwing darkwing requested a review from a team as a code owner February 9, 2023 16:21
@darkwing darkwing marked this pull request as draft February 9, 2023 16:21
@github-actions
Copy link
Contributor

github-actions bot commented Feb 9, 2023

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

@metamaskbot
Copy link
Collaborator

Builds ready [2109ac3]
Page Load Metrics (1266 ± 75 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint1083231474522
domContentLoaded9941621122916981
load10501622126615675
domInteractive9941621122916981
Bundle size diffs
  • background: 0 bytes
  • ui: 407 bytes
  • common: 0 bytes

@darkwing
Copy link
Contributor Author

Seeing this really weird error:

Finished 'scripts:core:dist:contentscript'
scripts:core:dist:standardEntryPoints: Bundling failed! See details below.
scripts:core:dist:standardEntryPoints: [SyntaxError: 'import' and 'export' may appear only with 'sourceType: module'] {
  line: 1,
  column: 1,
  annotated: '\n' +
    '/Users/davidwalsh/Projects/metamask-extension/ui/components/ui/typography/README.mdx:1\n' +
    "import { Story, Canvas, ArgsTable } from '@storybook/addon-docs';\n" +
    '^\n' +
    "ParseError: 'import' and 'export' may appear only with 'sourceType: module'",
  stream: Labeled {
    _readableState: ReadableState {
      objectMode: true,
      highWaterMark: 16,
      buffer: BufferList { length: 0 },
      length: 0,
      pipes: [Labeled],
      pipesCount: 1,
      flowing: true,
      ended: false,
      endEmitted: false,
      reading: true,
      sync: false,
      needReadable: true,
      emittedReadable: false,
      readableListening: false,
      resumeScheduled: false,
      destroyed: false,
      defaultEncoding: 'utf8',
      awaitDrain: 0,
      readingMore: false,
      decoder: null,
      encoding: null
    },
    readable: true,
    _events: [Object: null prototype] {
      end: [Array],
      finish: [Object],
      error: {},
      data: {},
      _mutate: [Object]
    },
    _eventsCount: 5,
    _maxListeners: undefined,
    _writableState: WritableState {
      objectMode: true,
      highWaterMark: 16,
      finalCalled: false,
      needDrain: false,
      ending: false,
      ended: false,
      finished: false,
      destroyed: false,
      decodeStrings: true,
      defaultEncoding: 'utf8',
      length: 1,
      writing: true,
      corked: 0,
      sync: true,
      bufferProcessing: false,
      onwrite: {},
      writecb: {},
      writelen: 1,
      bufferedRequest: null,
      lastBufferedRequest: null,
      pendingcb: 1,
      prefinished: false,
      errorEmitted: false,
      bufferedRequestCount: 0,
      corkedRequestsFree: [CorkedRequest]
    },
    writable: true,
    allowHalfOpen: true,
    _options: { objectMode: true },
    _wrapOptions: { objectMode: true },
    _streams: [ [DestroyableTransform] ],
    length: 1,
    label: 'syntax',
    [Symbol(kCapture)]: false
  }
}
MetaMask build: Encountered an error while running task "dist".
Error: MetaMask build: runInChildProcess for task "scripts:core:dist:standardEntryPoints" encountered an error "1".
  at ChildProcess.eval (/Users/davidwalsh/Projects/metamask-extension/development/build/task.js:100:13)
  at Object.onceWrapper (node:events:642:26)
  at ChildProcess.emit (node:events:527:28)
  at Process.ChildProcess._handle.onexit (node:internal/child_process:291:12)

@darkwing darkwing changed the title UX: Icon: WIP: Remove fa-exclamation-triangle usages UX: Icon: Remove fa-exclamation-triangle usages Feb 15, 2023
@darkwing darkwing marked this pull request as ready for review February 15, 2023 14:45
@metamaskbot
Copy link
Collaborator

Builds ready [52dcc9d]
Page Load Metrics (1326 ± 127 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint96151119168
domContentLoaded97717201292246118
load98718541326264127
domInteractive97717201292246118
Bundle size diffs
  • background: 0 bytes
  • ui: 1111 bytes
  • common: 0 bytes

Copy link
Member

@NidhiKJha NidhiKJha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks really good!! Just added a couple of suggestions

@metamaskbot
Copy link
Collaborator

Builds ready [59e54c6]
Page Load Metrics (1582 ± 40 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint1003081244321
domContentLoaded1417174515497536
load1440174515828240
domInteractive1417174515497536
Bundle size diffs
  • background: 0 bytes
  • ui: 928 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [de84856]
Page Load Metrics (1664 ± 89 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint98157122189
domContentLoaded14532021162014469
load14532210166418589
domInteractive14532021162014469
Bundle size diffs
  • background: 0 bytes
  • ui: 1059 bytes
  • common: 0 bytes

NidhiKJha
NidhiKJha previously approved these changes Feb 21, 2023
@metamaskbot
Copy link
Collaborator

Builds ready [19fd927]
Page Load Metrics (1594 ± 54 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint94142114136
domContentLoaded14301843155910048
load14311874159411254
domInteractive14301843155910048
Bundle size diffs
  • background: 0 bytes
  • ui: 1063 bytes
  • common: 0 bytes

Copy link
Contributor

@georgewrmarshall georgewrmarshall left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work on the story

NidhiKJha
NidhiKJha previously approved these changes Feb 22, 2023
@metamaskbot
Copy link
Collaborator

Builds ready [f2f89d3]
Page Load Metrics (1805 ± 74 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint1033581395225
domContentLoaded15932085176715172
load16142098180515374
domInteractive15932085176715172
Bundle size diffs
  • background: 0 bytes
  • ui: 1081 bytes
  • common: 0 bytes

DDDDDanica
DDDDDanica previously approved these changes Feb 22, 2023
darkwing and others added 17 commits February 23, 2023 11:41
…pover.stories.js

Co-authored-by: Nidhi Kumari <nidhi.kumari@consensys.net>
…pover.stories.js

Co-authored-by: Nidhi Kumari <nidhi.kumari@consensys.net>
…pover.stories.js

Co-authored-by: Nidhi Kumari <nidhi.kumari@consensys.net>
…pover.js

Co-authored-by: Nidhi Kumari <nidhi.kumari@consensys.net>
Co-authored-by: Nidhi Kumari <nidhi.kumari@consensys.net>
Co-authored-by: Nidhi Kumari <nidhi.kumari@consensys.net>
…tip.js

Co-authored-by: George Marshall <george.marshall@consensys.net>
…ing-modal.js

Co-authored-by: Danica Shen <zhaodanica@gmail.com>
…-original-warning/signature-request-original-warning.js

Co-authored-by: Danica Shen <zhaodanica@gmail.com>
Co-authored-by: Danica Shen <zhaodanica@gmail.com>
…pover.js

Co-authored-by: Danica Shen <zhaodanica@gmail.com>
@metamaskbot
Copy link
Collaborator

Builds ready [097912e]
Page Load Metrics (1712 ± 91 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint1011911312612
domContentLoaded12491986167417282
load12492099171218991
domInteractive12491986167417282
Bundle size diffs
  • background: 0 bytes
  • ui: 962 bytes
  • common: 0 bytes

@darkwing darkwing merged commit e3380ba into develop Feb 23, 2023
@darkwing darkwing deleted the icons-exclamation-triangle branch February 23, 2023 21:20
@github-actions github-actions bot locked and limited conversation to collaborators Feb 23, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants