Check the final merge,  I reset based on your formatting.  At least I think
I did
On Nov 12, 2013 1:21 PM, "Tanguy Krotoff" <notifications@github.com> wrote:

> In jquery.simplecolorpicker.js:
>
> > @@ -43,7 +43,14 @@
> >                       + ' role="button" tabindex="0">'
> >                       + '</span>').insertAfter(self.$select);
> >          self.$icon.on('click.' + self.type, $.proxy(self.showPicker, self));
> > -
> > +		self.$icon.on('keydown.' + self.type, function(e) {
>
> Please use 2 spaces for indentation instead of tabs.
> Also remove trailing spaces if any.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/tkrotoff/jquery-simplecolorpicker/pull/19/files#r7600991>
> .
>