So if you have the same need, great, enjoy! If you don’t, I don’t care!

http://www.forgeniuses.com/?p=3

* Indexable by search engines
* Works in any browser
* Works if JavaScript is disabled
* Faster loading
* Fewer hits on your server

I’m not saying there aren’t use cases where this approach has some benefit, just that the current sprinklejs.com page isn’t one of them. Put together a demo case where the dynamic loading actually offers some value, in fact enough extra value to overcome its inherent disadvantages, and then it will be easier to convince people of its merits.

Scott, how is this broken? That should work just fine. It just doesn’t work on the actual sprinklejs.com web site itself since the site doubles as a browser test. You can obviously populate the contents of your tag with HTML while adding src=”..” to the tag itself, so that its contents have HTML defaults and Javascripted replacements. I just chose not to do that on the actual web site.

I’m just kind of an inveterate code simplifier. Here’s one more for the road… :-)


if (antiCacheMethod == "querystring") {
var dt = new Date();
var dts = ""
+ dt.getFullYear()
+ dt.getMonth()
+ dt.getDate()
+ dt.getHours()
+ dt.getMinutes()
+ dt.getSeconds();
if (src.indexOf("?") == -1) src += "?nocachetimestamp=" + dts;
else src += "&nocachetimestamp=" + dts;
}


could be:


if (antiCacheMethod == "querystring") {
src +=
( src.match(/\?/) ? '&' : '?' ) +
'nocachetimestamp=' + (new Date).getTime();
}


Note that this also gives you a timestamp with “millisecond” resolution (actually around 15 milliseconds on many platforms), instead of the one second resolution in the original code.

I look forward to seeing your new streamlined revision… :-)

Thx for the feedback on my Sprinkle. I totally agree that the code should be cleaned up. I just my idea was a cool one, even if it isn’t the most beautiful implementation around. :) I’ll likely do a lot of refactoring.

Jon

Ajaxians, fix this thing. Kill it and get something that works. You should expect that people will want to post code in the comments, right?

Check my blog for an example of how you can have comments with good readable code formatting – in fact the same formatting as the posts themselves. Take this discussion for example:

http://mg.to/2006/01/25/json-for-jquery

If I can do it, you can do it. :-)

(Sorry, you wouldn’t be able to use my code “as is” because the site uses Drupal – but you’re welcome to use the code formatting part if you want to adapt it to WordPress – it’s based on the GeSHi code formatter with some tweaks of my own for the zebra striping and such.)

Here’s some more of the source code:


function csiGetInnerHtmlOfTag(str, tagName) {

} else return str;
}


That one function calls str.toLowerCase() five times. It calls str.toLowerCase().indexOf("<" + tagName) three times. It calls str.toLowerCase().indexOf("</" + tagName + ">", str.toLowerCase().indexOf("<" + tagName)) twice!

Clearly some refactoring would be in order here. Or you could do the whole thing with one regular expression:


function getInnerHtmlOfTag(str, tagName) {
var match = str.match(
new RegExp( '<' + tagName + '[^>]*>(.*)</' + tagName + '>', 'i' ) );
return match ? match[1] : str;
}


Tx jdalton!

note: 100% serious, I’ve found it rather amusing, interesting (and increasing my desire to sit down and learn one of these frameworks) when I see X project/source posted, followed by the 3-line jQuery alternative in the comments :)

//execute ‘loadSrc’ on queried elements
$$(‘textarea[src],div[src],span[src]‘).invoke(‘loadSrc’, options);