-
Notifications
You must be signed in to change notification settings - Fork 516
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 range slider pips and value 0 #2350
Conversation
Deploy preview ready! Built with commit 0dd17d3 https://deploy-preview-2350--algolia-instantsearch.netlify.com |
I have a question @iam4x , why does the header disappear when there is no pips? Is that a behaviour that we want? |
I just did not put a header on the second example, it's expected behaviour 👍 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd add some more tests with different without the bounds.
expect(helper.getNumericRefinement('price', '<=')).toEqual([30]); | ||
expect(helper.search.callCount).toBe(1); | ||
|
||
refine([0, undefined]); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just for the sake of being exhaustive, maybe we could try without the bounds. And also do the third case with [undefined, 20]
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool fix. That's a tough one 👍
<a name=2.1.5></a> ## [2.1.5](v2.1.4...v2.1.5) (2017-09-25) ### Bug Fixes * **deps:** update dependency algolia-frontend-components to v^0.0.33 ([#2341](#2341)) ([16994d8](16994d8)) * **price-ranges:** update call to refine ([#2377](#2377)) ([34915d7](34915d7)) * **slider:** Fix range slider pips and value 0 ([#2350](#2350)) ([fa0dc09](fa0dc09)), closes [#2343](#2343)
Summary
pips: false
it was still showing pipsFixes #2343