-
Notifications
You must be signed in to change notification settings - Fork 0
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
Extract ui components from ImageUpload component #138
Conversation
Code Climate has analyzed commit e16ecc8 and detected 10 issues on this pull request. Here's the issue category breakdown:
The test coverage on the diff in this pull request is 45.0% (50% is the threshold). This pull request will bring the total coverage in the repository to 16.3% (3.5% change). View more on Code Climate. |
} | ||
|
||
.dropzone:focus { | ||
outline: none; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would this be a problem a11y wise?
|
||
type DropzoneProps = { | ||
label?: string | ||
[x: string]: any; // Plugin context determines which props are needed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would unknown maybe better iso of any?
Approved with some comments |
What
Explain what changes inside the code, this can be as simple or complicated as you want as long as it's clear
Why
Explain why these things are changes. This explanation is for your colleagues and your future self.
Code Review
Please consider the following checklist when reviewing this Pull Request.
More background and details here.