Unity

Merge lp://qastaging/~azzar1/unity/fix-987674 into lp://qastaging/unity

  1. fix-987674
  2. Merge into trunk
Proposed by Andrea Azzarone
Status: Merged
Approved by: Andrea Azzarone
Approved revision: no longer in the source branch.
Merged at revision: 2413
Proposed branch: lp://qastaging/~azzar1/unity/fix-987674
Merge into: lp://qastaging/unity
Diff against target: 12 lines (+1/-1)
1 file modified
panel/WindowButtons.cpp (+1/-1)
To merge this branch: bzr merge lp://qastaging/~azzar1/unity/fix-987674
Reviewer Review Type Date Requested Status
Marco Trevisan (Treviño) Approve
Tim Penhey (community) Needs Information
Review via email: mp+110191@code.qastaging.launchpad.net

Commit message

The dash maximise button changes location if clicked just after the login.

Description of the change

== Problem ==
The dash maximise button changes location if clicked just after the login.

== Fix ==
We should use QueueRelayout in this case.

== Test ==
Not applicable

== Screenshots ==
Bug: http://ubuntuone.com/7M93yGVsnAFUhfNaCK3SZ2
Fix: http://ubuntuone.com/0NSePz8jS8x9xvt2cQ2IIX

To post a comment you must log in.
Revision history for this message
Tim Penhey (thumper) wrote :

Andy, do you have some before and after pics perhaps?

review: Needs Information
Revision history for this message
Marco Trevisan (Treviño) (3v1n0) wrote :

Looks good... Probably this should be changed also in other similar functions inside WindowButtons (like OnOverlayHidden and OnOverlayShown)

Revision history for this message
Marco Trevisan (Treviño) (3v1n0) :
review: Approve
Revision history for this message
Andrea Azzarone (azzar1) wrote :

> Andy, do you have some before and after pics perhaps?

Done.

Revision history for this message
Unity Merger (unity-merger) wrote :

No commit message specified.

Revision history for this message
Unity Merger (unity-merger) wrote :

The Jenkins job https://jenkins.qa.ubuntu.com/job/automerge-unity/807/console reported an error when processing this lp:~andyrock/unity/fix-987674 branch.
Not merging it.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.