Skip to content

feat: add global loading states (search, filters, quiz, progress)#223

Merged
MrunaliniPachpute merged 2 commits into
MrunaliniPachpute:mainfrom
bavi404:loading-states
Oct 31, 2025
Merged

feat: add global loading states (search, filters, quiz, progress)#223
MrunaliniPachpute merged 2 commits into
MrunaliniPachpute:mainfrom
bavi404:loading-states

Conversation

@bavi404

@bavi404 bavi404 commented Oct 31, 2025

Copy link
Copy Markdown
Contributor

Resolves #211
#211

Adds minimal, reusable loading UX across the site.
CSS: .loader-center, .loader-inline, .skeleton-block, .skeleton-line, .skeleton-quiz-line, #quiz-full-loader
JS helpers: showLoader(target, type), hideLoader(target)
Search: spinner + skeletons, no input disabling, always clears
Filters: inline spinner by dropdown + skeletons, results fade in, no disabling
Quiz: full-page “Loading quiz…” overlay + skeleton lines
Progress: inline spinner inside .progress-update-btn
Robust cleanup: loaders always clear; cards always restored; “No topics found.” shown when appropriate.

image (885×427)

Screenshot 2025-10-31 at 11:28:29 AM (1512×857)

Screenshot 2025-10-31 at 11:32:16 AM (1512×862)

@github-actions

Copy link
Copy Markdown

Thanks for creating a PR for your Issue! ☺️

Kindly Star the repo ⭐ We'll review Your PR as soon as possible.
In the meantime, please double-check the file changes and ensure that all commits are accurate.

If there are any unresolved review comments, feel free to resolve them. 🙌🏼

@MrunaliniPachpute

Copy link
Copy Markdown
Owner

Kindlt resolve the merge conflicts.

@MrunaliniPachpute

Copy link
Copy Markdown
Owner

Well done @bavi404 . I see it is working good. If you could resolve merge conflicts asap I will merge this PR .

@bavi404

bavi404 commented Oct 31, 2025

Copy link
Copy Markdown
Contributor Author

Well done @bavi404 . I see it is working good. If you could resolve merge conflicts asap I will merge this PR .

hi, thank you so much. I have resolved them, can you check and see if there are any other issues?

@MrunaliniPachpute

Copy link
Copy Markdown
Owner

Well done!

@MrunaliniPachpute MrunaliniPachpute merged commit e21614b into MrunaliniPachpute:main Oct 31, 2025
@github-actions

Copy link
Copy Markdown

🎉 Congrats on getting your PR merged in, @bavi404! 🙌🏼

Thanks for your contribution every effort helps improve the project.

Looking forward to seeing more from you! 🥳✨

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.

Add Loading States

2 participants