Skip to content
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.

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

ui v2: add end adornment support to text fields #776

Merged
merged 4 commits into from
Dec 11, 2020
Merged

Conversation

dschaller
Copy link
Contributor

Description

Add support to text fields for end adornment

Screen Shot 2020-12-10 at 2 44 43 PM

Testing Performed

In dev app and storybook

@dschaller dschaller requested a review from a team as a code owner December 10, 2020 22:45
frontend/packages/core/src/Input/text-field.tsx Outdated Show resolved Hide resolved
frontend/packages/core/src/Input/text-field.tsx Outdated Show resolved Hide resolved
InputProps={{ readOnly }}
InputProps={{
readOnly,
endAdornment: endAdornment && <IconButton type="submit">{endAdornment}</IconButton>,
Copy link
Collaborator

Choose a reason for hiding this comment

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

at first glance i wonder if we should take IconButton instead of wrapping it ourselves for further customization, this seems fine though.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I had that thought too but I can't imagine a situation where we wouldn't want an icon button. Also, the styling applied is overriding some of the icon buttons styles which by default provide a border radius of 50% and have padding

@dschaller dschaller merged commit ca43472 into UIV2 Dec 11, 2020
@dschaller dschaller deleted the textfieldadornment branch December 11, 2020 18:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants