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

search : add customizable areas into search result page #527

Merged
merged 1 commit into from
Dec 1, 2022

Conversation

zannkukai
Copy link
Contributor

@zannkukai zannkukai commented Nov 13, 2022

  • Adds some areas into the default record-search component (used to
    render a search result) to allow content injection :
    top-search-result, top-result, top-facets, bottom-result,
    bottom-search-result.
  • Add option to always hide a facet : In some case, we need to request
    to backend about a facet, but this facet should never be displayed
    by search components (other component can use content-injection to
    display them).

Co-authored-by: Renaud Michotte renaud.michotte@gmail.com

How to test

image

@zannkukai zannkukai requested review from jma and vgranata November 13, 2022 15:28
@zannkukai zannkukai self-assigned this Nov 13, 2022
* Adds some areas into the default `record-search` component (used to
  render a search result) to allow content injection :
  top-search-result, top-result, top-facets, bottom-result,
  bottom-search-result.
* Add option to always hide a facet : In some case, we need to request
  to backend about a facet, but this facet should never be displayed
  by search components (other component can use content-injection to
  display them).

Co-authored-by: Renaud Michotte <renaud.michotte@gmail.com>
@zannkukai zannkukai force-pushed the zan-searchpage-zone branch from f85ad35 to d1361cf Compare December 1, 2022 07:25
@zannkukai zannkukai merged commit e38f151 into rero:staging Dec 1, 2022
@zannkukai zannkukai deleted the zan-searchpage-zone branch December 1, 2022 16:31
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.

3 participants