Skip to content

Conversation

bidoubiwa
Copy link
Contributor

@bidoubiwa bidoubiwa commented Mar 17, 2022

fixes: #599

Some improvements were done on the readme as well

Screenshot 2022-03-17 at 18 16 54

@bidoubiwa bidoubiwa added the enhancement New feature or request label Mar 17, 2022
@bidoubiwa bidoubiwa requested a review from alallema March 21, 2022 10:28
@bidoubiwa bidoubiwa marked this pull request as ready for review March 21, 2022 10:28
Copy link
Contributor

@alallema alallema left a comment

Choose a reason for hiding this comment

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

Well done! I had just few comments

README.md Outdated
### Placeholder Search

Placeholders search means showing results even when the search query is empty. By default it is `true`.
If set to `false`, only when a request is made with at least one character, results will appear.
Copy link
Contributor

Choose a reason for hiding this comment

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

It is maybe came from me but I find this sentence hard to understand

Copy link
Contributor

Choose a reason for hiding this comment

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

If set to `false`, results will appear only when a request is made with at least one character.

Copy link
Contributor Author

@bidoubiwa bidoubiwa Mar 21, 2022

Choose a reason for hiding this comment

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

When placeholder search is set to false, no results appears when searching on no characters. For example, if the query is "" no results appear.

What do you think?

Copy link
Contributor

Choose a reason for hiding this comment

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

Perfect

bidoubiwa and others added 2 commits March 21, 2022 12:03
Co-authored-by: Amélie <alallema@users.noreply.github.com>
Copy link
Contributor

@alallema alallema left a comment

Choose a reason for hiding this comment

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

LGTM! 💪

@bidoubiwa
Copy link
Contributor Author

bors merge

@meili-bors
Copy link
Contributor

meili-bors bot commented Mar 21, 2022

@meili-bors meili-bors bot merged commit 2da557d into main Mar 21, 2022
@meili-bors meili-bors bot deleted the optionnal_null_facets branch March 21, 2022 11:29
@ar-siddiqui
Copy link

Is this in production yet? This is still a pain point for us.

@bidoubiwa
Copy link
Contributor Author

It is in production, could you open an issue with your specific setup that does not work?

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

Successfully merging this pull request may close these issues.

Filter selection removes other filters, add OR in refinementsList
3 participants