I appreciate your concerns. I will take your thoughts into consideration.


On Mon, Sep 9, 2013 at 7:37 PM, Jake LaCombe <notifications@github.com>wrote:

> In app/assets/javascripts/application.js:
>
> >  //= require turbolinks
> >  //= require_tree .
> > +$(document).ready(function(){
> > +  $('.carousel').carousel({
> > +    interval: 2000
> > +  });
> > +});
>
> I recommend combining all these blocks into the same document.ready
> function.
>
> For bonus points, a jQuery closure may be best suited here.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/trepidity/soldierup/pull/3/files#r6255200>
> .
>



-- 
Jonathan E. Emmett