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

Changed loading position to center #735

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

AyushShivhare79
Copy link
Contributor

PR Fixes:

  • 1 Position of loading to center

Resolves #734

Checklist before requesting a review

  • I have performed a self-review of my code
  • I assure there is no similar/duplicate pull request regarding same issue

While loading it is now on center

Screenshot from 2024-05-30 00-13-25

After loading

Screenshot from 2024-05-30 00-16-04

@Vasant79
Copy link

Vasant79 commented May 30, 2024

@AyushShivhare79, I think it's better to use the Loader component instead of just plain text.

@AyushShivhare79
Copy link
Contributor Author

AyushShivhare79 commented May 30, 2024

@Vasant79

This looks fine?

Screencast.from.30-05-24.08.26.50.PM.IST.webm

@AyushShivhare79
Copy link
Contributor Author

@hkirat Can you look at this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

bug: Small issue of loading position
2 participants