Compiz

Merge lp://qastaging/~3v1n0/compiz/scale-ignore-other-active-win into lp://qastaging/compiz/0.9.9

  1. scale-ignore-other-active-win
  2. Merge into 0.9.9
Proposed by Marco Trevisan (Treviño)
Status: Merged
Approved by: Brandon Schaefer
Approved revision: 3587
Merged at revision: 3587
Proposed branch: lp://qastaging/~3v1n0/compiz/scale-ignore-other-active-win
Merge into: lp://qastaging/compiz/0.9.9
Diff against target: 31 lines (+18/-3)
1 file modified
plugins/scale/src/scale.cpp (+18/-3)
To merge this branch: bzr merge lp://qastaging/~3v1n0/compiz/scale-ignore-other-active-win
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Brandon Schaefer (community) Approve
Review via email: mp+145516@code.qastaging.launchpad.net

Commit message

Scale: don't set the active window if it is not in the scale group

Description of the change

When we scale a window group, we should ignore an active window that is not currently scaled when setting the currently active/hovered windows or clients will get wrong infos.

This fixes the "gost glow" issue in unity.

To post a comment you must log in.
Revision history for this message
Brandon Schaefer (brandontschaefer) wrote :

Looks good to me, works.

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:3587
http://jenkins.qa.ubuntu.com/job/compiz-clang-ci/38/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/compiz-clang-ci/./build=pbuilder,distribution=quantal,flavor=amd64/38/console

Click here to trigger a rebuild:
http://jenkins.qa.ubuntu.com/job/compiz-clang-ci/38//rebuild/?

review: Approve (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:3587
http://jenkins.qa.ubuntu.com/job/compiz-gles-ci/37/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/compiz-gles-ci/./build=pbuilder,distribution=quantal,flavor=amd64/37/console

Click here to trigger a rebuild:
http://jenkins.qa.ubuntu.com/job/compiz-gles-ci/37//rebuild/?

review: Approve (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:3587
http://jenkins.qa.ubuntu.com/job/compiz-ci/387/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/compiz-ci/./build=pbuilder,distribution=quantal,flavor=amd64/387/console

Click here to trigger a rebuild:
http://jenkins.qa.ubuntu.com/job/compiz-ci/387//rebuild/?

review: Approve (continuous-integration)

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.

Subscribers

People subscribed via source and target branches