Okay. I just moved the app over without too much hassle. Currently,
there's no way to listen for a DOM ready event, forcing me to have to
use an onLoad. That's obviously a performance hit. There's no need
to wait for all of my images.
If I could place HTML after the script includes, but still place at
the bottom of the document, that would probably be ideal. But, as is,
this will break a lot of apps. Of course, so will the package
renames, so that ship has sailed.
You're right that I could just link in a Layout component. It just
sucks having to think about when I can and can't use
IncludeJavaScriptLibrary.
And coming full circle . . . the jQuery init code in question can just
be thrown in an onLoad without too much hassle, because all it does is
unbind $ so that Prototype (pulled in by Tapestry) can work fine. So,
while my issue can be mitigated fairly easily, I think it would be a
good idea if the location of the script stuff could be controlled by a
contribution. This sorta falls into the same class as the whitespace
output trimming optimization.
I'm +0. My vote is non-binding anyway, but the issue isn't large
enough to warrant a veto. I do think it's something that deserves a
bit more attention, though.
--
Kevin
On Jun 5, 2008, at 11:54 AM, Howard Lewis Ship wrote:
I think the workaround would be to place jquery.js in the <head>,
inside you own layout component template, along with a <script> for
the necessary jquery init.
Grouping normal scripts at the bottom is important for performance
(having scripts before the HTML content slows down the initial render,
as per YSlow).
Ultimately, Tapestry is going to be combining all the scripts for the
page into a virtual meta-script and streaming a minimized, compressed
version to the client. That will ultimately be a big performance win!
On Thu, Jun 5, 2008 at 6:20 AM, Jesse Kuhnert <[EMAIL PROTECTED]>
wrote:
I'll vote +1 on it if it's wanted but Kevin may be right about onLoad
and anything not in head not being the most ideal place to load some
of these libraries.
On Wed, Jun 4, 2008 at 7:51 AM, Kevin Menard
<[EMAIL PROTECTED]> wrote:
I just bumped versions and I had an app that broke with the
movement of the
javascript libraries to the bottom of the page. I haven't had
time to take
a look in depth at what's involved to fix it, but will try to do
so by the
end of the week.
The code in question was jQuery init code. I have to call
noConflict so in
order not to clash with Prototype. I think onLoad() binds too
late if a
document.ready is called. I'll look more and report back.
--
Kevin
On Jun 3, 2008, at 5:00 PM, Howard Lewis Ship wrote:
I've created and uploaded a release of Tapestry 5.0.12, ready to
be voted
upon.
The files are uploaded to:
http://people.apache.org/~hlship/tapestry-releases/
and a Maven repository:
http://people.apache.org/~hlship/tapestry-ibiblio-rsynch-
repository/
Please examine these files to determine if a new preview release,
5.0.12, is ready.
I've also created a 5.0.12 tag in Subversion:
http://svn.apache.org/viewvc/tapestry/tapestry5/tags/releases/5.0.12/
On a successful vote, I'll move the files from these directories to
the proper distribution directories.
Vote will run for three days; on success I'll move the voted
artifacts
into place and send out appropriate notifications.
--
Howard M. Lewis Ship
Creator Apache Tapestry and Apache HiveMind
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
--
Jesse Kuhnert
Tapestry / OGNL / Dojo team member/developer
Open source based consulting work centered around
dojo/tapestry/tacos/hivemind. http://blog.opencomponentry.com
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
--
Howard M. Lewis Ship
Creator Apache Tapestry and Apache HiveMind
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
