[jQuery] CODE REVIEW: the fruits of today's labor - now with REAL CODE!

2007-03-29 Thread Andy Matthews
Sorry guys...I tried posting the code last night using the Google Groups
interface, but it didn't come through.
 
Here you go:
http://www.commadelimited.com/uploads/psychic

  _  

From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] On
Behalf Of Andy Matthews
Sent: Wednesday, March 28, 2007 4:59 PM
To: [jQuery]
Subject: [jQuery] CODE REVIEW: the fruits of today's labor


Okay...
 
I finished my proof of concept for the project I've got. It works just as I
want it to in IE6, IE7 and FF2. I wondered if you guys would mind testing it
in whatever browsers you have as well as taking a look at the jQ code and
telling me where I might be able to thin things out a bit.
 
Please don't hesitate to offer any criticism, critiques or suggestions. I'd
love to know where I could improve this code, or if I'm doing anything
wrong.
 
Anyone out there that's wanting to do the same thing, feel free to snake
this code. It's all yours.

 
Andy Matthews
Senior Coldfusion Developer

Office:  877.707.5467 x747
Direct:  615.627.9747
Fax:  615.467.6249
[EMAIL PROTECTED]
www.dealerskins.com http://www.dealerskins.com/ 
 


dealerskinslogo.bmp
Description: Windows bitmap
___
jQuery mailing list
discuss@jquery.com
http://jquery.com/discuss/


Re: [jQuery] CODE REVIEW: the fruits of today's labor - now with REAL CODE!

2007-03-29 Thread Alex Ezell

I really like the idea. Thanks for sharing.

I noticed one behavior that was troubling, though perhaps intentional. After
I click an arrow to view more, I tried clicking it again to close the row,
but it seemed to just reload the data and flash back open the row. The only
way to close a row seems to open another one. Like I say, not what I would
expect, but perhaps what you intended.

This is in FF2 on WinXP. There were no errors reported in Firebug.

/alex

On 3/29/07, Erik Beeson [EMAIL PROTECTED] wrote:


Seems to work fine in FF2 and Safari on OS X.

--Erik

On 3/29/07, Andy Matthews [EMAIL PROTECTED]  wrote:

  Sorry guys...I tried posting the code last night using the Google
 Groups interface, but it didn't come through.

 Here you go:
 http://www.commadelimited.com/uploads/psychic

  --
 *From:* [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] *On
 Behalf Of *Andy Matthews
 *Sent:* Wednesday, March 28, 2007 4:59 PM
 *To:* [jQuery]
 *Subject:* [jQuery] CODE REVIEW: the fruits of today's labor

  Okay...

 I finished my proof of concept for the project I've got. It works just
 as I want it to in IE6, IE7 and FF2. I wondered if you guys would mind
 testing it in whatever browsers you have as well as taking a look at the jQ
 code and telling me where I might be able to thin things out a bit.

 Please don't hesitate to offer any criticism, critiques or suggestions.
 I'd love to know where I could improve this code, or if I'm doing anything
 wrong.

 Anyone out there that's wanting to do the same thing, feel free to snake
 this code. It's all yours.
 * 

 Andy Matthews
 *Senior Coldfusion Developer

 Office:  877.707.5467 x747
 Direct:  615.627.9747
 Fax:  615.467.6249
 [EMAIL PROTECTED]
 www.dealerskins.com


 ___
 jQuery mailing list
 discuss@jquery.com
 http://jquery.com/discuss/




___
jQuery mailing list
discuss@jquery.com
http://jquery.com/discuss/


___
jQuery mailing list
discuss@jquery.com
http://jquery.com/discuss/