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 :)

Reply via email to