tags 679560 + patch - moreinfo unreproducible
thanks

Hi,
a patch is available here:

https://bugzilla.gnome.org/show_bug.cgi?id=685439#c15

I attach it to this e-mail too.

Best regards,
Carlo

--
  .-.  | Registered Linux User #443882                        |  .''`.
  oo|  | http://linuxcounter.net/                             | : :'  :
 /`'\  |                           Registered Debian User #9  | `. `'`
(\_;/) |                 http://debiancounter.altervista.org/ |   `-

From b63fc3acbe9a28c9f9987af7d5236afa5ebdefe6 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Florian=20M=C3=BCllner?= <fmuell...@gnome.org>
Date: Wed, 17 Oct 2012 20:15:10 +0200
Subject: [PATCH] screen: Ignore num-workspaces when using dynamic workspaces

On startup, workspaces are initialized according to the num-workspaces
preference. However when using dynamic workspaces, the actual number
of workspaces in use might be greater than the preference (when
replacing the window manager), forcing windows on those workspaces
to the first workspace.
To fix, ignore the preference completely when using dynamic workspaces
and try to restore the previous number of workspaces (as read from
_NET_NUMBER_OF_DESKTOPS).

https://bugzilla.gnome.org/show_bug.cgi?id=685439
---
 src/core/screen.c | 25 ++++++++++++++++++++++++-
 1 file changed, 24 insertions(+), 1 deletion(-)

diff --git a/src/core/screen.c b/src/core/screen.c
index b9c5e05..3914ae1 100644
--- a/src/core/screen.c
+++ b/src/core/screen.c
@@ -1464,7 +1464,30 @@ update_num_workspaces (MetaScreen *screen,
   MetaWorkspace *last_remaining;
   gboolean need_change_space;
   
-  new_num = meta_prefs_get_num_workspaces ();
+  if (meta_prefs_get_dynamic_workspaces ())
+    {
+      int n_items;
+      gulong *list;
+
+      n_items = 0;
+      list = NULL;
+
+      if (meta_prop_get_cardinal_list (screen->display, screen->xroot,
+                                       
screen->display->atom__NET_NUMBER_OF_DESKTOPS,
+                                       &list, &n_items))
+        {
+          new_num = list[0];
+          meta_XFree (list);
+        }
+      else
+        {
+          new_num = 1;
+        }
+    }
+  else
+    {
+      new_num = meta_prefs_get_num_workspaces ();
+    }
 
   g_assert (new_num > 0);
 
-- 
1.7.12.1

Reply via email to