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

[GSoC2024] Enhancement of empty list components #7659

Merged

Conversation

syedbarimanjan
Copy link
Contributor

@syedbarimanjan syedbarimanjan commented Mar 22, 2024

Motivation and context

Fixes #7657 Made all of the empty-list components ui consistent.

Checklist

  • I have created a changelog fragment
  • I submit my changes into the develop branch
  • I have linked related issues (see GitHub docs)

License

  • I submit my code changes under the same MIT License that covers the project.
    Feel free to contact the maintainers if that's a concern.

@nmanovic
Copy link
Contributor

@syedbarimanjan , Is it ready? Can we look at the PR? If so, please remove draft label.

@syedbarimanjan syedbarimanjan marked this pull request as ready for review March 24, 2024 02:55
@syedbarimanjan
Copy link
Contributor Author

@klakhov could you take a look.

Copy link

codecov bot commented Mar 25, 2024

Codecov Report

Merging #7659 (a828f15) into develop (7e2f9d3) will increase coverage by 0.00%.
Report is 8 commits behind head on develop.
The diff coverage is n/a.

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     
Components Coverage Δ
cvat-ui 79.24% <ø> (ø)
cvat-server 87.33% <ø> (+<0.01%) ⬆️

@klakhov klakhov added the ui/ux label Mar 26, 2024
@klakhov klakhov requested review from klakhov and removed request for bsekachev and azhavoro March 26, 2024 08:15
Copy link
Contributor

@klakhov klakhov left a 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

@klakhov
Copy link
Contributor

klakhov commented Apr 3, 2024

Hi @syedbarimanjan ,
Are you still going to work on this PR?

@syedbarimanjan
Copy link
Contributor Author

Hi @syedbarimanjan , Are you still going to work on this PR?

Yes I will push the changes today my pc got damaged thats why it took long.

Copy link
Contributor

@klakhov klakhov left a 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.

Copy link
Contributor

@klakhov klakhov left a 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!

@klakhov klakhov merged commit f1db0fd into cvat-ai:develop Apr 5, 2024
32 checks passed
@cvat-bot cvat-bot bot mentioned this pull request Apr 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Alignment Issue in Cloud Storage page
4 participants