Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

expose MoveFocus().. #12

Open
rorywalsh opened this issue May 23, 2023 · 3 comments
Open

expose MoveFocus().. #12

rorywalsh opened this issue May 23, 2023 · 3 comments

Comments

@rorywalsh
Copy link

Not so much an issue as a request. I wonder if you wouldn't mind exposing, the MoveFocus() method on Windows. Without it, one can't get keyboard focus to the underlying webview, or at least that's how it appears in my tests.

@julianstorer
Copy link
Collaborator

Sorry, didn't see this issue until just now.

I actually can't remember why there's a commented-out MoveFocus line in the existing code, but this is stuff we're going to be fine-tuning soon. But I'd be reluctant to add a focus method for all platforms just to fix a windows-specific problem so would rather find a way to make it work as the other platforms do if that's possible somehow?

@rorywalsh
Copy link
Author

Sure. I'm happy to patch it myself for now. Mac is fine btw, no problems there 👍

@julianstorer
Copy link
Collaborator

Thanks - if you spot any changes that help without affecting the API, I'd be curious to know!

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

No branches or pull requests

2 participants