1) read the article
2) read the comments
3) refrain from plugging their own stuff

lol

Changing this to v.push(parseInt(arguments[i].substr(1+j*2,2), 16)) looks to resolve this issue.

But a couple of things that seem odd:

In the color function in part that is supposed to handle hex color codes – the c variable is only declared but never defined. That means the following line will fail:

... while(j--) v.push(parseInt(c.substr(1+j*2,2), 16));

…But the line will only ever be executed if a browser returns a hex color value through getComputedStyle/currentStyle – maybe the line is redundant?

And the following would be shorter and equally valid I think(?):

... comp = el.currentStyle || getComputedStyle(el, null),

instead of:

comp = el.currentStyle ? el.currentStyle : getComputedStyle(el, null),


Finally opacity isn’t supported by IE – vanilla flavour anyways – so leaving it in might as well open up to rgba colors? (Supported by roughly the same browsers).

Just my 2 cents ;P