Skip to main content
aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPaul Pazderski2019-10-04 16:26:57 -0400
committerNiraj Modi2019-10-25 07:42:07 -0400
commitd17f2303152f85b1cc84f9ca5d702a6e16705fdb (patch)
treebe0d8cd1577534de1f2eea136848e145812cdabc
parentc22ccfd9c4afbfde6f153de2b8e0d7aa68d21694 (diff)
downloadeclipse.platform.swt-d17f2303152f85b1cc84f9ca5d702a6e16705fdb.tar.gz
eclipse.platform.swt-d17f2303152f85b1cc84f9ca5d702a6e16705fdb.tar.xz
eclipse.platform.swt-d17f2303152f85b1cc84f9ca5d702a6e16705fdb.zip
Bug 366266 - [Browser] Spacebar keypress action on webpage object is
ignored Space keypress is processed twice. When using space to activate a check box it looks like it is not working but in fact it is checked and immediately unchecked. Change-Id: I8a64a13abacfb65a1ef7c75fb23ccccaec546b91 Signed-off-by: Paul Pazderski <paul-eclipse@ppazderski.de>
-rw-r--r--bundles/org.eclipse.swt/Eclipse SWT Browser/win32/org/eclipse/swt/browser/WebSite.java11
1 files changed, 6 insertions, 5 deletions
diff --git a/bundles/org.eclipse.swt/Eclipse SWT Browser/win32/org/eclipse/swt/browser/WebSite.java b/bundles/org.eclipse.swt/Eclipse SWT Browser/win32/org/eclipse/swt/browser/WebSite.java
index d9245fed72..4fc0d9f455 100644
--- a/bundles/org.eclipse.swt/Eclipse SWT Browser/win32/org/eclipse/swt/browser/WebSite.java
+++ b/bundles/org.eclipse.swt/Eclipse SWT Browser/win32/org/eclipse/swt/browser/WebSite.java
@@ -404,14 +404,15 @@ int TranslateAccelerator(long lpMsg, long pguidCmdGroup, int nCmdID) {
case OS.VK_NEXT:
/* Do not translate/consume IE's keys for scrolling content. */
break;
+ case OS.VK_SPACE:
case OS.VK_BACK:
case OS.VK_RETURN:
/*
- * Translating OS.VK_BACK or OS.VK_RETURN results in the native control
- * handling them twice (eg.- inserting two lines instead of one). So
- * these keys are not translated here, and instead are explicitly handled
- * in the keypress handler.
- */
+ * Translating OS.VK_BACK, OS.VK_RETURN or OS.VK_SPACE results in the native
+ * control handling them twice (eg.- inserting two lines instead of one). So
+ * these keys are not translated here, and instead are explicitly handled
+ * in the keypress handler.
+ */
break;
case OS.VK_L:
case OS.VK_N:

Back to the top