-
Notifications
You must be signed in to change notification settings - Fork 3k
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
[GSoC2024] Enhancement of empty list components #7659
[GSoC2024] Enhancement of empty list components #7659
Conversation
@syedbarimanjan , Is it ready? Can we look at the PR? If so, please remove draft label. |
@klakhov could you take a look. |
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## develop #7659 +/- ##
========================================
Coverage 83.44% 83.44%
========================================
Files 373 373
Lines 39726 39716 -10
Branches 3741 3741
========================================
- Hits 33149 33142 -7
+ Misses 6577 6574 -3
|
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.
Please, add a changelog fragment
cvat-ui/src/components/cloud-storages-page/cloud-storages-page.tsx
Outdated
Show resolved
Hide resolved
Hi @syedbarimanjan , |
Yes I will push the changes today my pc got damaged thats why it took long. |
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.
Hi @syedbarimanjan,
I've added a couple more comments, please check them.
cvat-ui/src/components/cloud-storages-page/cloud-storages-page.tsx
Outdated
Show resolved
Hide resolved
…m/syedbarimanjan/cvat into enhancement/empty-list-components
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.
Thanks for the contribution!
Motivation and context
Fixes #7657 Made all of the empty-list components ui consistent.
Checklist
develop
branchLicense
Feel free to contact the maintainers if that's a concern.