Skip to content

Commit 4c4e9c5

Browse files
JosephMccmtwebster
authored andcommitted
overview.js: Don't hide the global.window_group while animating
Hiding it also hides our background actor. During animation the temporary background we create to show in the overview fades out leaving us with only the black stage visible. This leads to the appearance of flickering.
1 parent 75ec894 commit 4c4e9c5

File tree

1 file changed

+0
-11
lines changed

1 file changed

+0
-11
lines changed

js/ui/overview.js

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -275,17 +275,8 @@ Overview.prototype = {
275275
this._coverPane.connect('event', () => true);
276276
this._coverPane.hide();
277277

278-
// All the the actors in the window group are completely obscured,
279-
// hiding the group holding them while the Overview is displayed greatly
280-
// increases performance of the Overview especially when there are many
281-
// windows visible.
282-
//
283-
// If we switched to displaying the actors in the Overview rather than
284-
// clones of them, this would obviously no longer be necessary.
285-
//
286278
// Disable unredirection while in the overview
287279
Meta.disable_unredirect_for_screen(global.screen);
288-
global.window_group.hide();
289280
this._group.show();
290281

291282
this.workspacesView = new WorkspacesView.WorkspacesView();
@@ -456,8 +447,6 @@ Overview.prototype = {
456447
// Re-enable unredirection
457448
Meta.enable_unredirect_for_display(global.display);
458449

459-
global.window_group.show();
460-
461450
this.workspacesView.destroy();
462451
this.workspacesView = null;
463452

0 commit comments

Comments
 (0)