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): hide light variant stepper pseudoelements on focus #8652

Conversation

emyarod
Copy link
Member

@emyarod emyarod commented May 11, 2021

Closes #8649

This PR updates the light number input styles so that hovered and focused stepper buttons are no longer missing the left and right focus borders

Testing / Reviewing

Verify that the light number input appears as expected

@netlify
Copy link

netlify bot commented May 11, 2021

Deploy preview for carbon-elements ready!

Built with commit dd3cf53

https://deploy-preview-8652--carbon-elements.netlify.app

@netlify
Copy link

netlify bot commented May 11, 2021

Deploy preview for carbon-components-react ready!

Built without sensitive environment variables with commit dd3cf53

https://deploy-preview-8652--carbon-components-react.netlify.app

Copy link
Contributor

@dakahn dakahn 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
Collaborator

@tw15egan tw15egan 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

@laurenmrice laurenmrice left a comment

Choose a reason for hiding this comment

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

Shows up correctly for light variant 👍

@kodiakhq kodiakhq bot merged commit d781c15 into carbon-design-system:main May 12, 2021
@emyarod emyarod deleted the 8649-numberinput-stepper-focus-hover branch May 13, 2021 15:23
@tw15egan tw15egan mentioned this pull request May 13, 2021
22 tasks
@emyarod emyarod mentioned this pull request May 17, 2021
94 tasks
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.

NumberInput: different button hover CSS for light option enabled and disabled
4 participants