Skip to content

feat: add Gnome 50 support#535

Open
bendavis78 wants to merge 1 commit into
domferr:mainfrom
bendavis78:gnome-50
Open

feat: add Gnome 50 support#535
bendavis78 wants to merge 1 commit into
domferr:mainfrom
bendavis78:gnome-50

Conversation

@bendavis78
Copy link
Copy Markdown

GNOME 50 support

Adds GNOME 50 to the supported shell versions and fixes Super key detection, which silently broke in GNOME 50 due to a modifier mask change.

esbuild bump to ^0.28.0

esbuild-sass-plugin@3.7.0 requires esbuild >= 0.27.3 as a peer dep. The previous ^0.25.10 pin caused npm install to fail with a dependency resolution error.

Super key modifier detection

global.get_pointer() returns XKB-serialized modifiers where Super is reported as MOD4_MASK (bit 6), not SUPER_MASK (bit 26). SUPER_MASK is a virtual modifier absent from the XKB state returned by xkb_state_serialize_mods(), so any feature that checks for Super (e.g. Super+drag to tile) was a silent no-op on GNOME 50.

Alt was already handled correctly via MOD1_MASK; this brings Super in line with the same approach. Also consistent with Mutter commit c0fd2d9556 ("core: Let key presses of special modifiers through"), which changed overlay key handling to propagate the initial Super press rather than consume it.

Closes #531

@Xake
Copy link
Copy Markdown

Xake commented Apr 13, 2026

What does this PR add that is not already in https://github.com/domferr/tilingshell/commits/v18.0/ ?

@marcelocampanelli
Copy link
Copy Markdown

brooo, i love you!!!

@alexanderadam
Copy link
Copy Markdown

alexanderadam commented Apr 29, 2026

What does this PR add that is not already in v18.0 (commits) ?

The MOD4_MASK was changed 1st of March and then merged in PR 506, the metadata got updated in January already.

Hence everything is in v18 already.
Does anyone know when v18.0 will be available on extensions.gnome.org?

Nevermind. I just had to update via https://extensions.gnome.org/extension/7065/tiling-shell/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants