Merge lp:~mc-return/unity/unity.merge-fix1063249 into lp:unity

[MC Return]

Opacity should be 0 if the state is invisible.
Added a manual "Show Desktop" test.

Please see https://bugs.launchpad.net/unity/+bug/1063249.

This fix has already been tested.

[lp:~mc-return/unity/unity.merge-fix1063249]

2833. By MC Return on 2012-10-12

Added manual "Show Desktop" test

2834. By MC Return on 2012-10-12

Re-worded the manual test and added more detail

2835. By MC Return on 2012-10-12

Better title for the manual test

[Brandon Schaefer (brandontschaefer)]

Awesome, confirmed working +1

review: Approve

=== modified file 'manual-tests/WindowManagement.txt'
--- manual-tests/WindowManagement.txt	2012-07-25 08:38:07 +0000
+++ manual-tests/WindowManagement.txt	2012-10-12 16:51:19 +0000
@@ -68,3 +68,17 @@
 #. Tap F11 to leave full screen.
 #. Press Super+S twice to force a full screen redraw.
 
+
+Show Desktop fades windows smoothly
+------------------------------------
+Tests visual regressions to the show desktop/restore windows behavior.
+
+Setup:
+
+Action:
+#. Open several windows.
+#. Press Super+D to show the desktop.
+#. Press Super+D again to restore the previous state.
+
+Expected Result:
+  All open windows must fade out and then fade back in smoothly, without any popping or pulsing effect.

=== modified file 'plugins/unityshell/src/UnityShowdesktopHandler.cpp'
--- plugins/unityshell/src/UnityShowdesktopHandler.cpp	2012-04-23 12:25:44 +0000
+++ plugins/unityshell/src/UnityShowdesktopHandler.cpp	2012-10-12 16:51:19 +0000
@@ -168,7 +168,7 @@
 
 void ShowdesktopHandler::PaintOpacity (unsigned short &opacity)
 {
-  if (progress_ == 1.0f || progress_ == 0.0f)
+  if (progress_ == 0.0f)
     opacity = std::numeric_limits <unsigned short>::max();
   else
     opacity *= (1.0f - progress_);