[ExpansionPanel] Rename to Accordion, add warning to current exports#21560
[ExpansionPanel] Rename to Accordion, add warning to current exports#21560mnajdova merged 24 commits intomui:masterfrom
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?
|
oliviertassinari
left a comment
There was a problem hiding this comment.
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
oliviertassinari
left a comment
There was a problem hiding this comment.
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?