Dan G. Switzer, II wrote: > Mike, > > Looks good. One tip to reduce the code. You could re-write your multiple > css() calls that look like: > > var wrapper = $("<div>") > .css({ > "width": settings.imagewidth + "px" > , "height": settings.imageheight + "px" > , "background": "url(" + settings.image + ") 0 0 no-repeat" > , "background-position": "right" > , "display": "inline" > , "position": "absolute" > , "overflow": "hidden" > , "cursor": "pointer" > }); >
Sweet, I didn't know you could do multiple properties in the one function call like that :)