perf(v2): improve blog mdx-loader and postcss loader#4355
Conversation
lex111
added
the
pr: performance
|
[V1] Deploy preview success Built with commit 774df2d |
|
Size Change: -483 B (0%) Total Size: 532 kB
ℹ️ View Unchanged
|
|
Deploy preview for docusaurus-2 ready! Built with commit 774df2d |
|
⚡️ Lighthouse report for the changes in this PR:
Lighthouse ran on https://deploy-preview-4355--docusaurus-2.netlify.app/classic/ |
|
Maybe we should improve Babel somehow? For example add such a plugin? https://babeljs.io/docs/en/babel-plugin-transform-react-constant-elements |
|
LGTM thanks |
|
@lex111 yes there are a few babel plugins that could be useful to try. Most of them are in https://github.com/jamiebuilds/babel-react-optimize In the past I had problems due to the order of these plugins (jamiebuilds/babel-react-optimize#33 ). but normally that works nicely. I think some of the plugins are even enabled in CRA now |
| "postcss": "^8.2.6", | ||
| "postcss": "^8.2.7", | ||
| "postcss-loader": "^4.1.0", | ||
| "postcss-preset-env": "^6.7.0", |
There was a problem hiding this comment.
Does this still need to be a dependency after the update?
4 participants
Motivation
Resolves one TODO comment related with perf of linkify function and also removes postcss-preset-env, which we essentially don't need it, because we can use autoprefixer directly instead.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
Preview
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/facebook/docusaurus, and link to your PR here.)