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

Key Modifier not being sent to Browser #321

Merged
merged 1 commit into from
Apr 14, 2014

Conversation

amaitland
Copy link
Member

Add modifiers parameter to ManagedCefBrowserAdapter SendKeyEvent function
Sent through key modifier so things like Shift-LeftArrow work as expected
Also include Home and End keys in list that gets sent to the browser directly (now Shift->Home and Shift->End work correctly when selecting items in input fields)

Sent through key modifier so things like Shift-LeftArrow as expected
Also include Home key in list that gets sent to the browser directly
@jornh
Copy link
Contributor

jornh commented Apr 14, 2014

Sweet! - no longer jumping to other tabs in CefSharp.Wpf.Example just because ...

... you press Home or End inside a textbox like this one - yes! ... And shift-arrow-key marking works perrrfectly on top of the previous Ctrl+X commit there is no excuse now for not using "dog-food" mode for commenting-before-merging here 😄

Can't wait to push the merge button on this one too! Excellent clean work @amaitland 👍

jornh added a commit that referenced this pull request Apr 14, 2014
@jornh jornh merged commit 20f2907 into cefsharp:master Apr 14, 2014
@jornh jornh added the bug label Apr 14, 2014
@jornh jornh added this to the 31.0.0 milestone Apr 14, 2014
@@ -20,6 +20,8 @@ namespace CefSharp.Wpf
{
public class WebView : ContentControl, IRenderWebBrowser, IWpfWebBrowser
{
private static readonly Key[] KeysToSendtoBrowser = new[] { Key.Tab, Key.Home, Key.End, Key.Left, Key.Right, Key.Up, Key.Down };
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You could put these on separate lines, for readability.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jornh, you fix? 😛

@jornh
Copy link
Contributor

jornh commented Apr 14, 2014

OK will do BDFL ... tomorrow ... muzzzt have sleezzpzz now zzzz..

@amaitland amaitland deleted the bug/key-modifier-not-being-sent branch April 15, 2014 12:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants