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

fix(number-input): fix positioning when no label is present #5376

Merged

Conversation

tw15egan
Copy link
Member

Closes #5358

Fixes positioning issues when no label is present

Changelog

Changed

  • Removed negative margin-top on helper text when the label is hidden

Testing / Reviewing

Remove label text, and check hide label. Add text in while hide label is active and make sure the number input container does not move

@tw15egan tw15egan requested a review from a team as a code owner February 18, 2020 13:46
@ghost ghost requested review from aledavila and joshblack February 18, 2020 13:46
@netlify
Copy link

netlify bot commented Feb 18, 2020

Deploy preview for carbon-elements ready!

Built with commit 6d1a459

https://deploy-preview-5376--carbon-elements.netlify.com

@netlify
Copy link

netlify bot commented Feb 18, 2020

Deploy preview for carbon-components-react ready!

Built with commit 6d1a459

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

@netlify
Copy link

netlify bot commented Feb 18, 2020

Deploy preview for carbon-elements ready!

Built with commit 08f7d5b

https://deploy-preview-5376--carbon-elements.netlify.com

@netlify
Copy link

netlify bot commented Feb 18, 2020

Deploy preview for carbon-components-react ready!

Built with commit 08f7d5b

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

Copy link
Contributor

@designertyler designertyler left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Member

@emyarod emyarod left a comment

Choose a reason for hiding this comment

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

looks good to me

Copy link
Contributor

@asudoh asudoh left a comment

Choose a reason for hiding this comment

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

LGTM 👍 - Thanks @tw15egan!

@asudoh asudoh merged commit 18f3c83 into carbon-design-system:master Feb 20, 2020
@tw15egan tw15egan deleted the number-input-position-fix branch April 28, 2021 18:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Number Input] incorrect position with no label
6 participants