Skip to content
This repository has been archived by the owner on Dec 16, 2022. It is now read-only.

fix: templates without facets #38

Merged
merged 9 commits into from
Jun 7, 2018

Conversation

samouss
Copy link
Contributor

@samouss samouss commented Jun 7, 2018

When we don't provide any attributesForFaceting the templates generate an empty div.

This PR remove this div when no facets are provided.

Copy link
Member

@francoischalifour francoischalifour left a comment

Choose a reason for hiding this comment

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

Thank you! I'm gonna update the CSS too because it breaks the grid system when there's no .filters.

@francoischalifour francoischalifour merged commit f860c2c into master Jun 7, 2018
@francoischalifour francoischalifour deleted the fix/templates-without-facets branch June 7, 2018 14:27
aymeric-giraudet pushed a commit to algolia/instantsearch that referenced this pull request Dec 14, 2022
…ntsearch-app#38)

* fix(Angular): don't generate an empty div without facets
* fix(React): don't generate an empty div without facets
* fix(Vue): don't generate an empty div without facets
* chore: ignore vscode workspace configuration
* fix(vanilla): don't generate an empty div without facets
* feat(angular): Use flexbox display
* feat(react): Use flexbox display
* feat(vue): Use flexbox display
* test(e2e): Update snapshots
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants