Use second dominent colour, when dominent colour is excluded about jquery.adaptive-backgrounds.js HOT 2 CLOSED

@smth I think the issue is with the underlying library (RGBaster.js) which does the color calculations. It seems to not be doing what it should when it returns colors.

Adaptive-backgrounds blindly uses the input from rgbaster

97 $parent.css({ backgroundColor: data.color });

Where data.color comes from:

77 $this.on( EVENT_CF, function(ev, data){

and

53           RGBaster.colors(img, { 
54             paletteSize: 20, 
55             exclude: opts.exclude, 
56             success: function(colors) { 
57               $this.attr(DATA_COLOR, colors.dominant); 
58               $this.trigger(EVENT_CF, { color: colors.dominant, palette: colors.palette }); 
59             } 
60           });

Based on this it seems to me that you may want to raise the case in RGBaster.js instead.

Another, more involved option, would be to replace rgbaster.js with something else. You may want to look at Lokesh's Color-Thief for that. Putting in this library at the right spots seems pretty straitforward.

from jquery.adaptive-backgrounds.js.

This should now be fixed.

As noted previously , there was an issue with the underlying RGBaster.js library which:

• skipped a high number of pixels
• only considered an area of 300 by 150px of the source image
• had some issues with transparent pixels and
• didn't keep a list of unique colors.

The latter issue is of no bearing to AD

Would you mind checking and validating if the issue still exists.

from jquery.adaptive-backgrounds.js.