Skip to content
This repository has been archived by the owner on Oct 19, 2021. It is now read-only.

refactor(Slider): replace Carbon TextInput #2085

Conversation

emyarod
Copy link
Member

@emyarod emyarod commented Mar 22, 2019

Closes IBM/carbon-components-react#2082

<TextInput> (and several other form components) include the <FormItem> wrapper out of the box. This PR replaces <TextInput> in the <Slider> component with native text inputs so that <Form> styles are not applied.

Related: https://github.com/IBM/carbon-components-react/issues/1681

@netlify
Copy link

netlify bot commented Mar 22, 2019

Deploy preview for carbon-components-react ready!

Built with commit c23b2c6

https://deploy-preview-2085--carbon-components-react.netlify.com

Copy link

@shixiedesign shixiedesign left a comment

Choose a reason for hiding this comment

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

Looks like visual's good. Thanks!

Copy link
Member

@alisonjoseph alisonjoseph left a comment

Choose a reason for hiding this comment

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

LGTM!

@alisonjoseph alisonjoseph merged commit a24a99a into carbon-design-system:master Mar 22, 2019
asudoh pushed a commit that referenced this pull request Mar 27, 2019
* feat(DataTable): Add sorting params to header onClick callback

* chore(build): update size threshold

* fix(progressIndicatorSkeleton): Update html to fix styling (#2084)

* fix: update skeleton svg for progress indicator

* chore: update html

* refactor(Slider): replace Carbon TextInput (#2085)

* feat(DataTable): Add sorting params to header onClick callback
@carbon-bot
Copy link

🎉 This PR is included in version 6.112.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@emyarod emyarod deleted the 2082-legacy-slider-component-broken-markup branch March 28, 2019 14:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants