GDK-Win32: Remove extraneous call to _gdk_win32_append_event()

Apparently, by comparing with the other backends, we should not call
_gdk_win32_append_event() after calling gdk_scroll_event_new() but we should
call it after calling gdk_scroll_event_new_discrete(), which was why we didn't
restore the cursor after we scroll using the mouse wheel and didn't manage to
remove the shade that appears after we scrolled to the very top or very bottom.

Also, as suggested by the reporter, use IDC_SIZEALL for the system cursor that
we fall back to if no cursor theme is installed, as with other Windows
programs.

This should really fix issue #3581.
This commit is contained in:
Chun-wei Fan 2021-03-17 11:09:37 +08:00
parent 45c8c6c77f
commit 38ad068d9b
2 changed files with 1 additions and 3 deletions

View File

@ -61,7 +61,7 @@ static DefaultCursor default_cursors[] = {
{ "text", IDC_IBEAM }, { "text", IDC_IBEAM },
{ "move", IDC_SIZEALL }, { "move", IDC_SIZEALL },
{ "not-allowed", IDC_NO }, { "not-allowed", IDC_NO },
{ "all-scroll", IDC_ARROW }, { "all-scroll", IDC_SIZEALL },
{ "ew-resize", IDC_SIZEWE }, { "ew-resize", IDC_SIZEWE },
{ "e-resize", IDC_SIZEWE }, { "e-resize", IDC_SIZEWE },
{ "w-resize", IDC_SIZEWE }, { "w-resize", IDC_SIZEWE },

View File

@ -2485,8 +2485,6 @@ gdk_event_translate (MSG *msg,
delta_y, delta_y,
FALSE); FALSE);
_gdk_win32_append_event (event);
/* Append the discrete version too */ /* Append the discrete version too */
direction = 0; direction = 0;
if (msg->message == WM_MOUSEWHEEL) if (msg->message == WM_MOUSEWHEEL)