Skip to content

Fix keyboard shortcut focus issues by passing proper timestamps#226

Open
vkareh wants to merge 1 commit intoubuntu-mate:masterfrom
vkareh:keybinding-startup-notification
Open

Fix keyboard shortcut focus issues by passing proper timestamps#226
vkareh wants to merge 1 commit intoubuntu-mate:masterfrom
vkareh:keybinding-startup-notification

Conversation

@vkareh
Copy link
Copy Markdown
Member

@vkareh vkareh commented Aug 20, 2025

Applications launched via Super+# keyboard shortcuts sometimes don't get focus because start_app() used Gtk.get_current_event_time() which returned 0 when called from X11 KeyPress events instead of GTK events.

This change ensures applications launched via keyboard shortcuts get proper startup notification timestamps, allowing the window manager to focus them correctly instead of just blinking in the taskbar.

Applications launched via Super+# keyboard shortcuts sometimes don't get
focus because start_app() used Gtk.get_current_event_time() which
returned 0 when called from X11 KeyPress events instead of GTK events.

This change ensures applications launched via keyboard shortcuts get
proper startup notification timestamps, allowing the window manager to
focus them correctly instead of just blinking in the taskbar.
@vkareh
Copy link
Copy Markdown
Member Author

vkareh commented Aug 28, 2025

@flexiondotorg @lah7 - I can see that you are the only ones with merge access. Any chance this can get reviewed? (Even after 3 years of this applet not being updated, I still rely on it daily 😄)

@lah7
Copy link
Copy Markdown
Member

lah7 commented Sep 13, 2025

Sorry for the delay. I've never been involved with this applet, I don't think I have access to packaging for mate-dock-applet either. I can only be as useful as merging and hoping for the best (by letting it sit in master).

Hunt down @flexiondotorg and politely request he does a quick pass of the open pull requests so it can land in a new release/package. Might be too late for inclusion with Ubuntu 25.10 though.

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.

2 participants