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(date-picker): remove redundant date picker border radius #8259

Conversation

emyarod
Copy link
Member

@emyarod emyarod commented Mar 31, 2021

Related #8190
Related #8198

This PR removes a redundant border-radius rule on the flatpickr styles

the flatpickr styles are not being linted, should I enable the linter?

Testing / Reviewing

The date picker calendar should not have a border radius value

@github-actions
Copy link
Contributor

github-actions bot commented Mar 31, 2021

DCO Assistant Lite bot All contributors have signed the DCO.

@netlify
Copy link

netlify bot commented Mar 31, 2021

Deploy preview for carbon-elements ready!

Built with commit 8ee063c

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

@netlify
Copy link

netlify bot commented Mar 31, 2021

Deploy preview for carbon-components-react ready!

Built without sensitive environment variables with commit 8ee063c

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

@tw15egan
Copy link
Collaborator

tw15egan commented Apr 1, 2021

@emyarod I think enabling the linting would be beneficial, but doesn't need to be bundled in here if you don't want to comb through the warnings

@jnm2377 jnm2377 requested review from a team and johnbister and removed request for a team April 2, 2021 21:45
@emyarod
Copy link
Member Author

emyarod commented Apr 5, 2021

I have read the DCO document and I hereby sign the DCO.

@kodiakhq kodiakhq bot merged commit a067cfc into carbon-design-system:main Apr 5, 2021
@emyarod emyarod deleted the date-picker-redundant-border-radius branch April 5, 2021 19:58
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.

5 participants