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.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/865239

Title:
  Launcher - The background of the BFB, Workspace Switcher, Lens, and
  Trash launcher icons tiles need to use a higher saturation of the
  average background colour of the wallpaper

To manage notifications about this bug go to:
https://bugs.launchpad.net/ayatana-design/+bug/865239/+subscriptions

-- 
ubuntu-bugs mailing list
[email protected]
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to