Skip to content

Commit

Permalink
fix build
Browse files Browse the repository at this point in the history
  • Loading branch information
@oliviertassinari
oliviertassinari committed Aug 18, 2024
1 parent e4aad5a commit 5d4ad7e
Showing 1 changed file with 11 additions and 7 deletions.
18 changes: 11 additions & 7 deletions docs/pages/blog/introducing-pigment-css.md
View file
Original file line number Diff line number Diff line change
Expand Up @@ -89,10 +89,14 @@ For users of Emotion and styled-components, the benefits of adopting Pigment CS

When comparing the same Material UI app built with Next.js and either Emotion or Pigment CSS, we've observed the following page load performance gains using the same code, but powered by different style engines:

Check warning on line 90 in docs/pages/blog/introducing-pigment-css.md

View workflow job for this annotation

GitHub Actions / runner / vale 

[vale] reported by reviewdog 🐶
[Google.We] Try to avoid using first-person plural like 'we'.

Raw Output:
{"message": "[Google.We] Try to avoid using first-person plural like 'we'.", "location": {"path": "docs/pages/blog/introducing-pigment-css.md", "range": {"start": {"line": 90, "column": 95}}}, "severity": "WARNING"}

| Metrics | Emotion | Pigment CSS | Change |
| :--------------------------- | -------------------------------------------------------------------------------------------------------------: | -----------------------------------------------------------------------------------------------------------------: | -----: |
| First Load JS | [131 kB](https://emotion-demo.vercel.app/?metric=1) | [104 kB](https://pigment-css-demo.vercel.app/?metric=1) | -20% |
| Total blocking time | [280 ms](https://pagespeed.web.dev/analysis/https-emotion-demo-vercel-app/uxfpkvbp31?hl=fr&form_factor=mobile) | [210 ms](https://pagespeed.web.dev/analysis/https-pigment-css-demo-vercel-app/azcw1qxkec?hl=fr&form_factor=mobile) | -25% |
<!-- vale MUI.CorrectReferenceAllCases = NO -->

| Metrics | Emotion | Pigment CSS | Change |
| :------------------ | -------------------------------------------------------------------------------------------------------------: | -----------------------------------------------------------------------------------------------------------------: | -----: |
| First Load JS | [131 kB](https://emotion-demo.vercel.app/?metric=1) | [104 kB](https://pigment-css-demo.vercel.app/?metric=1) | -20% |
| Total blocking time | [280 ms](https://pagespeed.web.dev/analysis/https-emotion-demo-vercel-app/uxfpkvbp31?hl=fr&form_factor=mobile) | [210 ms](https://pagespeed.web.dev/analysis/https-pigment-css-demo-vercel-app/azcw1qxkec?hl=fr&form_factor=mobile) | -25% |

<!-- vale MUI.CorrectReferenceAllCases = YES -->

:::info
**Reproduction**. Because you should only trust numbers that you can reproduce, we've shared below how we found these numbers.
Expand All @@ -114,9 +118,9 @@ When comparing the same Material UI app built with Next.js and either Emotion o

| Metrics | Emotion | Pigment CSS | Change |
| :-------------------------------------- | ------: | ----------: | -----: |
| Create and mount a new button | 17.3 ms | 10.1 ms | -42% |
| Change a variant on a mounted component | 14.0 ms | 9.13 ms | -34% |
| Change a value inside a CSS prop | 13.6 ms | 8.63 ms | -37% |
| Create and mount a new button | 17.3 ms | 10.1 ms | -42% |
| Change a variant on a mounted component | 14.0 ms | 9.13 ms | -34% |
| Change a value inside a CSS prop | 13.6 ms | 8.63 ms | -37% |

:::info
**Reproduction**. Because you should only trust numbers that you can reproduce, we've shared below how we found these numbers.
Expand Down

0 comments on commit 5d4ad7e

Please sign in to comment.