GNOME Bugzilla – Bug 699350
window-list: Fix edge case with static workspaces
Last modified: 2013-04-30 19:54:41 UTC
I'm sure this issue has been reported before, but I'm unable to find the corresponding bug: - enable static workspaces - open windows on all workspaces - restart the shell The workspace indicator now reports one too many workspaces (both in the indicator and inside the menu).
Created attachment 242959 [details] [review] window-list: Use notify::n-workspaces to recreate workspace menu When the number of workspaces changes, we clear the existing menu and recreate it based on the screen::n-workspaces property, so it is slightly more correct to track changes to the property directly instead of using the workspace-added/workspace-removed signals. This change also fixes a corner case, where changes to the property before workspaces are initialized are missed and we end up with the wrong number of workspaces.
Review of attachment 242959 [details] [review]: Looks good.
Attachment 242959 [details] pushed as f38d61b - window-list: Use notify::n-workspaces to recreate workspace menu