Created attachment 144930 [details] [review]
patch for metacity to support workspace switch by hotkey wrapping

Is there any good reason why when I have multiple columns and rows of
workspaces that the hotkeys for switching workspaces do not wrap?  I.e. I would
prefer if ctrl+alt+up/down by themselves would let you cycle through all
available workspaces, instead of having to also use ctrl+alt+left/right.

Alternately what I want is to effectively have a 1x12 grid of workspaces, but
displayable in the switcher applet in 2 columns (so that the switcher applet on
a vertical panel doesn't take up the whole panel)

I googled and was surprised it was difficult to find anyone else asking about
this.  So I figure feedback here is my best way to find out why this has never
been implemented/merged.

In any event, I'll attach a firstpass patch that seems to work.

If there is no reason not to apply something like this patch, I could extend
the patch so that the wrapping behavior is non-default and only enabled via a
gconf setting, which would make me happy enough. 

Unfortunately actually using this patch exposes something else I want- that
with a vertical panel's switcher applet and multiple columns of workspaces,
that the mousewheel goes up and down within a single column, then wraps to
other columns.  Instead of the current behavior of going the workspace to the
right and left, with wrapping.  

It is noteworthy that wrapping is supported like that in the workspace
switcher, but not with the hotkeys.

Anyway, worst case, the next person that this really bugs may at least find
this bug report and patch.