[iOS] Pressing Tab key doesn't move to next cell with Google Sheets
authordbates@webkit.org <dbates@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 25 Jan 2019 23:12:43 +0000 (23:12 +0000)
committerdbates@webkit.org <dbates@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 25 Jan 2019 23:12:43 +0000 (23:12 +0000)
https://bugs.webkit.org/show_bug.cgi?id=193048
<rdar://problem/46433836>

Reviewed by Brent Fulgham.

WebCore knows how interpret the Tab and Shift + Tab keys. When building with UIKit keyboard additions
we do not need to register key commands for handling them from the UIProcess code. Instead UIKit will
sends key events for these commands to us. This will make the behavior of these key commands when executed
in an editable field match the behavior of these command when executed in a non-editable field since
both code paths will now go through WebCore.

* UIProcess/ios/WKContentViewInteraction.mm:
(-[WKContentView keyCommands]): Only compile when building with !USE(UIKIT_KEYBOARD_ADDITIONS).

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@240514 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebKit/ChangeLog
Source/WebKit/UIProcess/ios/WKContentViewInteraction.mm

index 681a2a0..99c99aa 100644 (file)
@@ -1,3 +1,20 @@
+2019-01-25  Daniel Bates  <dabates@apple.com>
+
+        [iOS] Pressing Tab key doesn't move to next cell with Google Sheets
+        https://bugs.webkit.org/show_bug.cgi?id=193048
+        <rdar://problem/46433836>
+
+        Reviewed by Brent Fulgham.
+
+        WebCore knows how interpret the Tab and Shift + Tab keys. When building with UIKit keyboard additions
+        we do not need to register key commands for handling them from the UIProcess code. Instead UIKit will
+        sends key events for these commands to us. This will make the behavior of these key commands when executed
+        in an editable field match the behavior of these command when executed in a non-editable field since
+        both code paths will now go through WebCore.
+
+        * UIProcess/ios/WKContentViewInteraction.mm:
+        (-[WKContentView keyCommands]): Only compile when building with !USE(UIKIT_KEYBOARD_ADDITIONS).
+
 2019-01-25  Alex Christensen  <achristensen@webkit.org>
 
         Fix Linux build.
index 22de419..8ca9fc2 100644 (file)
@@ -3306,6 +3306,7 @@ static void selectionChangedWithTouch(WKContentView *view, const WebCore::IntPoi
     [self resignFirstResponder];
 }
 
+#if !USE(UIKIT_KEYBOARD_ADDITIONS)
 - (NSArray *)keyCommands
 {
     if (!_page->editorState().isContentEditable)
@@ -3317,6 +3318,7 @@ static void selectionChangedWithTouch(WKContentView *view, const WebCore::IntPoi
     ] retain];
     return editableKeyCommands;
 }
+#endif
 
 - (void)_nextAccessoryTabForWebView:(id)sender
 {