Scale should not select a window when clicking on an empty area
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Compiz |
Fix Released
|
Medium
|
Marco Trevisan (Treviño) | ||
0.9.11 |
Fix Released
|
Medium
|
Marco Trevisan (Treviño) | ||
compiz (Ubuntu) |
Fix Released
|
Medium
|
Marco Trevisan (Treviño) | ||
Trusty |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
[Impact]
Scale selects a window also if the action has been cancelled
[Test Case]
Make sure the "Click on empty area" setting in scale plugin is set to "Terminate Scale"
1. Open some windows
2. Press Super+W to initiate scale
3. Highlight via keyboard or mouse a previously non-focused window
4. Click on an empty area of your screen
5. The scale should be terminated and the previously focused window should still be the
one with keyboard focus.
[Regression Potential]
Regression potential is basically none here, we just made sure that the mouse/keyboard-
-----
Make sure the "Click on empty area" setting in scale plugin is set to "Show Desktop" or "Terminate Scale" (latter is better for testing)
1) Trigger scale
2) Select via keyboard or mouse a previously non-focused window
3) Click on an empty area of your screen
Expected result:
4) Scale is terminated, and the scale-selected window is not focused (while it's still focused the one it was before scaling)
Actual result:
4) Scale is terminated, but the scale-selected window is now focused
Related branches
- Christopher Townsend (community): Approve
- PS Jenkins bot (community): Approve (continuous-integration)
-
Diff: 717 lines (+273/-106)9 files modifieddebian/compiz-core.install (+0/-1)
debian/compiz-plugins-default.install (+1/-0)
debian/compiz-plugins.install (+1/-0)
debian/compiz-plugins.install.armel (+1/-0)
debian/compiz-plugins.install.armhf (+1/-0)
plugins/scale/include/scale/scale.h (+0/-1)
plugins/scale/scale.xml.in (+14/-0)
plugins/scale/src/privates.h (+10/-4)
plugins/scale/src/scale.cpp (+245/-100)
- Stephen M. Webb: Approve
- PS Jenkins bot (community): Approve (continuous-integration)
-
Diff: 754 lines (+289/-106)11 files modifieddebian/changelog (+14/-0)
debian/compiz-core.install (+0/-1)
debian/compiz-plugins-default.install (+1/-0)
debian/compiz-plugins.install (+1/-0)
debian/compiz-plugins.install.armel (+1/-0)
debian/compiz-plugins.install.armhf (+1/-0)
debian/control (+2/-0)
plugins/scale/include/scale/scale.h (+0/-1)
plugins/scale/scale.xml.in (+14/-0)
plugins/scale/src/privates.h (+10/-4)
plugins/scale/src/scale.cpp (+245/-100)
Changed in compiz: | |
status: | In Progress → Fix Committed |
Changed in compiz: | |
milestone: | 0.9.11.0 → 0.9.12.0 |
Changed in compiz (Ubuntu): | |
status: | New → Fix Released |
importance: | Undecided → Medium |
assignee: | nobody → Marco Trevisan (Treviño) (3v1n0) |
description: | updated |
Changed in compiz: | |
status: | Fix Committed → Fix Released |
Hello Marco, or anyone else affected,
Accepted compiz into trusty-proposed. The package will build now and be available at http:// launchpad. net/ubuntu/ +source/ compiz/ 1:0.9.11. 2+14.04. 20140714- 0ubuntu1 in a few hours, and then in the -proposed repository.
Please help us by testing this new package. See https:/ /wiki.ubuntu. com/Testing/ EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.
If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification- failed. In either case, details of your testing will help us make a better decision.
Further information regarding the verification process can be found at https:/ /wiki.ubuntu. com/QATeam/ PerformingSRUVe rification . Thank you in advance!