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

fix: disable progress bar if search results are empty (DSP-1575) #442

Merged
merged 6 commits into from May 26, 2021

Conversation

waychal
Copy link
Contributor

@waychal waychal commented May 19, 2021

resolves DSP-1575

@flavens When page is refreshed with search string in the url, it displays search results on the page but same search string is not displayed in the search input box.

This bug requires changes in dsp-ui-lib and can be fixed in separate PR.

Screenshot 2021-05-19 at 13 25 50

@waychal waychal added the bug A bug fix label May 19, 2021
@waychal waychal requested a review from flavens May 19, 2021 11:28
Copy link
Contributor

@flavens flavens left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks good for the fulltext search - just a little css position to change. However, I have tried the advanced search and here, nothing: no result, no message, no console log.. It may not be related but it should be investigated in case the display problem comes from this PR.


<!-- In case of 0 result -->
<div class="no-results" *ngIf="!loading && numberOfAllResults === 0">
<p>Your search <span *ngIf="searchMode === 'fulltext'">- <strong> {{searchQuery}}</strong> -</span> did not match any documents.</p>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this message should be centered in the middle of the page instead of being displayed on the left side

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

@waychal
Copy link
Contributor Author

waychal commented May 25, 2021

It looks good for the fulltext search - just a little css position to change. However, I have tried the advanced search and here, nothing: no result, no message, no console log.. It may not be related but it should be investigated in case the display problem comes from this PR.

I have fixed the issue in commit 7d5443e

@waychal waychal requested a review from flavens May 25, 2021 11:49
Copy link
Contributor

@flavens flavens left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When I test the advanced search, I get the results as before but the loading spiner is running whereas you have set it to false...

Screen.Recording.2021-05-25.at.14.48.40.mov

@waychal waychal requested a review from flavens May 25, 2021 14:56
@waychal waychal force-pushed the wip/DSP-1575-empty-search-results-issue branch from be1814f to 2c7130f Compare May 26, 2021 10:57
@waychal waychal merged commit 8c67d60 into main May 26, 2021
@waychal waychal deleted the wip/DSP-1575-empty-search-results-issue branch May 26, 2021 11:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug A bug fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants