From ed66a430c9eba3454c83f606738e5c25601d5446 Mon Sep 17 00:00:00 2001 From: kichikuou Date: Tue, 29 Jan 2019 12:14:54 +0000 Subject: [PATCH] Emscripten: Do not consume mouseup event outside of the canvas --- src/video/emscripten/SDL_emscriptenevents.c | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/src/video/emscripten/SDL_emscriptenevents.c b/src/video/emscripten/SDL_emscriptenevents.c index bcbefe184..b6a18efea 100644 --- a/src/video/emscripten/SDL_emscriptenevents.c +++ b/src/video/emscripten/SDL_emscriptenevents.c @@ -345,6 +345,7 @@ Emscripten_HandleMouseButton(int eventType, const EmscriptenMouseEvent *mouseEve Uint8 sdl_button; Uint8 sdl_button_state; SDL_EventType sdl_event_type; + double css_w, css_h; switch (mouseEvent->button) { case 0: @@ -371,6 +372,14 @@ Emscripten_HandleMouseButton(int eventType, const EmscriptenMouseEvent *mouseEve sdl_event_type = SDL_MOUSEBUTTONUP; } SDL_SendMouseButton(window_data->window, 0, sdl_button_state, sdl_button); + + /* Do not consume the event if the mouse is outside of the canvas. */ + emscripten_get_element_css_size(NULL, &css_w, &css_h); + if (mouseEvent->canvasX < 0 || mouseEvent->canvasX >= css_w || + mouseEvent->canvasY < 0 || mouseEvent->canvasY >= css_h) { + return 0; + } + return SDL_GetEventState(sdl_event_type) == SDL_ENABLE; }