-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Add loading indicator and no results message for search #4824
Conversation
This pull request has merge conflicts. Please resolve the conflicts so the PR can be successfully reviewed and merged. |
with native <Loading />
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.
Addition of
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.
Resolved Issues.
This pull request has merge conflicts. Please resolve the conflicts so the PR can be successfully reviewed and merged. |
Seems to be all good to go |
This pull request has merge conflicts. Please resolve the conflicts so the PR can be successfully reviewed and merged. |
Change SearchResultsEmpty to have a substitution Co-authored-by: Bill Thornton <thornbill@users.noreply.github.com>
Update SearchResults Co-authored-by: Bill Thornton <thornbill@users.noreply.github.com>
Oops! Co-authored-by: Bill Thornton <thornbill@users.noreply.github.com>
Co-authored-by: Bill Thornton <thornbill@users.noreply.github.com>
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Cloudflare Pages deployment
|
Resolves a search not returning anything with some text. A loading circle was also added in case search takes a long while.
Changes
Text added. Example:
Loading circle added (may be hard to spot since search is fast)
Feedback is more than welcome!
Recording.2023-10-01.181554.mp4