scratchfoundation/scratch-www
1
0
Fork 0
mirror of https://github.com/scratchfoundation/scratch-www.git synced 2024-11-22 23:27:54 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

separate out arrows var from settings object

Browse source 
seems to be returning false when it shouldn't. When I was testing before it appears I was getting a cached js, so it didn't actually work.
This commit is contained in:
Matthew Taylor 2015-10-23 14:47:49 -04:00
parent 9e9dc0221d
commit c6095d6425
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/components/carousel/carousel.jsx
View file

@ -23,7 +23,6 @@ var Carousel = React.createClass({
render: function () {
var settings = this.props.settings || {};
defaults(settings, {
arrows: this.props.items.length > settings.slidesToShow,
dots: false,
infinite: false,
lazyLoad: true,
@ -31,9 +30,10 @@ var Carousel = React.createClass({
slidesToScroll: 5,
variableWidth: true
});
var arrows = this.props.items.length > settings.slidesToShow;
return (
<Slider className={'carousel ' + this.props.className} {... settings}>
<Slider className={'carousel ' + this.props.className} arrows={arrows} {... settings}>
{this.props.items.map(function (item) {
var href = '';
switch (item.type) {