Skip to content

Commit

Permalink
feat(text-input): change prop naming convention
Browse files Browse the repository at this point in the history
  • Loading branch information
tw15egan committed Jan 17, 2020
1 parent df69045 commit e4bfbb0
Show file tree
Hide file tree
Showing 5 changed files with 11 additions and 11 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -44,11 +44,11 @@
}
}

.#{$prefix}--text-input--xl {
.#{$prefix}--text-input--large {
height: rem(48px);
}

.#{$prefix}--text-input--sm {
.#{$prefix}--text-input--small {
height: rem(32px);
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -222,7 +222,7 @@ ControlledPasswordInput.propTypes = {
tooltipAlignment: PropTypes.oneOf(['start', 'center', 'end']),

/**
* Specify the size of the Text Input. Currently supports either `sm`, `lg` or `xl` as an option.
* Specify the size of the Text Input. Currently supports either `small` or `large` as an option. If omitted, defaults to standard size
*/
size: PropTypes.string,
};
Expand All @@ -236,7 +236,7 @@ ControlledPasswordInput.defaultProps = {
invalidText: '',
helperText: '',
light: false,
size: 'lg',
size: '',
};

export default ControlledPasswordInput;
4 changes: 2 additions & 2 deletions packages/react/src/components/TextInput/PasswordInput.js
Original file line number Diff line number Diff line change
Expand Up @@ -227,7 +227,7 @@ PasswordInput.propTypes = {
showPasswordLabel: PropTypes.string,

/**
* Specify the size of the Text Input. Currently supports either `sm`, `lg` or `xl` as an option.
* Specify the size of the Text Input. Currently supports either `small` or `large` as an option. If omitted, defaults to standard size
*/
size: PropTypes.string,
};
Expand All @@ -241,5 +241,5 @@ PasswordInput.defaultProps = {
invalidText: '',
helperText: '',
light: false,
size: 'lg',
size: '',
};
6 changes: 3 additions & 3 deletions packages/react/src/components/TextInput/TextInput-story.js
Original file line number Diff line number Diff line change
Expand Up @@ -20,9 +20,9 @@ const types = {
};

const sizes = {
'Extra large size (xl)': 'xl',
'Regular size (lg)': '',
'Small size (sm)': 'sm',
Large: 'large',
Default: '',
Small: 'small',
};

function ControlledPasswordInputApp(props) {
Expand Down
4 changes: 2 additions & 2 deletions packages/react/src/components/TextInput/TextInput.js
Original file line number Diff line number Diff line change
Expand Up @@ -146,7 +146,7 @@ TextInput.propTypes = {
placeholder: PropTypes.string,

/**
* Specify the size of the Text Input. Currently supports either `sm`, `lg` or `xl` as an option.
* Specify the size of the Text Input. Currently supports either `small` or `large` as an option. If omitted, defaults to standard size
*/
size: PropTypes.string,

Expand Down Expand Up @@ -195,7 +195,7 @@ TextInput.defaultProps = {
invalidText: '',
helperText: '',
light: false,
size: 'lg',
size: '',
};

export default TextInput;

0 comments on commit e4bfbb0

Please sign in to comment.