From 491100c7ee3b87519a432521e888e3070ea08ba0 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Florian=20M=C3=BCllner?= Date: Mon, 8 Sep 2014 21:50:51 +0200 Subject: [PATCH] appDisplay: Only allow ctrl-click for apps that can open new windows Trust the heuristics in shell_app_can_open_new_window() to get it right more often than not, and add an appropriate check in activate(). This makes the behavior consistent with the dash, e.g. we will try to open a new window (and show the corresponding animation) for apps that don't have a "New window" item in their dash context menu. https://bugzilla.gnome.org/show_bug.cgi?id=736329 --- js/ui/appDisplay.js | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/js/ui/appDisplay.js b/js/ui/appDisplay.js index 664096664..8f5c146df 100644 --- a/js/ui/appDisplay.js +++ b/js/ui/appDisplay.js @@ -1687,7 +1687,8 @@ const AppIcon = new Lang.Class({ activate: function (button) { let event = Clutter.get_current_event(); let modifiers = event ? event.get_state() : 0; - let openNewWindow = modifiers & Clutter.ModifierType.CONTROL_MASK && + let openNewWindow = this.app.can_open_new_window () && + modifiers & Clutter.ModifierType.CONTROL_MASK && this.app.state == Shell.AppState.RUNNING || button && button == 2;