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

feat: Add TextField component #143

Merged
merged 12 commits into from
Jan 28, 2020
Merged

feat: Add TextField component #143

merged 12 commits into from
Jan 28, 2020

Conversation

mikemurray
Copy link
Member

@mikemurray mikemurray commented Jan 28, 2020

Resolves #140
Impact: minor
Type: feature

Component

This is a drop-in replacement for the standard MUI TextField component that defaults to the outlined style. Styles have been updated to match the Catalyst spec, otherwise, it's a standard MUI TextField under the hood.

This gives us all of its features including

  • Single line text input
  • Multiline text input
  • Select dropdown (works well for forms with validation)

Screenshots

image

image

Breaking changes

none

Testing

  1. Test the TextField components, mostly visually as the only real change is styling from MUI

Copy link

@cassytaylor cassytaylor left a comment

Choose a reason for hiding this comment

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

Looking good, Mike!! Thanks for jumping on this so quick yesterday! One super small thing I noticed - it looks like the help text is styled the same as the label text (or at least, it's the same color) - we have it styled a little lighter in the mocks as black-55/#737373 to create hierarchy between the label/help text. but other than that it looks great!!

Signed-off-by: Mike Murray <[email protected]>
Signed-off-by: Mike Murray <[email protected]>
Signed-off-by: Mike Murray <[email protected]>
Signed-off-by: Mike Murray <[email protected]>
Signed-off-by: Mike Murray <[email protected]>
@netlify
Copy link

netlify bot commented Jan 28, 2020

Deploy preview for heuristic-perlman-0ef024 ready!

Built with commit 64761d1

https://deploy-preview-143--heuristic-perlman-0ef024.netlify.com

Copy link
Member

@willopez willopez left a comment

Choose a reason for hiding this comment

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

LGTM

@willopez willopez merged commit 5e781bc into trunk Jan 28, 2020
@willopez willopez deleted the feat-140-mikemurray-textfield branch January 28, 2020 18:49
@rc-publisher
Copy link
Collaborator

🎉 This PR is included in version 1.17.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@rc-publisher rc-publisher added the released Applied automatically by semantic-release label Jan 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
released Applied automatically by semantic-release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add TextField/TextArea component
4 participants