Skip to content

menu: launch list-mode items from the button's clicked signal - #579

Open
borisi1 wants to merge 1 commit into
WayfireWM:masterfrom
borisi1:fix/menu-list-click
Open

borisi1 wants to merge 1 commit into
WayfireWM:masterfrom
borisi1:fix/menu-list-click

Conversation

@borisi1

@borisi1 borisi1 commented Oct 22, 2026

Copy link
Copy Markdown
Contributor

Fixes #577.

In list mode the item's box sits inside a Gtk::Button, and the left-click GestureClick was attached to that box. GTK 4.18's button consumes the primary click, so the gesture never fired and nothing was launched. Connect on_click() to button.signal_clicked() instead (added to signals like the others); the right-click and long-press controllers stay on the box.

Tested with GTK 4.18.6: list-mode items launch on left click again. Grid mode doesn't use this code path.

🤖 Generated with Claude Code

In list mode the item's box sits inside a Gtk::Button and the left-click
GestureClick was attached to that box. GTK 4.18's button consumes the
primary click, so the gesture never fired and nothing was launched.
Connect on_click() to button.signal_clicked() instead; the right-click and
long-press controllers stay on the box.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@borisi1

borisi1 commented Oct 22, 2026

Copy link
Copy Markdown
Contributor Author

The failing "Check code style with uncrustify" job isn't about this change: it fails while building uncrustify ("Compatibility with CMake < 3.5 has been removed from CMake", "Configuring incomplete"), the same as on network-nullptr and util-version-arg since Oct 20. Building ammen99/uncrustify locally with -DCMAKE_POLICY_VERSION_MINIMUM=3.5 and running it with wayfire's uncrustify.ini on the changed file gives no diff.

@trigg

trigg commented Oct 23, 2026

Copy link
Copy Markdown
Collaborator

Fixing uncrustify is on my TODO still, my apologies

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.

menu: with menu_list = true, left-clicking an application does nothing (GTK 4.18)

2 participants