search

redacademy / it-winter-2018

2 stars 2 forks source link

refactor suggestion #26

Closed resonantdoghouse closed 6 years ago

resonantdoghouse commented 6 years ago

https://github.com/redacademy/it-winter-2018/blob/aadfa0eae0dad1c4deb8156a34b7773247323aa9/themes/instanttheatre/js/carousel.js#L3-L19

Since you are basically repeating the same thing you could try to use multiple selectors instead and see if that works...

e.g. 

$('.hire-us-grid, .testimonial-carousel-container, .studio-grid, .community-gallery-images, .show-cast-container').flickity({
   prevNextButtons: false
});
GPChappell commented 6 years ago

Refactored jQuery selectors as suggested commit 07af12d68cc4ba1de93d161227794a14954c64ed