Skip to content

Commit

Permalink
Remove loading prop from Form.Textarea and Form.Input
Browse files Browse the repository at this point in the history
  • Loading branch information
kennethnym committed Jan 2, 2021
1 parent 8eaf9bf commit f927308
Show file tree
Hide file tree
Showing 5 changed files with 2 additions and 16 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ exports[`Textarea component Should have textarea classname 1`] = `

exports[`Textarea component Should support various states 1`] = `
<textarea
className="textarea is-loading is-focused is-hovered"
className="textarea is-focused is-hovered"
/>
`;

Expand Down
2 changes: 1 addition & 1 deletion src/components/form/components/__test__/textarea.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ describe('Textarea component', () => {
expect(component.toJSON()).toMatchSnapshot();
});
it('Should support various states', () => {
const component = renderer.create(<Textarea loading focused hovered />);
const component = renderer.create(<Textarea focused hovered />);
expect(component.toJSON()).toMatchSnapshot();
});
it('Should have a fixed size', () => {
Expand Down
5 changes: 0 additions & 5 deletions src/components/form/components/input.js
Original file line number Diff line number Diff line change
Expand Up @@ -53,10 +53,6 @@ Input.propTypes = {
* are removed.
*/
isStatic: PropTypes.bool,
/**
* Whether this input is in a loading state.
*/
loading: PropTypes.bool,
/**
* Whether this input is in a focused state.
*/
Expand Down Expand Up @@ -87,7 +83,6 @@ Input.defaultProps = {
focused: false,
hovered: false,
rounded: false,
loading: false,
renderAs: 'input',
};

Expand Down
7 changes: 0 additions & 7 deletions src/components/form/components/textarea.js
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,6 @@ const Textarea = ({
className,
size,
color,
loading,
focused,
hovered,
fixedSize,
Expand All @@ -21,7 +20,6 @@ const Textarea = ({
<Element
{...props}
className={classnames('textarea', className, {
'is-loading': loading,
'is-focused': focused,
'is-hovered': hovered,
'has-fixed-size': fixedSize,
Expand All @@ -47,10 +45,6 @@ Textarea.propTypes = {
* The color of this input.
*/
color: PropTypes.oneOf(colors),
/**
* Whether this textarea is in a loading state.
*/
loading: PropTypes.bool,
/**
* Whether this textarea is in a focused state.
*/
Expand All @@ -77,7 +71,6 @@ Textarea.defaultProps = {
size: undefined,
focused: false,
hovered: false,
loading: false,
fixedSize: false,
color: undefined,
};
Expand Down
2 changes: 0 additions & 2 deletions src/components/form/index.d.ts
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,6 @@ interface InputProps<T> {
color?: Color;
readOnly?: boolean;
isStatic?: boolean;
loading?: boolean;
focused?: boolean;
hovered?: boolean;
value?: T;
Expand All @@ -46,7 +45,6 @@ interface TextareaProps {
color?: Color;
focused?: boolean;
hovered?: boolean;
loading?: boolean;
fixedSize?: boolean;
}

Expand Down

0 comments on commit f927308

Please sign in to comment.