Update color-thief.js

[guillaumehilaire]

for get dominant color if the file is white dominant

[ksubileau]

Hi,
In fact, the script skips the white pixels (>rgb(250,250,250)) of the image in order not to return white as the dominant color of an image with a white background like below :

I think that asserting that the dominant color of an image as the one above is white is not really accurate. However, if white is really part of the image (clouds in the sky, white clothes,...), it's true that it should be taken into account.
So maybe it would be better to add an option to indicate whether or not you want to ignore white pixels. Or, even better, add an option to specify one or more colors to ignore ?

[programmin1]

I see similar behavior for a mostly-dark image. Can this be behavior be turned off in an option/parameter?

[ksubileau]

No, not yet.

[alexiskattan]

it would be very nice to be able to have a flag to detect if white is the dominant color

[frumbert]

I'd like to specifically detect when white is the dominant colour of the image.

[zamartz]

@ksubileau @alexiskattan
Would it be better to:

1. Manual = send a color list to ignore as dominant
2. Auto = allow for ignoring the first dominant color

[DoimoInterlogica]

RGB components are never > 255 ...

[AlfredJKwack]

Hi,

Thanks for the contribution. While this change adresses the issue from the OP it also breaks the intended behavior referred to earlier by @ksubileau. If you submit a PR which introduces a parameter so this can be turned off and on with appropriate defaults (ie. 'off' so people who update don't get unxepected behaviour) I'm sure someone would add appropriate tests and update the doc accordingly.