Skip to content

gtk: avoid physical fallback for XKB modifiers - #13967

Merged
jcollie merged 1 commit into
ghostty-org:mainfrom
tothedarktowercame:fix/xkb-unsupported-modifiers
Aug 22, 2026
Merged

jcollie merged 1 commit into
ghostty-org:mainfrom
tothedarktowercame:fix/xkb-unsupported-modifiers

Conversation

@tothedarktowercame

@tothedarktowercame tothedarktowercame commented Aug 22, 2026 •

Copy link
Copy Markdown
Contributor

Hi, I noticed that this XKB config line was causing problems for Ghosttty:

    key <DELE> {   [ ISO_Level3_Shift ] };

That's a valid reconfiguration of the "delete key" as a modifier, and the same worked fine in other terminal emulators (like Alacritty). With Ghosttty, I was getting actual <delete> behaviour whenever I pressed the modifier key (although the modifier engaged after that).

This patch fixes it. In line with your AI disclose policy: I made the patch with Codex Sol. The logical fix is very succinct, and the bulk of the patch works around an underlying issue to do with GDK not recognising some modifiers as modifiers. I've tested the implementation and the patch definitely resolves it!

@tothedarktowercame
tothedarktowercame requested a review from a team as a code owner August 22, 2026 07:50
@ghostty-bot ghostty-bot Bot added the gtk Issues exclusive to GTK which are not OS-specific label Aug 22, 2026
@ghostty-vouch

ghostty-vouch Bot commented Aug 22, 2026

Copy link
Copy Markdown
Contributor

Hi @tothedarktowercame, thanks for your interest in contributing!

This project requires that pull request authors are vouched, and you are not in the list of vouched users.

This PR will be closed automatically. See https://github.com/ghostty-org/ghostty/blob/main/CONTRIBUTING.md for more details.

@ghostty-vouch ghostty-vouch Bot closed this Aug 22, 2026
@pluiedev pluiedev reopened this Aug 22, 2026

@pluiedev pluiedev left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Separating out remapKey is also a good idea. Thanks!

@jcollie
jcollie merged commit 5851d98 into ghostty-org:main Aug 22, 2026
104 checks passed
@github-actions github-actions Bot added this to the 1.4.0 milestone Aug 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gtk Issues exclusive to GTK which are not OS-specific

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants