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

feat: InstantSearch root props #770

Merged
merged 3 commits into from
Dec 27, 2017
Merged

feat: InstantSearch root props #770

merged 3 commits into from
Dec 27, 2017

Conversation

samouss
Copy link
Collaborator

@samouss samouss commented Dec 27, 2017

Summary

Fix #285

Enable the possibility to pass custom root props to InstantSearch. Don't introduce any breaking change, since we use the root of createInstantSearch as defaultProps.

Result

The following example is with React Native, when we don't pass flex: 1 to InstantSearch the list is crop before the end.

Before

simulator screen shot - iphone x - 2017-12-27 at 12 18 14

After

simulator screen shot - iphone x - 2017-12-27 at 12 18 35

@samouss samouss requested review from bobylito and Haroenv December 27, 2017 11:22
@algobot
Copy link
Contributor

algobot commented Dec 27, 2017

Deploy preview ready!

Built with commit 20e9e55

https://deploy-preview-770--react-instantsearch.netlify.com

Copy link
Contributor

@bobylito bobylito left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@samouss samouss merged commit 2d458f8 into master Dec 27, 2017
@samouss samouss deleted the feature/root-props branch December 27, 2017 16:48
@mike623
Copy link

mike623 commented Jan 1, 2018

Did it push to NPM? It seems not don't have effect

@samouss
Copy link
Collaborator Author

samouss commented Jan 1, 2018

Nope it’s not released yet, it will be published today or tomorrow.

samouss added a commit that referenced this pull request Jan 3, 2018
<a name="4.4.0"></a>

* **createInstantSearch:** remove the client from the Snapshot ([#749](#749)) ([700d8f4](700d8f4))
* refresh cache memory leak example ([#784](#784)) ([cf228ac](cf228ac))
* **stories:** rename InstantSearch to <InstantSearch> ([#789](#789)) ([05efda5](05efda5))

* InstantSearch root props ([#770](#770)) ([2d458f8](2d458f8))
@samouss samouss mentioned this pull request Jan 3, 2018
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.

4 participants