Skip to content

Commit

Permalink
feat(Checkbox/Radio): replace isLabelBeforeButton with labelPosition=…
Browse files Browse the repository at this point in the history
…"start"
  • Loading branch information
adamviktora committed Feb 21, 2024
1 parent 1ded4ca commit f5ea9a0
Show file tree
Hide file tree
Showing 5 changed files with 79 additions and 0 deletions.
Original file line number Diff line number Diff line change
@@ -0,0 +1,17 @@
### checkbox-radio-replace-isLabelBeforeButton [(#10016)](https://github.com/patternfly/patternfly-react/pull/10016)

The `isLabelBeforeButton` prop in Checkbox and Radio has been replaced with `labelPosition="start"`

#### Examples

In:

```jsx
%inputExample%
```

Out:

```jsx
%outputExample%
```
Original file line number Diff line number Diff line change
@@ -0,0 +1,29 @@
const ruleTester = require("../../ruletester");
import * as rule from "./checkbox-radio-replace-isLabelBeforeButton";

ruleTester.run("checkbox-radio-replace-isLabelBeforeButton", rule, {
valid: [
{
code: `<Checkbox isLabelBeforeButton />`,
},
{
code: `import { Checkbox } from '@patternfly/react-core'; <Checkbox someOtherProp />`,
},
{
code: `<Radio isLabelBeforeButton />`,
},
{
code: `import { Radio } from '@patternfly/react-core'; <Radio someOtherProp />`,
},
],
invalid: ["Checkbox", "Radio"].map((component) => ({
code: `import { ${component} } from '@patternfly/react-core'; <${component} isLabelBeforeButton />`,
output: `import { ${component} } from '@patternfly/react-core'; <${component} labelPosition="start" />`,
errors: [
{
message: `isLabelBeforeButton prop for ${component} has been replaced with labelPosition="start"`,
type: "JSXOpeningElement",
},
],
})),
});
Original file line number Diff line number Diff line change
@@ -0,0 +1,17 @@
import { renameProps } from "../../helpers";

const renames = {
isLabelBeforeButton: {
newName: 'labelPosition="start"',
replace: true,
},
};

// https://github.com/patternfly/patternfly-react/pull/10016
module.exports = {
meta: { fixable: "code" },
create: renameProps({
Checkbox: renames,
Radio: renames,
}),
};
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
import { Checkbox, Radio } from "@patternfly/react-core";

export const CheckboxReplaceIsLabelBeforeButtonInput = () => (
<Checkbox isLabelBeforeButton />
);
export const RadioReplaceIsLabelBeforeButtonInput = () => (
<Radio isLabelBeforeButton />
);
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
import { Checkbox, Radio } from "@patternfly/react-core";

export const CheckboxReplaceIsLabelBeforeButtonInput = () => (
<Checkbox labelPosition="start" />
);
export const RadioReplaceIsLabelBeforeButtonInput = () => (
<Radio labelPosition="start" />
);

0 comments on commit f5ea9a0

Please sign in to comment.