diff options
| author | Leon Krause <lk@leonkrause.com> | 2018-05-09 15:57:10 +0200 |
|---|---|---|
| committer | Leon Krause <lk@leonkrause.com> | 2018-05-10 02:42:47 +0200 |
| commit | 9080e96bc81328e658a03e2cd248c06ddbf9f2ec (patch) | |
| tree | 068885cd973a53889b69ce1ddb6daa7a489cd3fe | |
| parent | 0f930f831410bc57d0e975b4ab73715d0d034735 (diff) | |
Fix keyboard focus lock-out with HTML5 canvas in iframe
| -rw-r--r-- | misc/dist/html/default.html | 2 | ||||
| -rw-r--r-- | platform/javascript/engine.js | 4 | ||||
| -rw-r--r-- | platform/javascript/os_javascript.cpp | 10 |
3 files changed, 10 insertions, 6 deletions
diff --git a/misc/dist/html/default.html b/misc/dist/html/default.html index 4e3515a7b6..4aa9347fec 100644 --- a/misc/dist/html/default.html +++ b/misc/dist/html/default.html @@ -195,7 +195,7 @@ $GODOT_HEAD_INCLUDE </head> <body> <div id="container"> - <canvas id="canvas" oncontextmenu="event.preventDefault();" width="640" height="480"> + <canvas id="canvas" width="640" height="480"> HTML5 canvas appears to be unsupported in the current browser.<br /> Please try updating or use a different browser. </canvas> diff --git a/platform/javascript/engine.js b/platform/javascript/engine.js index e4839af433..e3a28083a0 100644 --- a/platform/javascript/engine.js +++ b/platform/javascript/engine.js @@ -161,6 +161,10 @@ actualCanvas.style.padding = 0; actualCanvas.style.borderWidth = 0; actualCanvas.style.borderStyle = 'none'; + // disable right-click context menu + actualCanvas.addEventListener('contextmenu', function(ev) { + ev.preventDefault(); + }, false); // until context restoration is implemented actualCanvas.addEventListener('webglcontextlost', function(ev) { alert("WebGL context lost, please reload the page"); diff --git a/platform/javascript/os_javascript.cpp b/platform/javascript/os_javascript.cpp index 1b5463e40d..cb942b6bba 100644 --- a/platform/javascript/os_javascript.cpp +++ b/platform/javascript/os_javascript.cpp @@ -167,10 +167,9 @@ static EM_BOOL _mousebutton_callback(int event_type, const EmscriptenMouseEvent int mask = _input->get_mouse_button_mask(); int button_flag = 1 << (ev->get_button_index() - 1); if (ev->is_pressed()) { - // since the event is consumed, focus manually - if (!is_canvas_focused()) { - focus_canvas(); - } + // Since the event is consumed, focus manually. The containing iframe, + // if used, may not have focus yet, so focus even if already focused. + focus_canvas(); mask |= button_flag; } else if (mask & button_flag) { mask &= ~button_flag; @@ -181,7 +180,8 @@ static EM_BOOL _mousebutton_callback(int event_type, const EmscriptenMouseEvent ev->set_button_mask(mask); _input->parse_input_event(ev); - // prevent selection dragging + // Prevent multi-click text selection and wheel-click scrolling anchor. + // Context menu is prevented through contextmenu event. return true; } |