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

[material-ui][docs] Provide more details about why to use material-nextjs package in "Next.js integration" doc #40798

Open
popey456963 opened this issue Jan 26, 2024 · 3 comments
Assignees
Labels
docs Improvements or additions to the documentation nextjs package: material-ui Specific to @mui/material support: docs-feedback Feedback from documentation page

Comments

@popey456963
Copy link

popey456963 commented Jan 26, 2024

Related page

https://mui.com/material-ui/guides/nextjs/

Kind of issue

Missing information

Issue description

The documentation doesn't seem to specify what the difference is between material-nextjs and the default package. For those of us with an existing page router project are there reasons to convert, etc.

Context

I'm coming from an existing Next / MUI project and want to know how much I should priotitise the migration to the new package.

Search keywords: next integration

@popey456963 popey456963 added status: waiting for maintainer These issues haven't been looked at yet by a maintainer support: docs-feedback Feedback from documentation page labels Jan 26, 2024
@danilo-leal danilo-leal changed the title [docs] Next.js integration doesn't point out what changes the 'material-nextjs' package makes [material-ui][docs] Next.js integration page doesn't point out what changes the 'material-nextjs' package makes Jan 27, 2024
@danilo-leal danilo-leal added docs Improvements or additions to the documentation package: material-ui Specific to @mui/material nextjs labels Jan 27, 2024
@samuelsycamore
Copy link
Contributor

Hey @popey456963 ! Based on your description, it sounds like you may be confused about what the material-nextjs package is. It's not meant to be a replacement for any other package but rather a complementary package for integrating Material UI with Next.js. If you already have an app running fine with Next.js and Material UI, I would consider it a low priority to incorporate the integration package—though it may be useful in the future if/when we need to make changes on the Material UI side to better support Next.js. Does that make sense?

@samuelsycamore samuelsycamore removed the status: waiting for maintainer These issues haven't been looked at yet by a maintainer label Mar 4, 2024
@samuelsycamore samuelsycamore changed the title [material-ui][docs] Next.js integration page doesn't point out what changes the 'material-nextjs' package makes [material-ui][docs] Provide more details about why to use material-nextjs package in "Next.js integration" doc Mar 4, 2024
@popey456963
Copy link
Author

Thanks, that does make things clearer! Might be useful to still include it in the documentation as I can't imagine I'm the only person with an existing Next + MUI page looking to migrate :)

@siriwatknp
Copy link
Member

Similar to #43267, the author missed to install Material UI before using the integration package.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Improvements or additions to the documentation nextjs package: material-ui Specific to @mui/material support: docs-feedback Feedback from documentation page
Projects
None yet
Development

No branches or pull requests

4 participants