# HG changeset patch # User Bram Moolenaar # Date 1594932304 -7200 # Node ID a4f21363015110c4089224303c212ef3b6da5463 # Parent e5e32c9f711ffcde2098cf188b719af3774cca89 patch 8.2.1226: MS-Windows: windows positioning wrong depending on taskbar Commit: https://github.com/vim/vim/commit/98af99f2d79b310e81003f5e27862a7b522d8372 Author: Bram Moolenaar Date: Thu Jul 16 22:30:31 2020 +0200 patch 8.2.1226: MS-Windows: windows positioning wrong depending on taskbar Problem: MS-Windows: windows positioning wrong when the taskbar is placed at the top or left of the screen. Solution: Use GetWindowRect and MoveWindow APIs. (Yukihiro Nakadaira, Ken Takata, closes #6455) diff --git a/src/gui_w32.c b/src/gui_w32.c --- a/src/gui_w32.c +++ b/src/gui_w32.c @@ -1429,7 +1429,7 @@ gui_mch_create_scrollbar( /* * Find the scrollbar with the given hwnd. */ - static scrollbar_T * + static scrollbar_T * gui_mswin_find_scrollbar(HWND hwnd) { win_T *wp; @@ -5395,28 +5395,21 @@ gui_mch_set_shellsize( int direction) { RECT workarea_rect; + RECT window_rect; int win_width, win_height; - WINDOWPLACEMENT wndpl; // Try to keep window completely on screen. // Get position of the screen work area. This is the part that is not // used by the taskbar or appbars. get_work_area(&workarea_rect); - // Get current position of our window. Note that the .left and .top are - // relative to the work area. - wndpl.length = sizeof(WINDOWPLACEMENT); - GetWindowPlacement(s_hwnd, &wndpl); - // Resizing a maximized window looks very strange, unzoom it first. // But don't do it when still starting up, it may have been requested in // the shortcut. - if (wndpl.showCmd == SW_SHOWMAXIMIZED && starting == 0) - { + if (IsZoomed(s_hwnd) && starting == 0) ShowWindow(s_hwnd, SW_SHOWNORMAL); - // Need to get the settings of the normal window. - GetWindowPlacement(s_hwnd, &wndpl); - } + + GetWindowRect(s_hwnd, &window_rect); // compute the size of the outside of the window win_width = width + (GetSystemMetrics(SM_CXFRAME) + @@ -5432,34 +5425,24 @@ gui_mch_set_shellsize( // The following should take care of keeping Vim on the same monitor, no // matter if the secondary monitor is left or right of the primary // monitor. - wndpl.rcNormalPosition.right = wndpl.rcNormalPosition.left + win_width; - wndpl.rcNormalPosition.bottom = wndpl.rcNormalPosition.top + win_height; + window_rect.right = window_rect.left + win_width; + window_rect.bottom = window_rect.top + win_height; // If the window is going off the screen, move it on to the screen. - if ((direction & RESIZE_HOR) - && wndpl.rcNormalPosition.right > workarea_rect.right) - OffsetRect(&wndpl.rcNormalPosition, - workarea_rect.right - wndpl.rcNormalPosition.right, 0); - - if ((direction & RESIZE_HOR) - && wndpl.rcNormalPosition.left < workarea_rect.left) - OffsetRect(&wndpl.rcNormalPosition, - workarea_rect.left - wndpl.rcNormalPosition.left, 0); - - if ((direction & RESIZE_VERT) - && wndpl.rcNormalPosition.bottom > workarea_rect.bottom) - OffsetRect(&wndpl.rcNormalPosition, - 0, workarea_rect.bottom - wndpl.rcNormalPosition.bottom); - - if ((direction & RESIZE_VERT) - && wndpl.rcNormalPosition.top < workarea_rect.top) - OffsetRect(&wndpl.rcNormalPosition, - 0, workarea_rect.top - wndpl.rcNormalPosition.top); - - // set window position - we should use SetWindowPlacement rather than - // SetWindowPos as the MSDN docs say the coord systems returned by - // these two are not compatible. - SetWindowPlacement(s_hwnd, &wndpl); + if ((direction & RESIZE_HOR) && window_rect.right > workarea_rect.right) + OffsetRect(&window_rect, workarea_rect.right - window_rect.right, 0); + + if ((direction & RESIZE_HOR) && window_rect.left < workarea_rect.left) + OffsetRect(&window_rect, workarea_rect.left - window_rect.left, 0); + + if ((direction & RESIZE_VERT) && window_rect.bottom > workarea_rect.bottom) + OffsetRect(&window_rect, 0, workarea_rect.bottom - window_rect.bottom); + + if ((direction & RESIZE_VERT) && window_rect.top < workarea_rect.top) + OffsetRect(&window_rect, 0, workarea_rect.top - window_rect.top); + + MoveWindow(s_hwnd, window_rect.left, window_rect.top, + win_width, win_height, TRUE); SetActiveWindow(s_hwnd); SetFocus(s_hwnd); diff --git a/src/version.c b/src/version.c --- a/src/version.c +++ b/src/version.c @@ -755,6 +755,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ /**/ + 1226, +/**/ 1225, /**/ 1224,