Merge lp:~compiz-team/compiz-core/compiz-core.fix_724093 into lp:compiz-core/0.9.5

Proposed by Sam Spilsbury
Status: Merged
Merged at revision: 2813
Proposed branch: lp:~compiz-team/compiz-core/compiz-core.fix_724093
Merge into: lp:compiz-core/0.9.5
Diff against target: 22 lines (+8/-2)
1 file modified
plugins/compiztoolbox/src/compiztoolbox.cpp (+8/-2)
To merge this branch: bzr merge lp:~compiz-team/compiz-core/compiz-core.fix_724093
Reviewer Review Type Date Requested Status
Robert Carr (community) Approve
Review via email: mp+74811@code.launchpad.net

Description of the change

Fixes window titles not showing up in the switcher and probably bug 724093 since the atk property was attached to the text property.

To post a comment you must log in.
Revision history for this message
Robert Carr (robertcarr) wrote :

+1

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'plugins/compiztoolbox/src/compiztoolbox.cpp'
--- plugins/compiztoolbox/src/compiztoolbox.cpp 2011-03-14 16:12:45 +0000
+++ plugins/compiztoolbox/src/compiztoolbox.cpp 2011-09-09 15:06:40 +0000
@@ -129,10 +129,16 @@
129 CompOption::Vector opts;129 CompOption::Vector opts;
130 CompOption::Value v;130 CompOption::Value v;
131131
132 if (focus && selectedWindow && !selectedWindow->destroyed ())132 if (selectedWindow && !selectedWindow->destroyed ())
133 {133 {
134 selectedWindowId = selectedWindow->id ();134 selectedWindowId = selectedWindow->id ();
135 selectedWindow->moveInputFocusTo ();135
136 /* FIXME: Changing the input focus here will
137 * screw up the ordering of windows in
138 * the switcher, so we probably want to avoid that
139 */
140 if (focus)
141 selectedWindow->moveInputFocusTo ();
136 }142 }
137143
138 v = CompOption::Value ((int) selectedWindowId);144 v = CompOption::Value ((int) selectedWindowId);

Subscribers

People subscribed via source and target branches