Re: [PATCH 01/10] comdlg32: Add IFileDialog navigation bar control.

[Jactry Zeng]

Hi Vladislav,

On 6/6/23 22:48, Vladislav Timonin wrote:
> - x = 0;
> - info->back_btn_hwnd = CreateWindowExW(0, WC_BUTTONW, NULL,
> -
>                                      WS_CHILD | WS_VISIBLE | BS_ICON | BS_BITMAP,
> -
>                                      x, 0, cs->cy, cs->cy,
> -
>                                      hwnd, (HMENU)IDC_NAVBACK, COMDLG32_hInstance, NULL);
> - SendMessageW(info->back_btn_hwnd, WM_SETFONT, (WPARAM)gui_font, FALSE);
> - set_icon(info->icons, ILI_BACK, info->back_btn_hwnd);
> - set_title_and_add_tooltip(info, info->back_btn_hwnd, IDS_BACK);
> -
> - x += cs->cy + 1;

If I understand it correctly, the 1 here is a gap between these arrow 
buttons, right?
So the 1 might need to be calculated with DPI, or they will look closer 
when in HiDPI.
> - info->fwd_btn_hwnd = CreateWindowExW(0, WC_BUTTONW, NULL,
> -
>                                     WS_CHILD | WS_VISIBLE | BS_ICON | BS_BITMAP,
> -
>                                     x, 0, cs->cy, cs->cy,
> -
>                                     hwnd, (HMENU)IDC_NAVFORWARD, COMDLG32_hInstance, NULL);
> - SendMessageW(info->fwd_btn_hwnd, WM_SETFONT, (WPARAM)gui_font, FALSE);
> - set_icon(info->icons, ILI_FORWARD, info->fwd_btn_hwnd);
> - set_title_and_add_tooltip(info, info->fwd_btn_hwnd, IDS_FORWARD);