define(["t5/core/dom", "t5/core/events", "jquery", "// code.jquery.com/ui/1.10.3/jquery-ui.js"], ...
That's interesting - loading of scripts directly within a define. Seems very useful and convenient. Thought I might use same approach in some scenarios. I tried to lookup on documentation of that to see use cases, but seems like requirejs officially describes the paths and shim configurations as ways to load non AMD modules. (In my still limited knowledge) A heads up though. Looks like requirejs does not guarantee execution order for non AMD scripts. So according to some sources there is a chance that jqueryui may execute before jquery in this case. Look at the "update" and comments in this post: http://iamnotmyself.com/2012/09/07/getting-started-managing-client-side-scripts-with-require-js/ Also I wonder what would happen if there were more modules that depended on the same jqueryui script..? On Thu, Oct 3, 2013 at 4:40 AM, Lenny Primak <lpri...@hope.nyc.ny.us> wrote: > I tried to do exactly what you are trying to do and that's what > precipitated the discussion about JQuery and Bootstrap from CDN. > I have brought back T5.4 zone highlight effect with this: > > https://code.google.com/p/flowlogix/source/browse/tapestry-services/src/main/resources/META-INF/modules/flowlogix/HighlightEffect.js?name=tapestry-5.4 > It requires JQueryUI and I just load it from CDN and it works perfectly. > > > On Oct 2, 2013, at 7:14 PM, Barry Books <trs...@gmail.com> wrote: > > > > When I created the datapicker I also needed the JQuery UI library but it > > does not appear to be an option in the Tapestry Core. I think many jquery > > components may need this and without it in the core many modules will > each > > end up with their own version. Is there any reason this is not included? > > --------------------------------------------------------------------- > To unsubscribe, e-mail: users-unsubscr...@tapestry.apache.org > For additional commands, e-mail: users-h...@tapestry.apache.org > >
