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.

Sign up for GitHub

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

Jump to bottom

CEF3: Keyboard input is sent to wrong WebView instance sometimes #242

Closed
perlun opened this issue Feb 5, 2014 · 5 comments
Closed

CEF3: Keyboard input is sent to wrong WebView instance sometimes #242

perlun opened this issue Feb 5, 2014 · 5 comments
Assignees
@amaitland
Labels
bug cef3 wpf
Milestone
33.0.0

Comments

@perlun
Copy link
Member

perlun commented Feb 5, 2014

This has been fixed already, in 0cd876e.
@perlun perlun mentioned this issue Feb 5, 2014
Closed
perlun added a commit that referenced this issue Feb 5, 2014
@perlun
Fixes #243. Logical focus !== keyboard focus, and since we changed th…
55ecf77 
…e logic for #242 we must also change this (since without it, the caret is seen in multiple WebViews simultaneously. :)
@perlun perlun added cef3 and removed cef1 labels Feb 11, 2014
@perlun
Copy link
Member Author

perlun commented Feb 11, 2014

Has been fixed for CEF1, so rerouting now towards CEF3.

@perlun perlun added this to the 3.29.0 milestone Mar 19, 2014
@perlun perlun modified the milestones: 31.0.0, 31.0.0-pre1 Apr 15, 2014
@amaitland
Copy link
Member

Is this as simple as changing IsFocused to IsKeyboardFocused?

@perlun
Copy link
Member Author

perlun commented Jun 2, 2014

Yes. So feel free to do it and submit a PR for it (you can merge it yourself if you like, since the change is so trivial).

@amaitland
Copy link
Member

No problem, I'll take care of it.

@amaitland amaitland self-assigned this Jun 2, 2014
amaitland added a commit to amaitland/CefSharp that referenced this issue Jun 3, 2014
@amaitland
Resolve issue cefsharp#242 CEF3: Keyboard input is sent to wrong WebV…
42c0419 
…iew instance sometimes
@amaitland amaitland mentioned this issue Jun 3, 2014
Merged
@amaitland
Copy link
Member

Resolved with PR #383
Closing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@amaitland amaitland

Labels
bug cef3 wpf
Projects
None yet
Milestone
33.0.0
Development

No branches or pull requests
2 participants
@amaitland @perlun