Skip to content

Conversation

@cyysummer
Copy link
Contributor

I've tested using Win32 API to create menus with returned window handle.

@rthomasv3
Copy link
Contributor

I just created an issue for this (#54) and opened a PR, but I then I saw this one already existed.

These changes are basically the same as mine, so I'll go ahead and close mine.

I've tested the changes on Windows and Linux and everything appears to be working.

In Windows, the handle returned matches Process.MainWindowHandle, and in Linux where MainWindowHandle is always 0, the new binding is working great.

GetWindow(): 97212317951280

@geaz
Copy link
Member

geaz commented Oct 31, 2025

Thanks for your work and testing @cyysummer & @rthomasv3! Merged 👍

@geaz geaz merged commit 9e0739e into webview:master Oct 31, 2025
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.

3 participants