Andrea, you patch seems dangerously arbitrary:
- hsv_color.saturation = std::min(base_hsv.saturation, hsv_color.saturation); + hsv_color.saturation = std::min(base_hsv.saturation, hsv_color.saturation) * 1.36;
I suggest making this a compiz config option instead, and you can make 1.36 the default value. Anything to do with choosing colours or opacities should be configurable. People won't log bugs if they have the option of changing colours themselves.
Andrea, you patch seems dangerously arbitrary:
- hsv_color. saturation = std::min( base_hsv. saturation, hsv_color. saturation) ; saturation = std::min( base_hsv. saturation, hsv_color. saturation) * 1.36;
+ hsv_color.
I suggest making this a compiz config option instead, and you can make 1.36 the default value. Anything to do with choosing colours or opacities should be configurable. People won't log bugs if they have the option of changing colours themselves.