Gesture Improvements + Tiling Shell custom keybindings

[thyttan]

Hi!

The Gesture Improvements extension is almost as integral to how I use my laptop as Tiling Shell is.

Sadly the original Gesture Improvements (TypeScript) repo is not being maintained, but @sidevesh stepped in to put together a JS transpiled fork that I run on Gnome 47.1.

Sadly again, even the JS transpiled extension breaks when I change the keybindings for tiling left/right to anything other than SUPER+LEFT_ARROW/RIGHT_ARROW.

I have gone ahead and fixed this for me! By applying the patch in this commit diff the two extensions now complement each other very nicely!

// extension.js
@@ -117,7 +117,7 @@ const TilePreview = registerClass(class TilePreview extends St.Widget {
				// snap-left,normal,snap-right
				else {
					if (state !== GestureTileState.NORMAL) {
/* - */					const keys = [Clutter.KEY_Super_L, (state === GestureTileState.LEFT_TILE ? Clutter.KEY_Left : Clutter.KEY_Right)];
/* + */					const keys = [Clutter.KEY_Alt_L, Clutter.KEY_Super_L, (state === GestureTileState.LEFT_TILE ? Clutter.KEY_h : Clutter.KEY_l)];
						this._virtualDevice.sendKeys(keys);
					}
				}

To adapt to other keybindings, go to https://gnome.pages.gitlab.gnome.org/mutter/clutter/index.html#constants to find the relevant key mapped constants.

Hope this is of use to someone!

Demonstration

Screencast.From.2024-12-11.18-57-58.mp4

Cheers and as always thanks for Tiling Shell!

[thyttan]

After writing this I stumbled upon Window Gestures which does many of the same things as Gesture Improvements. And has an active maintainer - thanks @amarullz! It's on GSE as well.

It comes with the same problem of needing to adapt the key mapped constants behind the scenes, which I've done for me in this commit.

// extension.js
@@ -640,10 +640,11 @@ class Manager {
        }
        // TODO: Using non key shortcut for snap left/right
        if (snapRight) {
/* - */  this._sendKeyPress([Clutter.KEY_Super_L, Clutter.KEY_Right]);
/* + */  this._sendKeyPress([Clutter.KEY_Alt_L, Clutter.KEY_Super_L, Clutter.KEY_l]);
        }
        else {
/* - */  this._sendKeyPress([Clutter.KEY_Super_L, Clutter.KEY_Left]);
/* + */  this._sendKeyPress([Clutter.KEY_Alt_L, Clutter.KEY_Super_L, Clutter.KEY_h]);
        }