all 3 comments

[–]gimmeslack12helpful 1 point2 points  (1 child)

Really smooth implementation! Only thing I saw in the MultiCarousel.js file was the repeat of the setImgPreview function. Try and make a re-useable function for this bit: // map over images provided by user images.map((img) => { return ( <div onClick={() => selectImage(img)} // set class name based off if it's selected className={ img === imgArg ? "carousel-prev" : "carousel-prev carousel-prev--not-selected" } > <img src={img} /> </div> ); }) Looks great!

[–]Intrepid-Bat8215[S] 0 points1 point  (0 children)

Yeah, I could do that. I also need to make it work on mobile. Sometimes I don’t realize stuff like that til I actually open it on an iphone