Skip to content

feat: run predefined menu items from a menu snapshot - #418

Merged
amrbashir merged 2 commits into
devfrom
amr/feat/snapshot-predefeined
Sep 25, 2026
Merged

amrbashir merged 2 commits into
devfrom
amr/feat/snapshot-predefeined

Conversation

@amrbashir

Copy link
Copy Markdown
Member

Add PredefinedMenuItemSnapshot::activate, so a snapshot shown from another thread or process can run what its menu items do: the about dialog, the edit commands, and on GTK 4 the window items. It is queued on the platform's main thread like the other snapshot callbacks.

Add `PredefinedMenuItemSnapshot::activate`, so a snapshot shown from another
thread or process can run what its menu items do: the about dialog, the edit
commands, and on GTK 4 the window items. It is queued on the platform's main
thread like the other snapshot callbacks.
`PredefinedMenuItemType` holds an optional `AboutMetadata` whose icon is
`Send` but not `Sync` on Windows, so capturing it in the `activate`
closure made the closure non-`Sync` and failed to build. Read it from the
shared state when the item is activated instead.
@amrbashir
amrbashir force-pushed the amr/feat/snapshot-predefeined branch from 09e94f0 to 38158b6 Compare September 25, 2026 00:57
@amrbashir
amrbashir merged commit 4f4b276 into dev Sep 25, 2026
15 checks passed
@amrbashir
amrbashir deleted the amr/feat/snapshot-predefeined branch September 25, 2026 01:15
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.

1 participant