Re: Should the JQuery UI library be include with 5.4

2013-10-03 Thread Lenny Primak
I remember seeing requireJS handling this case correctly somewhere in its documentation. I don't see any other good way resolving this in T5.4 as of now. > On Oct 3, 2013, at 12:17 AM, Magnus Kvalheim wrote: > > define(["t5/core/dom", "t5/core/events", "jquery", "// > code.jquery.com/ui/1.10.3

Re: Should the JQuery UI library be include with 5.4

2013-10-03 Thread Barry Books
I was not a big fan of breaking it up. jQuery UI is not that large and if you use different parts on different pages one download seems better. On Thu, Oct 3, 2013 at 5:47 AM, Dmitry Gusev wrote: > Before Tapestry 5.4 I used tapestry-jquery as jQuery UI provider (I don't > use T5.4 yet in any o

Re: Should the JQuery UI library be include with 5.4

2013-10-03 Thread Barry Books
I'm not sure how it would break anything to just include it although I can see issues with the themes. My concern is to get the DateField component to work I created a custom JQuery UI download and included it. If everyone that wants to add some jQuery component does this there will be lots of dup

Re: Should the JQuery UI library be include with 5.4

2013-10-03 Thread Dmitry Gusev
Before Tapestry 5.4 I used tapestry-jquery as jQuery UI provider (I don't use T5.4 yet in any of my projects). There is a @ImportJQueryUI annotation you can use to import jQuery UI parts you need: @ImportJQueryUI(value = { "jquery.ui.tabs" }) Not sure how it will work in Tapestry 5.4 and what ar

Re: Should the JQuery UI library be include with 5.4

2013-10-03 Thread Bård Magnus Kvalheim
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

Re: Should the JQuery UI library be include with 5.4

2013-10-03 Thread Magnus Kvalheim
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

Re: Should the JQuery UI library be include with 5.4

2013-10-02 Thread Lenny Primak
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/flowl

Re: Should the JQuery UI library be include with 5.4

2013-10-02 Thread Thiago H de Paula Figueiredo
On Wed, 02 Oct 2013 23:14:32 -0300, Barry Books 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 thei

Should the JQuery UI library be include with 5.4

2013-10-02 Thread Barry Books
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?