-
-
Notifications
You must be signed in to change notification settings - Fork 32.3k
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
[ExpansionPanel] Rename to Accordion, add warning to current exports #21560
[ExpansionPanel] Rename to Accordion, add warning to current exports #21560
Conversation
* wip * wip * reverted some changes * sorting * migration * fix in migration * fix in migration * fix in migration * Update docs/src/pages/getting-started/supported-components/supported-components.md Co-authored-by: Olivier Tassinari <olivier.tassinari@gmail.com> * added redirect * fixed material design links * codemod fix * prettier + formatted * changed accordions to accordion * docs:api * renamed accordions to accordion * Update docs/src/pages/components/accordion/accordion.md Co-authored-by: Olivier Tassinari <olivier.tassinari@gmail.com> * Update docs/src/pages/components/accordion/accordion.md Co-authored-by: Olivier Tassinari <olivier.tassinari@gmail.com> * Update docs/src/pages/components/accordion/accordion.md Co-authored-by: Olivier Tassinari <olivier.tassinari@gmail.com> * added deprecated exports for ExpansionPanel components * added motivation for renaming component * Update docs/src/pages/components/accordion/accordion.md Co-authored-by: Olivier Tassinari <olivier.tassinari@gmail.com> * Update docs/src/pages/guides/migration-v4/migration-v4.md Co-authored-by: Olivier Tassinari <olivier.tassinari@gmail.com> * Update docs/src/pages/guides/migration-v4/migration-v4.md Co-authored-by: Olivier Tassinari <olivier.tassinari@gmail.com> * cleandup migration * prettier * changed comments on deprecated components * removed * reverted del files * renamed * fixed d.ts fiels * r * added * renamed * renamed types * redirects * Revert markdown source changes * Move markdown source from expansion-panel to accordion * Expansion Panel -> Accordion * Revert attempt at renaming Co-authored-by: Olivier Tassinari <olivier.tassinari@gmail.com> Co-authored-by: Sebastian Silbermann <silbermann.sebastian@gmail.com>
I don't think so, I think that we should apply the very same changes. We rename the component in v4, we make the imports to |
@material-ui/core: parsed: +1.42% , gzip: +0.27% Details of bundle changes.Comparing: 9e7c0c1...39b2241 Details of page changes
|
I wondered that too, but depending on the timing of the release of this change in v4, it might be helpful to document the new usage, so that developers using it for the first time aren't faced with a breaking change when upgrading to v5. We could perhaps add a menu Entry for Expansion Panel that links to the Accordion page, with a note on that page that ExpansionPanel has been renamed. |
So the plan would be?
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perfect, we are almost done, this is the only issue I could spot :)
packages/material-ui/src/ExpansionPanelActions/ExpansionPanelActions.js
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I had to add the describe conformance test back as we use it to generate the API. I couldn't spot anything else. Good from my side
|
Co-authored-by: Matt <github@nospam.33m.co>
Co-authored-by: Matt <github@nospam.33m.co>
packages/material-ui/src/ExpansionPanelActions/ExpansionPanelActions.js
Outdated
Show resolved
Hide resolved
packages/material-ui/src/ExpansionPanelActions/ExpansionPanelActions.js
Outdated
Show resolved
Hide resolved
packages/material-ui/src/ExpansionPanelDetails/ExpansionPanelDetails.js
Outdated
Show resolved
Hide resolved
packages/material-ui/src/ExpansionPanelDetails/ExpansionPanelDetails.js
Outdated
Show resolved
Hide resolved
packages/material-ui/src/ExpansionPanelSummary/ExpansionPanelSummary.js
Outdated
Show resolved
Hide resolved
packages/material-ui/src/ExpansionPanelSummary/ExpansionPanelSummary.js
Outdated
Show resolved
Hide resolved
packages/material-ui/src/ExpansionPanelDetails/ExpansionPanelDetails.d.ts
Outdated
Show resolved
Hide resolved
packages/material-ui/src/ExpansionPanelActions/ExpansionPanelActions.d.ts
Outdated
Show resolved
Hide resolved
packages/material-ui/src/ExpansionPanelSummary/ExpansionPanelSummary.d.ts
Outdated
Show resolved
Hide resolved
Is there a way to hide these warnings? |
@nfailla You can downgrade or follow the instructions of the warning. |
Deprecation
The ExpansionPanel components were renamed to Accordion. The old imports will be removed in v5.
This PR cherry picks changes from #21494 to be merged in master.
Also cover #21630
### QuestionShould we revert the changes in the examples and docs API pages?