-
Notifications
You must be signed in to change notification settings - Fork 541
feat: Slider widget #86
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
Conversation
eyes only, webpack build in error
I dunno how to author ES6 modules and provide them to our clients. It seems that https://github.com/bloodyowl/stile/blob/68b8350f4134ae2182a60c9f55442be92b4ce26e/scripts/build.js and https://github.com/bloodyowl/stile/blob/68b8350f4134ae2182a60c9f55442be92b4ce26e/webpack.config.js is one solution
+ using `algolia/react-nouislider` was able to externalize it even being written in ES6 (just use babel-cli) + moved lib/widget-utils to lib/utils because used in both widgets and components
so you can access it when on the same network
* Instantiate a slider based on a numeric attribute | ||
* @param {String|DOMElement} options.container Valid CSS Selector as a string or DOMElement | ||
* @param {String} options.facetName Name of the attribute for faceting | ||
* @param {Boolean|Object} [options.tooltips=true] Should we show tooltips or not. |
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.
What is the default tooltip message if I use only true?
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.
What is the default tooltip message if I use only true?
We just show the formatted tooltip value. So in the current case it would be "5.99" instead of "$5.99"
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.
Would be cool to have some information about that in the doc :)
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.
Adding
We now use our style-loader fork to be able to `prepend` a component `<style>` instead of appending it to the `<head>`. So that we can now override component style from a stylesheet in the html page.
@algolia/javascript This PR is done on my side, if ok with you, merge! |
LGTM |
* feat(CurrentRefinements): add query and remove reset * feat: includedAttributes
* feat(CurrentRefinements): add query and remove reset * feat: includedAttributes
components