A function err'd on my visit, looks like an uncaught exception, when I
went to check it out, I found this comment:

// This is a hack for now. The zip->city map for the lookup service
// most likely doesn't match the zip->city in the WX XML file. We
// we're going to lookup the id in the lookup servcie for the city name so we
// have some consistancy.

Two issues: First, why they even have comments in production js is
mind boggling. Second, if you're going to take the time to add
comments to your code and let the whole world see them, why not run
over them with a quick check for spelling (servcie & consistancy) or
grammar (we're != were).

- jake

On 10/4/07, Sam Collett <[EMAIL PROTECTED]> wrote:
>
> It's good that they are using it quite a bit. There are some site that
> use it, but only $(document).ready to call a function that manipulates
> the DOM the old fashioned way.
>
> On 4 Oct, 08:23, "John Resig" <[EMAIL PROTECTED]> wrote:
> > Wow! Very awesome :-)
> >
> > It looks like they're getting a pretty good coverage of features, too.
> > (DOM Manipulation, animations, events, etc.)
> >
> > --John
> >
> > On 10/4/07, Karl Swedberg <[EMAIL PROTECTED]> wrote:
> >
> > > Looks like nbc.com is heavily into the jQuery love now, using 1.1.4.1 on
> > > their site. Looks like they're using Interface and Mike Alsup's Media
> > > plugin, and Klaus Hartl's cookie plugin. Also, pulling in some data with
> > > $.ajax, doing some accordion stuff, etc.. They could be using the
> >
> > > This line appears in
> > >http://www.nbc.com/assets/js/global/nbc.com.jsright after
> > > all of the jQuery code:
> >
> > > // Legacy crap --------->
> >
> > >  Pretty funny.
> >
> > > Here's another file you might want to poke around in:
> >
> > >http://www.nbc.com/assets/js/video/fun.js
> >
> > > --Karl
> > > _________________
> > > Karl Swedberg
> > >www.englishrules.com
> > >www.learningjquery.com
>
>

Reply via email to