From 3d7afe5197f64bca00c090893f2a84893133f300 Mon Sep 17 00:00:00 2001 From: eps1lon Date: Mon, 19 Jul 2021 11:20:26 +0200 Subject: [PATCH] [docs] Fix ExpansionPanel migration notes - Fix `focused` removal falsely pointing to `Accordion` (instead of `AccordionSummary`) - Fix some migration notes not having their own heading --- .../pages/guides/migration-v4/migration-v4.md | 21 ++++++++++++------- 1 file changed, 13 insertions(+), 8 deletions(-) diff --git a/docs/src/pages/guides/migration-v4/migration-v4.md b/docs/src/pages/guides/migration-v4/migration-v4.md index f7207784d36924..abad907b39d7cc 100644 --- a/docs/src/pages/guides/migration-v4/migration-v4.md +++ b/docs/src/pages/guides/migration-v4/migration-v4.md @@ -1243,21 +1243,26 @@ As the core components use emotion as their style engine, the props used by emot + {}} /> ``` +### ExpansionPanelDetails + +- Remove `display: flex` from `AccordionDetails` (formerly `ExpansionPanelDetails`) as its too opinionated. + Most developers expect a display block. + +### ExpansionPanelSummary + - Rename `focused` to `focusVisible` for consistency: ```diff - + }} + /> ``` -- Remove `display: flex` from AccordionDetails as its too opinionated. - Most developers expect a display block. -- Remove `IconButtonProps` prop from AccordionSummary. - The component renders a `
` element instead of an IconButton. +- Remove `IconButtonProps` prop from `AccordionSummary` (formerly `ExpansionPanelSummary`). + The component renders a `
` element instead of an `IconButton`. The prop is no longer necessary. ### Fab