Skip to content

Commit c3937a9

Browse files
authored
[ImageListItemBar] Remove deprecated CSS classes (#48064)
1 parent 1da1624 commit c3937a9

6 files changed

Lines changed: 24 additions & 101 deletions

File tree

docs/data/material/migration/upgrade-to-v9/upgrade-to-v9.md

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1016,6 +1016,21 @@ Use `sx={{ opacity : "0.6" }}` (or any opacity):
10161016
/>
10171017
```
10181018

1019+
#### ImageListItemBar deprecated CSS classes removed
1020+
1021+
Use the [image-list-item-bar-classes codemod](https://github.com/mui/material-ui/tree/HEAD/packages/mui-codemod#image-list-item-bar-classes) below to migrate the code as described in the following section:
1022+
1023+
```bash
1024+
npx @mui/codemod@latest deprecations/image-list-item-bar-classes <path>
1025+
```
1026+
1027+
The following deprecated `ImageListItemBar` CSS classes have been removed:
1028+
1029+
- `titleWrapBelow` → use `.MuiImageListItemBar-titleWrap` with `.MuiImageListItemBar-positionBelow` on the root
1030+
- `titleWrapActionPosLeft` → use `.MuiImageListItemBar-titleWrap` with `.MuiImageListItemBar-actionPositionLeft` on the root
1031+
- `titleWrapActionPosRight` → use `.MuiImageListItemBar-titleWrap` with `.MuiImageListItemBar-actionPositionRight` on the root
1032+
- `actionIconActionPosLeft` → use `.MuiImageListItemBar-actionIcon` with `.MuiImageListItemBar-actionPositionLeft` on the root
1033+
10191034
#### FormControlLabel deprecated props removed
10201035

10211036
Use the [form-control-label-props codemod](https://github.com/mui/material-ui/tree/HEAD/packages/mui-codemod#form-control-label-props) below to migrate the code as described in the following section:

docs/pages/material-ui/api/image-list-item-bar.json

Lines changed: 0 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -35,13 +35,6 @@
3535
"description": "Styles applied to the actionIcon if supplied.",
3636
"isGlobal": false
3737
},
38-
{
39-
"key": "actionIconActionPosLeft",
40-
"className": "MuiImageListItemBar-actionIconActionPosLeft",
41-
"description": "Styles applied to the actionIcon if `actionPosition=\"left\"`.",
42-
"isGlobal": false,
43-
"isDeprecated": true
44-
},
4538
{
4639
"key": "actionPositionLeft",
4740
"className": "MuiImageListItemBar-actionPositionLeft",
@@ -95,27 +88,6 @@
9588
"className": "MuiImageListItemBar-titleWrap",
9689
"description": "Styles applied to the title and subtitle container element.",
9790
"isGlobal": false
98-
},
99-
{
100-
"key": "titleWrapActionPosLeft",
101-
"className": "MuiImageListItemBar-titleWrapActionPosLeft",
102-
"description": "Styles applied to the container element if `actionPosition=\"left\"`.",
103-
"isGlobal": false,
104-
"isDeprecated": true
105-
},
106-
{
107-
"key": "titleWrapActionPosRight",
108-
"className": "MuiImageListItemBar-titleWrapActionPosRight",
109-
"description": "Styles applied to the container element if `actionPosition=\"right\"`.",
110-
"isGlobal": false,
111-
"isDeprecated": true
112-
},
113-
{
114-
"key": "titleWrapBelow",
115-
"className": "MuiImageListItemBar-titleWrapBelow",
116-
"description": "Styles applied to the title and subtitle container element if `position=\"below\"`.",
117-
"isGlobal": false,
118-
"isDeprecated": true
11991
}
12092
],
12193
"spread": true,

docs/translations/api-docs/image-list-item-bar/image-list-item-bar.json

Lines changed: 0 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -19,12 +19,6 @@
1919
"nodeName": "the actionIcon",
2020
"conditions": "supplied"
2121
},
22-
"actionIconActionPosLeft": {
23-
"description": "Styles applied to {{nodeName}} if {{conditions}}.",
24-
"nodeName": "the actionIcon",
25-
"conditions": "<code>actionPosition=\"left\"</code>",
26-
"deprecationInfo": "Combine the <a href=\"/material-ui/api/image-list-item-bar/#image-list-item-bar-classes-MuiImageListItemBar-actionIcon\">.MuiImageListItemBar-actionIcon</a> and <a href=\"/material-ui/api/image-list-item-bar/#image-list-item-bar-classes-MuiImageListItemBar-actionPositionLeft\">.MuiImageListItemBar-actionPositionLeft</a> classes instead. See <a href=\"/material-ui/migration/migrating-from-deprecated-apis/\">Migrating from deprecated APIs</a> for more details."
27-
},
2822
"actionPositionLeft": {
2923
"description": "Styles applied to {{nodeName}} if {{conditions}}.",
3024
"nodeName": "the action container element",
@@ -62,24 +56,6 @@
6256
"titleWrap": {
6357
"description": "Styles applied to {{nodeName}}.",
6458
"nodeName": "the title and subtitle container element"
65-
},
66-
"titleWrapActionPosLeft": {
67-
"description": "Styles applied to {{nodeName}} if {{conditions}}.",
68-
"nodeName": "the container element",
69-
"conditions": "<code>actionPosition=\"left\"</code>",
70-
"deprecationInfo": "Combine the <a href=\"/material-ui/api/image-list-item-bar/#image-list-item-bar-classes-MuiImageListItemBar-titleWrap\">.MuiImageListItemBar-titleWrap</a> and <a href=\"/material-ui/api/image-list-item-bar/#image-list-item-bar-classes-MuiImageListItemBar-actionPositionLeft\">.MuiImageListItemBar-actionPositionLeft</a> classes instead. See <a href=\"/material-ui/migration/migrating-from-deprecated-apis/\">Migrating from deprecated APIs</a> for more details."
71-
},
72-
"titleWrapActionPosRight": {
73-
"description": "Styles applied to {{nodeName}} if {{conditions}}.",
74-
"nodeName": "the container element",
75-
"conditions": "<code>actionPosition=\"right\"</code>",
76-
"deprecationInfo": "Combine the <a href=\"/material-ui/api/image-list-item-bar/#image-list-item-bar-classes-MuiImageListItemBar-titleWrap\">.MuiImageListItemBar-titleWrap</a> and <a href=\"/material-ui/api/image-list-item-bar/#image-list-item-bar-classes-MuiImageListItemBar-actionPositionRight\">.MuiImageListItemBar-actionPositionRight</a> classes instead. See <a href=\"/material-ui/migration/migrating-from-deprecated-apis/\">Migrating from deprecated APIs</a> for more details."
77-
},
78-
"titleWrapBelow": {
79-
"description": "Styles applied to {{nodeName}} if {{conditions}}.",
80-
"nodeName": "the title and subtitle container element",
81-
"conditions": "<code>position=\"below\"</code>",
82-
"deprecationInfo": "Combine the <a href=\"/material-ui/api/image-list-item-bar/#image-list-item-bar-classes-MuiImageListItemBar-titleWrap\">.MuiImageListItemBar-titleWrap</a> and <a href=\"/material-ui/api/image-list-item-bar/#image-list-item-bar-classes-MuiImageListItemBar-positionBelow\">.MuiImageListItemBar-positionBelow</a> classes instead. See <a href=\"/material-ui/migration/migrating-from-deprecated-apis/\">Migrating from deprecated APIs</a> for more details."
8359
}
8460
}
8561
}

packages/mui-material/src/ImageListItemBar/ImageListItemBar.js

Lines changed: 5 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -10,22 +10,18 @@ import capitalize from '../utils/capitalize';
1010
import { getImageListItemBarUtilityClass } from './imageListItemBarClasses';
1111

1212
const useUtilityClasses = (ownerState) => {
13-
const { classes, position, actionIcon, actionPosition } = ownerState;
13+
const { classes, position, actionPosition } = ownerState;
1414

1515
const slots = {
1616
root: [
1717
'root',
1818
`position${capitalize(position)}`,
1919
`actionPosition${capitalize(actionPosition)}`,
2020
],
21-
titleWrap: [
22-
'titleWrap',
23-
`titleWrap${capitalize(position)}`,
24-
actionIcon && `titleWrapActionPos${capitalize(actionPosition)}`,
25-
],
21+
titleWrap: ['titleWrap'],
2622
title: ['title'],
2723
subtitle: ['subtitle'],
28-
actionIcon: ['actionIcon', `actionIconActionPos${capitalize(actionPosition)}`],
24+
actionIcon: ['actionIcon'],
2925
};
3026

3127
return composeClasses(slots, getImageListItemBarUtilityClass, classes);
@@ -85,13 +81,7 @@ const ImageListItemBarTitleWrap = styled('div', {
8581
name: 'MuiImageListItemBar',
8682
slot: 'TitleWrap',
8783
overridesResolver: (props, styles) => {
88-
const { ownerState } = props;
89-
90-
return [
91-
styles.titleWrap,
92-
styles[`titleWrap${capitalize(ownerState.position)}`],
93-
ownerState.actionIcon && styles[`titleWrapActionPos${capitalize(ownerState.actionPosition)}`],
94-
];
84+
return [styles.titleWrap];
9585
},
9686
})(
9787
memoTheme(({ theme }) => {
@@ -161,12 +151,7 @@ const ImageListItemBarActionIcon = styled('div', {
161151
name: 'MuiImageListItemBar',
162152
slot: 'ActionIcon',
163153
overridesResolver: (props, styles) => {
164-
const { ownerState } = props;
165-
166-
return [
167-
styles.actionIcon,
168-
styles[`actionIconActionPos${capitalize(ownerState.actionPosition)}`],
169-
];
154+
return [styles.actionIcon];
170155
},
171156
})({
172157
variants: [

packages/mui-material/src/ImageListItemBar/ImageListItemBar.test.js

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -76,25 +76,23 @@ describe('<ImageListItemBar />', () => {
7676
expect(container.querySelector('div')).to.have.class(classes.positionBelow);
7777
});
7878

79-
it('should render a child div with the titleWrapBelow class', () => {
79+
it('should render the titleWrap class and positionBelow on the root', () => {
8080
const { container } = render(<ImageListItemBar title="text" position="below" />);
8181

82+
expect(container.querySelector('div')).to.have.class(classes.positionBelow);
8283
expect(container.firstChild.querySelector('div')).to.have.class(classes.titleWrap);
83-
expect(container.firstChild.querySelector('div')).to.have.class(classes.titleWrapBelow);
8484
});
8585
});
8686
});
8787

8888
describe('prop: actionPosition', () => {
89-
it('should render a child div with the titleWrapActionPosLeft class', () => {
89+
it('should render the titleWrap class and actionPositionLeft on the root', () => {
9090
const { container } = render(
9191
<ImageListItemBar title="text" actionIcon={<div />} actionPosition="left" />,
9292
);
9393

94+
expect(container.querySelector('div')).to.have.class(classes.actionPositionLeft);
9495
expect(container.firstChild.querySelector('div')).to.have.class(classes.titleWrap);
95-
expect(container.firstChild.querySelector('div')).to.have.class(
96-
classes.titleWrapActionPosLeft,
97-
);
9896
});
9997
});
10098
});

packages/mui-material/src/ImageListItemBar/imageListItemBarClasses.ts

Lines changed: 0 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -16,28 +16,12 @@ export interface ImageListItemBarClasses {
1616
actionPositionRight: string;
1717
/** Styles applied to the title and subtitle container element. */
1818
titleWrap: string;
19-
/** Styles applied to the title and subtitle container element if `position="below"`.
20-
* @deprecated Combine the [.MuiImageListItemBar-titleWrap](/material-ui/api/image-list-item-bar/#image-list-item-bar-classes-MuiImageListItemBar-titleWrap) and [.MuiImageListItemBar-positionBelow](/material-ui/api/image-list-item-bar/#image-list-item-bar-classes-MuiImageListItemBar-positionBelow) classes instead. See [Migrating from deprecated APIs](/material-ui/migration/migrating-from-deprecated-apis/) for more details.
21-
*/
22-
titleWrapBelow: string;
23-
/** Styles applied to the container element if `actionPosition="left"`.
24-
* @deprecated Combine the [.MuiImageListItemBar-titleWrap](/material-ui/api/image-list-item-bar/#image-list-item-bar-classes-MuiImageListItemBar-titleWrap) and [.MuiImageListItemBar-actionPositionLeft](/material-ui/api/image-list-item-bar/#image-list-item-bar-classes-MuiImageListItemBar-actionPositionLeft) classes instead. See [Migrating from deprecated APIs](/material-ui/migration/migrating-from-deprecated-apis/) for more details.
25-
*/
26-
titleWrapActionPosLeft: string;
27-
/** Styles applied to the container element if `actionPosition="right"`.
28-
* @deprecated Combine the [.MuiImageListItemBar-titleWrap](/material-ui/api/image-list-item-bar/#image-list-item-bar-classes-MuiImageListItemBar-titleWrap) and [.MuiImageListItemBar-actionPositionRight](/material-ui/api/image-list-item-bar/#image-list-item-bar-classes-MuiImageListItemBar-actionPositionRight) classes instead. See [Migrating from deprecated APIs](/material-ui/migration/migrating-from-deprecated-apis/) for more details.
29-
*/
30-
titleWrapActionPosRight: string;
3119
/** Styles applied to the title container element. */
3220
title: string;
3321
/** Styles applied to the subtitle container element. */
3422
subtitle: string;
3523
/** Styles applied to the actionIcon if supplied. */
3624
actionIcon: string;
37-
/** Styles applied to the actionIcon if `actionPosition="left"`.
38-
* @deprecated Combine the [.MuiImageListItemBar-actionIcon](/material-ui/api/image-list-item-bar/#image-list-item-bar-classes-MuiImageListItemBar-actionIcon) and [.MuiImageListItemBar-actionPositionLeft](/material-ui/api/image-list-item-bar/#image-list-item-bar-classes-MuiImageListItemBar-actionPositionLeft) classes instead. See [Migrating from deprecated APIs](/material-ui/migration/migrating-from-deprecated-apis/) for more details.
39-
*/
40-
actionIconActionPosLeft: string;
4125
}
4226

4327
export type ImageListItemBarClassKey = keyof ImageListItemBarClasses;
@@ -56,16 +40,9 @@ const imageListItemBarClasses: ImageListItemBarClasses = generateUtilityClasses(
5640
'actionPositionLeft',
5741
'actionPositionRight',
5842
'titleWrap',
59-
'titleWrapBottom',
60-
'titleWrapTop',
61-
'titleWrapBelow',
62-
'titleWrapActionPosLeft',
63-
'titleWrapActionPosRight',
6443
'title',
6544
'subtitle',
6645
'actionIcon',
67-
'actionIconActionPosLeft',
68-
'actionIconActionPosRight',
6946
],
7047
);
7148

0 commit comments

Comments
 (0)