Skip to content
New issue

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

#

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

Already on GitHub? # to your account

[docs] Modularize Imports for Nextjs #35457

Closed
wants to merge 3 commits into from
Closed

[docs] Modularize Imports for Nextjs #35457

Changes from 2 commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
a301bfa
[docs] Modularize Imports for Nextjs
dev-ahmedhany Dec 13, 2022
c3fc875
Update docs/data/material/guides/minimizing-bundle-size/minimizing-bu…
dev-ahmedhany Dec 15, 2022
4ddd6de
docs modularize Imports for nextjs remove expermintal
dev-ahmedhany Jan 10, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
25 changes: 25 additions & 0 deletions docs/data/material/guides/minimizing-bundle-size/minimizing-bundle-size.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -194,6 +194,31 @@ Modify your `package.json` commands:
}
```

If you are using Next.js >= v12.1.1, you can make use of their [Modularize Imports](https://nextjs.org/docs/advanced-features/compiler#modularize-imports) feature.
Copy link
Member

Choose a reason for hiding this comment

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

This section of the docs starts to be quite long. I think that it could be good to change the headers to help navigate the content.

This does not require adding Babel configuration, so you can still use [SWC](https://nextjs.org/docs/advanced-features/compiler#why-swc) if you prefer.

Create a `next.config.js` file in the root directory:

```js
/* next.config.js */
/** @type {import('next').NextConfig} */
const nextConfig = {
...
experimental: {
Copy link
Member

Choose a reason for hiding this comment

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

It doesn't seems to be under experimental in https://nextjs.org/docs/advanced-features/compiler#modularize-imports

Copy link
Member

Choose a reason for hiding this comment

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

It's recently promoted

modularizeImports: {
'@mui/material': {
transform: '@mui/material/{{member}}'
},
Copy link
Member

@oliviertassinari oliviertassinari Jan 4, 2023 •
edited
Loading

Choose a reason for hiding this comment

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

I wouldn't do this, this is probably overkill. There are also cases where it doesn't work because the code is not consistent enough.

'@mui/icons-material': {
transform: '@mui/icons-material/{{member}}'
}
}
}
}

module.exports = nextConfig
```

Enjoy significantly faster start times.

#### 2. Convert all your imports
Expand Down