Merge lp:~mterry/unity-greeter/986967 into lp:unity-greeter

Proposed by Michael Terry
Status: Merged
Merged at revision: 455
Proposed branch: lp:~mterry/unity-greeter/986967
Merge into: lp:unity-greeter
Diff against target: 10 lines (+1/-0)
1 file modified
src/background.vala (+1/-0)
To merge this branch: bzr merge lp:~mterry/unity-greeter/986967
Reviewer Review Type Date Requested Status
Robert Ancell Approve
Ken VanDine Pending
Review via email: mp+104622@code.launchpad.net

Description of the change

So bug 986967 seems to be caused by the background alpha being stuck at 0.0 instead of driving toward 1.0. We wait for the background to finish fading in before logging the user in, because we want a smooth transition to the session.

However, imagine the following sequence:
1) The background is currently the default.
2) We start loading a new background, setting the alpha to 0.0 and starting the animation
3) It shortly fails to load, so we go back to the old background, which stops the timer.
4) When drawing again, we see that now the old background and the new background are the same, so we don't restart the timer.

So the quickest fix is to make sure that whenever we stop the timer, we 'fast-forward' the alpha to 1.0 again. That's done here.

To post a comment you must log in.
Revision history for this message
Robert Ancell (robert-ancell) :
review: Approve
Revision history for this message
Tom Dimiduk (tdimiduk) wrote :

I had bug 986967 and the ppa version of this patch fixed it.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/background.vala'
--- src/background.vala 2012-04-16 08:33:55 +0000
+++ src/background.vala 2012-05-03 20:52:18 +0000
@@ -433,6 +433,7 @@
433 {433 {
434 old = current;434 old = current;
435 current = new_background;435 current = new_background;
436 alpha = 1.0; /* if the timer isn't going, we should always be at 1.0 */
436 timer.stop ();437 timer.stop ();
437 }438 }
438439

Subscribers

People subscribed via source and target branches