Skip to content

[material-ui][Stepper] Generate class for nonLinear prop #42620

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.

Sign up for GitHub

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

Merged
ZeeshanTamboli merged 11 commits into from
Jun 18, 2024
Merged

[material-ui][Stepper] Generate class for nonLinear prop #42620

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
24607ab
feat: nonLinear appears in Stepper utility classes
alexismo Jun 11, 2024
025e2a4
generate docs for new class
alexismo Jun 13, 2024
8b31ed1
Merge remote-tracking branch 'upstream/next' into stepper-nonlinear-c…
alexismo Jun 13, 2024
7deec59
add tests
alexismo Jun 13, 2024
f9e6969
use already imported classes, per review
alexismo Jun 13, 2024
f804561
remove unnecessary test
alexismo Jun 13, 2024
b65e39c
add nonLinear to Stepper styleOverrides
alexismo Jun 14, 2024
54f4876
remove un-used const
alexismo Jun 15, 2024
eb86f77
Merge branch 'next' into stepper-nonlinear-css-class
ZeeshanTamboli Jun 18, 2024
741e1cb
change wording of test description
ZeeshanTamboli Jun 18, 2024
19eb8c7
prettier
ZeeshanTamboli Jun 18, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
19 changes: 18 additions & 1 deletion packages/mui-material/src/Stepper/Stepper.test.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ import Step, { StepProps, stepClasses } from '@mui/material/Step';
import StepLabel from '@mui/material/StepLabel';
import StepConnector, { stepConnectorClasses } from '@mui/material/StepConnector';
import StepContent, { stepContentClasses } from '@mui/material/StepContent';
import Stepper, { stepperClasses as classes } from '@mui/material/Stepper';
import Stepper, { stepperClasses as classes, stepperClasses } from '@mui/material/Stepper';
Copy link
Contributor

@sai6855 sai6855 Jun 13, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

stepperClasses is already imported and named as classes, so you can use classes instead of importing stepperClasses again

import describeConformance from '../../test/describeConformance';

describe('<Stepper />', () => {
Expand Down Expand Up @@ -258,4 +258,21 @@ describe('<Stepper />', () => {
expect(stepContent[0]).not.to.have.class(stepContentClasses.last);
expect(stepContent[1]).to.have.class(stepContentClasses.last);
});

it('is applies non-linear styling', () => {
const { container, setProps } = render(
<Stepper nonLinear activeStep={0}>
<Step />
<Step />
<Step />
</Stepper>,
);

const stepper = container.querySelector(`.${stepperClasses.root}`);
expect(stepper).to.have.class(stepperClasses.nonLinear)

setProps({ alternativeLabel: true });

expect(stepper).to.have.class(stepperClasses.nonLinear)
});
});