From 695b89dce85e4ff41fd1b9944c913a88b481bf39 Mon Sep 17 00:00:00 2001 From: rithik56 Date: Sat, 8 Nov 2025 15:36:36 +0530 Subject: [PATCH 1/6] =?UTF-8?q?[codemod]=20Fix=20incorrect=20slot=20name?= =?UTF-8?q?=20in=20tab-props.js=20(scrollButton=20=E2=86=92=20scrollButton?= =?UTF-8?q?s)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../mui-codemod/src/deprecations/tabs-props/tabs-props.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/packages/mui-codemod/src/deprecations/tabs-props/tabs-props.js b/packages/mui-codemod/src/deprecations/tabs-props/tabs-props.js index 665695f1fa47cc..44f513237cb668 100644 --- a/packages/mui-codemod/src/deprecations/tabs-props/tabs-props.js +++ b/packages/mui-codemod/src/deprecations/tabs-props/tabs-props.js @@ -15,14 +15,14 @@ export default function transformer(file, api, options) { packageName: options.packageName, componentName: 'Tabs', propName: 'ScrollButtonComponent', - slotName: 'scrollButton', + slotName: 'scrollButtons', }); movePropIntoSlotProps(j, { root, packageName: options.packageName, componentName: 'Tabs', propName: 'TabScrollButtonProps', - slotName: 'scrollButton', + slotName: 'scrollButtons', }); movePropIntoSlotProps(j, { From e754c034dbc307600e56d6fde764efd3ec904b6d Mon Sep 17 00:00:00 2001 From: rithik56 Date: Sat, 8 Nov 2025 17:03:48 +0530 Subject: [PATCH 2/6] fix(tabs-props): rename scrollButton to scrollButtons in test cases --- .../src/deprecations/tabs-props/test-cases/expected.js | 8 ++++---- .../tabs-props/test-cases/package.expected.js | 8 ++++---- .../deprecations/tabs-props/test-cases/theme.expected.js | 4 ++-- 3 files changed, 10 insertions(+), 10 deletions(-) diff --git a/packages/mui-codemod/src/deprecations/tabs-props/test-cases/expected.js b/packages/mui-codemod/src/deprecations/tabs-props/test-cases/expected.js index 2e14c180312f7b..6d3b5b7ff6d712 100644 --- a/packages/mui-codemod/src/deprecations/tabs-props/test-cases/expected.js +++ b/packages/mui-codemod/src/deprecations/tabs-props/test-cases/expected.js @@ -3,19 +3,19 @@ import { Tabs as MyTabs } from '@mui/material'; ; ; diff --git a/packages/mui-codemod/src/deprecations/tabs-props/test-cases/package.expected.js b/packages/mui-codemod/src/deprecations/tabs-props/test-cases/package.expected.js index b623e9cb808706..e847c67963947f 100644 --- a/packages/mui-codemod/src/deprecations/tabs-props/test-cases/package.expected.js +++ b/packages/mui-codemod/src/deprecations/tabs-props/test-cases/package.expected.js @@ -3,19 +3,19 @@ import { Tabs as MyTabs } from '@org/ui/material'; ; ; diff --git a/packages/mui-codemod/src/deprecations/tabs-props/test-cases/theme.expected.js b/packages/mui-codemod/src/deprecations/tabs-props/test-cases/theme.expected.js index 759db5e1873a8e..89db240f0a284b 100644 --- a/packages/mui-codemod/src/deprecations/tabs-props/test-cases/theme.expected.js +++ b/packages/mui-codemod/src/deprecations/tabs-props/test-cases/theme.expected.js @@ -2,11 +2,11 @@ fn({ MuiTabs: { defaultProps: { slots: { - scrollButton: CustomScrollButton + scrollButtons: CustomScrollButton }, slotProps: { - scrollButton: { disableRipple: true }, + scrollButtons: { disableRipple: true }, indicator: { className: 'indicator' } } }, From 1b003b3002e17bf4f0bef19ccca813122bf995d8 Mon Sep 17 00:00:00 2001 From: rithik56 Date: Tue, 11 Nov 2025 00:18:03 +0530 Subject: [PATCH 3/6] Trigger CI From f2fab7938460f502ce4d5d3a0634b58bd5becee6 Mon Sep 17 00:00:00 2001 From: ZeeshanTamboli Date: Sat, 13 Dec 2025 12:42:39 +0530 Subject: [PATCH 4/6] fix scrollButton to scrollButtons --- .../migrating-from-deprecated-apis.md | 8 ++++---- packages/mui-material/src/Tabs/Tabs.d.ts | 2 +- packages/mui-material/src/Tabs/Tabs.js | 2 +- 3 files changed, 6 insertions(+), 6 deletions(-) diff --git a/docs/data/material/migration/migrating-from-deprecated-apis/migrating-from-deprecated-apis.md b/docs/data/material/migration/migrating-from-deprecated-apis/migrating-from-deprecated-apis.md index c619dbc17b34db..a2546888769599 100644 --- a/docs/data/material/migration/migrating-from-deprecated-apis/migrating-from-deprecated-apis.md +++ b/docs/data/material/migration/migrating-from-deprecated-apis/migrating-from-deprecated-apis.md @@ -2289,23 +2289,23 @@ Here's how to migrate: ### ScrollButtonComponent -The Tabs's `ScrollButtonComponent` prop was deprecated in favor of `slots.scrollButton`: +The Tabs's `ScrollButtonComponent` prop was deprecated in favor of `slots.scrollButtons`: ```diff ``` ### TabScrollButtonProps -The Tabs's `TabScrollButtonProps` prop was deprecated in favor of `slotProps.scrollButton`: +The Tabs's `TabScrollButtonProps` prop was deprecated in favor of `slotProps.scrollButtons`: ```diff ``` diff --git a/packages/mui-material/src/Tabs/Tabs.d.ts b/packages/mui-material/src/Tabs/Tabs.d.ts index 24ec83124669b5..d215b32fbe8d70 100644 --- a/packages/mui-material/src/Tabs/Tabs.d.ts +++ b/packages/mui-material/src/Tabs/Tabs.d.ts @@ -95,7 +95,7 @@ export type TabsSlotsAndSlotProps = CreateSlotsAndSlotProps< */ indicator: SlotProps<'span', TabsIndicatorSlotPropsOverrides, TabsOwnerState>; /** - * Props forwarded to the scrollButton slot. + * Props forwarded to the scrollButtons slot. * By default, the available props are based on the [TabScrollButton](https://mui.com/material-ui/api/tab-scroll-button/#props) component. */ scrollButtons: SlotProps< diff --git a/packages/mui-material/src/Tabs/Tabs.js b/packages/mui-material/src/Tabs/Tabs.js index 22223d0bfd54bd..3f64a752883929 100644 --- a/packages/mui-material/src/Tabs/Tabs.js +++ b/packages/mui-material/src/Tabs/Tabs.js @@ -384,7 +384,7 @@ const Tabs = React.forwardRef(function Tabs(inProps, ref) { slots, slotProps: { indicator: TabIndicatorProps, - scrollButton: TabScrollButtonProps, + scrollButtons: TabScrollButtonProps, ...slotProps, }, }; From 466e418a0f1d8e9a54170e9d27132ee7b3cca2bc Mon Sep 17 00:00:00 2001 From: ZeeshanTamboli Date: Sat, 13 Dec 2025 13:40:21 +0530 Subject: [PATCH 5/6] rerun CI From 2deda0a0061816c01ba33389c62de06a206cba79 Mon Sep 17 00:00:00 2001 From: ZeeshanTamboli Date: Sat, 13 Dec 2025 14:48:35 +0530 Subject: [PATCH 6/6] update in test --- packages/mui-material/src/Tabs/Tabs.test.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/mui-material/src/Tabs/Tabs.test.js b/packages/mui-material/src/Tabs/Tabs.test.js index 7bfc8d6a016a65..4242b9fefe5446 100644 --- a/packages/mui-material/src/Tabs/Tabs.test.js +++ b/packages/mui-material/src/Tabs/Tabs.test.js @@ -1555,7 +1555,7 @@ describe.skipIf(isSafari)('', () => { }); }); - describe('scrollButton slot', () => { + describe('scrollButtons slot', () => { it('should render start and end scroll buttons', () => { render(