Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
Fixed regression in bug 4966 - KMSDRM: Add dynamic modeset support
Anthony Pesch

I was just communicating with one of the Retropie developers regarding this.

This change removed the forced window focus change on creation (inolen/SDL-mirror@3534cb3) as part of the change no longer assumes there's only a single window being created. This was perhaps an over-aggressive removal.

Due to that change, joystick events are only received if SDL_SetKeyboardFocus is called explicitly, or if the app has specified SDL_HINT_JOYSTICK_ALLOW_BACKGROUND_EVENTS.

I think that part of my change should be reverted to continue setting mouse / keyboard focus to the window being created. If SDL_WINDOW_INPUT_FOCUS is to be used as an input flag the code could be conditional, but that would still leave existing software broken.
  • Loading branch information
slouken committed Mar 7, 2020
1 parent 37faac0 commit 82be6dd
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions src/video/kmsdrm/SDL_kmsdrmvideo.c
Expand Up @@ -808,6 +808,10 @@ KMSDRM_CreateWindow(_THIS, SDL_Window * window)

viddata->windows[viddata->num_windows++] = window;

/* Focus on the newly created window */
SDL_SetMouseFocus(window);
SDL_SetKeyboardFocus(window);

return 0;

error:
Expand Down

0 comments on commit 82be6dd

Please sign in to comment.