Cool, glad it worked! That's a nice-looking site. One thing, I forgot to use my own optimization:
var zmax = 0; $( '.draggable' ).click( function () { $( this ).siblings( '.draggable' ).each(function() { var cur = $( this ).css( 'zIndex'); zmax = cur > zmax ? cur : zmax; // use 'cur' here instead of $( this ).css( 'zIndex') }); $( this ).css( 'zIndex', zmax+1 ); }); ----- Original Message ----- From: <[EMAIL PROTECTED]> To: "jQuery (English)" <jquery-en@googlegroups.com> Sent: Wednesday, April 23, 2008 8:23 PM Subject: [jQuery] Re: Can you improve my Bring-to-Front code? > > Wow, brilliant, Josh! :D > > It was missing an extra > }); > at the end, but that's it afaik. > > If you want play with it, it's here (development page; only the first > half-dozen are real divs at present) > http://vanilla-spa.homeholistics.com/products.php > > Nice one :) > > > > On Apr 24, 1:20 am, "Josh Nathanson" <[EMAIL PROTECTED]> wrote: >> Give this a try. It will set the clicked div to the highest current z-index >> plus 1, without disturbing the other divs (untested): >> >> var zmax = 0; >> >> $( '.draggable' ).click( function () { >> $( this ).siblings( '.draggable' ).each(function() { >> var cur = $( this ).css( 'zIndex'); >> zmax = cur > zmax ? $( this ).css( 'zIndex') : zmax; >> }); >> $( this ).css( 'zIndex', zmax+1 ); >> >> }); >> ----- Original Message ----- >> From: <[EMAIL PROTECTED]> >> To: "jQuery (English)" <jquery-en@googlegroups.com> >> Sent: Wednesday, April 23, 2008 4:22 PM >> Subject: [jQuery] Re: Can you improve my Bring-to-Front code? >> >> > Bother, I got Scite to fill in all the spaces & returns before >> > posting, too! Sorry the lines got mangled :/ >> >> > On Apr 24, 12:18 am, "[EMAIL PROTECTED]" >> > <[EMAIL PROTECTED]> wrote: >> >> Hi :) >> >> >> Fed up trying to figure out why my wonderful drag layers don't work in >> >> IE, I decided to get on with some more pretty stuff :) As we have >> >> layers fading in & out, and draggable, I reckon users will expect that >> >> clicking on a partially visible layer will bring it to front - like in >> >> desktop windows. >> >> >> So - well, I partially achieved it ;) You can bring a couple of layers >> >> to front, but my snippet doesn't handle the changed order well so it >> >> can't bring a previously-promoted layer further forwards. >> >> >> I'm OK with it for now, but I thought this would be useful for lots of >> >> other jQuery users ... and, if your Javascript is better than mine >> >> (very likely!), perhaps you'll be kind enough to post back an improved >> >> version? >> >> >> Cheers! >> >> Cherry >> >> >> $( '.draggable' ).click( function () { >> >> var i = 1; i++; >> >> otherZ = $ >> >> ( this ).siblings( '.draggable' ).css( 'zIndex' ); >> >> myZ = parseInt(otherZ) + i; >> >> $( this ).css( 'zIndex', myZ ); >> >> reset; >> >> });