Re: [PATCH v2 0/10] MR2993: comdlg32: IFileDialog navigation bar control. Part 1.

[Vladislav Timonin (＠vt)]

On Wed Jun  7 03:05:30 2023 +0000, **** wrote:
> Jactry Zeng replied on the mailing list:
> 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.

It is indeed. Changed it to `MulDiv(1, info->dpi_x, USER_DEFAULT_SCREEN_DPI)`, doesn't affect 96/120 dpi, but starting from 144 dpi it gets scaled to 2px now.