Merge lp://qastaging/~sil2100/compiz/scale_all into lp://qastaging/compiz/0.9.8

Proposed by Łukasz Zemczak
Status: Merged
Approved by: Daniel van Vugt
Approved revision: 3257
Merged at revision: 3271
Proposed branch: lp://qastaging/~sil2100/compiz/scale_all
Merge into: lp://qastaging/compiz/0.9.8
Diff against target: 64 lines (+43/-0)
2 files modified
plugins/scale/src/privates.h (+2/-0)
plugins/scale/src/scale.cpp (+41/-0)
To merge this branch: bzr merge lp://qastaging/~sil2100/compiz/scale_all
Reviewer Review Type Date Requested Status
Daniel van Vugt Approve
Review via email: mp+111570@code.qastaging.launchpad.net

Commit message

Add support for initiating window picker in other than nomal mode. For now added only the additional 'All windows' picker, essentially fixing LP: #933776.
The code from the all mode comes from a patch made by Doug McMahon. Thanks!

Description of the change

Original description from the 0.9.7 MRQ:

The scale plugin has many actions defined: initiate for current workspace, for all windows, for groups and for current output device. But the scale plugin only supports one handler for all actions (the type of the action is not considered at all). For now I added a distinction between ScaleTypeAll and ScaleTypeNormal. I took the code proposed by Doug in the bug report as the code for ScaleTypeAll.

To post a comment you must log in.
Revision history for this message
Didier Roche-Tolomelli (didrocks) wrote :

Tests are needed to enter distro

Revision history for this message
Daniel van Vugt (vanvugt) wrote :

Looks like this fixes bug 955035 too.

Revision history for this message
Daniel van Vugt (vanvugt) wrote :

OK, it works well and solves bug 933776 (and bug 955035 it seems).

Only a minor style issue in layoutThumbs() that I'll fix.

review: Approve

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