Merge lp:~gala-dev/gala/always-focus-on-map into lp:gala

[Tom Beckmann]

plugin: add gsettings key toggling whether windows should always be focused when they're mapped

This adds a key always-focus-on-map to gala's behavior gsettings. Once enabled, this will make gala call activate() on each window that is mapped, so windows will always be focused once mapped. Normally, mutter does quite some work to determine whether a newly mapped window should be focused or not, most importantly in our case, if a newer interaction with a different window has occurred since that window was "started". And this very part is currently broken in Gtk, the startup time is not correctly set for multi window apps, like the terminal, and the last interaction with any window of the app is used for the new window for comparision instead. See here: https://bugzilla.gnome.org/show_bug.cgi?id=721145

As the fix proposed in the GNOME bug is part of a larger patch restructuring the GtkApplication code, I don't think we'll be able to backport it. Instead I would propose this temporary fix for luna so we can get rid of this annoying problem, hopefully for isis, we have a Gtk with that patch applied.

An important question to be talked about would also be whether we want to have this enabled by default. Enabling works with a simple "gsettings set org.pantheon.desktop.gala.behavior always-focus-on-map true".
I've set it to false by default currently, but I doubt the current behavior can be considered better than this.

Relevant gala bug: https://bugs.launchpad.net/gala/+bug/1220179

[Rico Tzschichholz (ricotz)]

As said on IRC. I would prefer to find out what fixed this problem in Gtk+ 3.11.x rather than introducing this additional hacky option ;)

Unmerged revisions

367. By Tom Beckmann on 2014-01-22

plugin: add gsettings key toggling whether windows should always be focused when they're mapped

=== modified file 'data/org.pantheon.desktop.gala.gschema.xml'
--- data/org.pantheon.desktop.gala.gschema.xml	2013-09-28 22:35:41 +0000
+++ data/org.pantheon.desktop.gala.gschema.xml	2014-01-22 15:13:05 +0000
@@ -51,6 +51,11 @@
 				followed by a ':' and then the command. ';;' is used as delimiter between command defintions.
 			</description>
 		</key>
+		<key type="b" name="always-focus-on-map">
+			<default>false</default>
+			<summary>Always raise and focus windows when they map</summary>
+			<description>When enabled windows are always raised and focused when mapped, even if they would disrupt a newer user interaction with a different window</description>
+		</key>
 		<key type="b" name="edge-tiling">
 			<default>true</default>
 			<summary>Enable edge tiling when dropping windows on screen edges</summary>

=== modified file 'src/Plugin.vala'
--- src/Plugin.vala	2013-11-22 20:56:09 +0000
+++ src/Plugin.vala	2014-01-22 15:13:05 +0000
@@ -625,6 +625,10 @@
 						map_completed (actor);
 						return;
 					}
+
+					if (BehaviorSettings.get_default ().always_focus_on_map) {
+						window.activate(window.get_display ().get_current_time ());
+					}
 					
 					mapping.add (actor);
 					

=== modified file 'src/Settings.vala'
--- src/Settings.vala	2013-07-11 21:51:43 +0000
+++ src/Settings.vala	2014-01-22 15:13:05 +0000
@@ -21,6 +21,7 @@
 	{
 		public bool dynamic_workspaces { get; set; }
 		public bool edge_tiling { get; set; }
+		public bool always_focus_on_map { get; set; }
 		public string panel_main_menu_action { get; set; }
 		public string toggle_recording_action { get; set; }
 		public string overlay_action { get; set; }