Skip to content

Use mixins for typography instead of placeholder selectors#9072

Merged
brad-decker merged 1 commit intodevelopfrom
convert-typography-to-mixins
Jul 29, 2020
Merged

Use mixins for typography instead of placeholder selectors#9072
brad-decker merged 1 commit intodevelopfrom
convert-typography-to-mixins

Conversation

@brad-decker
Copy link
Copy Markdown
Contributor

@brad-decker brad-decker commented Jul 24, 2020

  1. Updates typography placeholder selectors to use mixins instead.
  2. Updates usages of the selectors to use the mixins instead.

Depends on

  1. Add euclid fontface #9018
  2. Update css folder structure #9071

@metamaskbot
Copy link
Copy Markdown
Collaborator

Builds ready [4dba1cb]
Page Load Metrics (684 ± 55 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint3095522211
domContentLoaded35379368111555
load35479568411655
domInteractive35379368111555

@brad-decker
Copy link
Copy Markdown
Contributor Author

Update fonts

@brad-decker brad-decker force-pushed the update-css-folder-structure branch 3 times, most recently from cb319d3 to e2b50b4 Compare July 28, 2020 19:52
Base automatically changed from update-css-folder-structure to develop July 28, 2020 20:16
Using extend would not work inside of some css, namely
inside of media queries. This made it a clear choice to
use mixins for these styles.
@brad-decker brad-decker force-pushed the convert-typography-to-mixins branch from 4dba1cb to 0a65568 Compare July 28, 2020 20:17
@brad-decker brad-decker marked this pull request as ready for review July 28, 2020 20:17
@brad-decker brad-decker requested a review from a team as a code owner July 28, 2020 20:17
@metamaskbot
Copy link
Copy Markdown
Collaborator

Builds ready [0a65568]
Page Load Metrics (610 ± 59 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint33104542512
domContentLoaded38176960812359
load38277161012259
domInteractive38076860812359

Copy link
Copy Markdown
Member

@Gudahtt Gudahtt left a comment

Choose a reason for hiding this comment

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

LGTM!

@brad-decker brad-decker merged commit 1582855 into develop Jul 29, 2020
@brad-decker brad-decker deleted the convert-typography-to-mixins branch July 29, 2020 15:35
Gudahtt added a commit that referenced this pull request Jul 30, 2020
* origin/develop: (582 commits)
  Use async/await for seedPhraseVerifier.verifyAccounts (#9100)
  Use async/await for getRestrictedMethods (#9099)
  Update dependencies (#9105)
  update email us to contact us (#9104)
  Improve source maps (#9101)
  Update font family globally (#9073)
  rpc-cap@3.1.0 (#9103)
  Use environment variable for production Sentry DSN (#9097)
  Only log error on first occurrence of missing substitution (#9096)
  Use mixins for typography instead of placeholder selectors (#9072)
  Update css folder structure (#9071)
  Disable Sentry in development (#9095)
  Use environment variable for MetaMetrics project ID (#9094)
  Use development metametrics project during tests (#9093)
  json-rpc-engine@5.2.0 (#9091)
  fixup! call initializeProvider where necessary
  call initializeProvider where necessary
  Add euclid fontface (#9018)
  fix timing-reliant network controller test
  Robustify permissions controller requestUserApproval tests (#9064)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants