- Wine-gitlab - list.winehq.org

Re: [PATCH v4 0/6] MR1857: Draft: comdlg32: IFileDialog navigation bar
by Vladislav Timonin (＠vt) 28 Dec '22

28 Dec '22

On Wed Dec 28 19:30:28 2022 +0000, Esme Povirk wrote: > I think you could remove the need for this by drawing the left border > outside the window. If you mean extending it to be painted under crumbs, it originally was, static control with `SS_GRAYFRAME` taking up full width. But that had issues with frame being painted over crumbs on initial dialog show. And right border trailing on resize that I was ignoring thinking it was caused by [54137](https://bugs.winehq.org/show_bug.cgi?id=54137). -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1857#note_20181

1 0

[PATCH 0/1] MR1890: po: Update Dutch translation.
by Floris Renaud (＠jkfloris) 28 Dec '22

28 Dec '22

- Avoid using the pronouns "u", "uw", "je" and "jouw" Signed-off-by: Floris Renaud <jkfloris(a)dds.nl> -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1890

2 1

Re: [PATCH v4 0/6] MR1857: Draft: comdlg32: IFileDialog navigation bar
by Vladislav Timonin (＠vt) 28 Dec '22

28 Dec '22

On Wed Dec 28 19:30:30 2022 +0000, Esme Povirk wrote: > I think removing items from the list like this is confusing. It also > results in the need for a special case "preserving" the first crumb in > the layout code. This is similar to what Windows does, but it indeed does complicate things a little, I'll remove it. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1857#note_20180

1 0

Re: [PATCH v4 0/6] MR1857: Draft: comdlg32: IFileDialog navigation bar
by Esme Povirk (＠madewokherd) 28 Dec '22

28 Dec '22

I haven't yet looked at "comdlg32: Add address edit to IFileDialog navigation bar." and the commit after it, as I've hit my mental limit for the day. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1857#note_20179

1 0

Re: [PATCH v4 0/6] MR1857: Draft: comdlg32: IFileDialog navigation bar
by Esme Povirk (＠madewokherd) 28 Dec '22

28 Dec '22

Esme Povirk (@madewokherd) commented about dlls/comdlg32/navbar.c: > while (ILRemoveLastID(pidl)); > ILFree(pidl); > > + NAVBAR_OVERFLOW_Clear(info); > + > + /* move the first crumb to overflow menu, if there are at least 2 crumbs */ > + crumb1 = LIST_ENTRY(list_head(&new_crumbs), struct crumb, entry); > + if (crumb1 && crumb1->entry.next != &new_crumbs) > + { > + NAVBAR_OVERFLOW_Insert(info, crumb1->pidl, crumb1->display_name); > + DestroyWindow(crumb1->hwnd); > + > + list_remove(&crumb1->entry); I think removing items from the list like this is confusing. It also results in the need for a special case "preserving" the first crumb in the layout code. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1857#note_20178

1 0

Re: [PATCH v4 0/6] MR1857: Draft: comdlg32: IFileDialog navigation bar
by Esme Povirk (＠madewokherd) 28 Dec '22

28 Dec '22

Esme Povirk (@madewokherd) commented about dlls/comdlg32/navbar.c: > hwnd, 0, COMDLG32_hInstance, NULL); > SetWindowSubclass(info->background_hwnd, NAVBAR_BACKGROUND_SubclassProc, 0, (DWORD_PTR)info); > > + info->overflow_hwnd = CreateWindowExW(0, WC_BUTTONW, NULL, For accessibility it'd be good to give this button a title. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1857#note_20177

1 0

Re: [PATCH v4 0/6] MR1857: Draft: comdlg32: IFileDialog navigation bar
by Esme Povirk (＠madewokherd) 28 Dec '22

28 Dec '22

Esme Povirk (@madewokherd) commented about dlls/comdlg32/navbar.c: > + menu_item.hbmpItem = HBMMENU_CALLBACK; /* see NAVBAR_OVERFLOW_DrawIcon */ > + > + InsertMenuItemW(info->overflow_menu, -1, TRUE, &menu_item); > +} > + > +static void NAVBAR_OVERFLOW_Clear(NAVBAR_INFO *info) > +{ > + INT i, menu_item_count = GetMenuItemCount(info->overflow_menu); > + MENUITEMINFOW menu_item = {0}; > + > + menu_item.cbSize = sizeof(MENUITEMINFOW); > + menu_item.fMask = MIIM_DATA; > + > + for (i = 0; i < menu_item_count; i++) > + { > + /* deleting an item shifts further indices, so don't use `i` */ It would probably be simpler to delete the items from the end, and slightly more efficient because it removes the need to shift indices. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1857#note_20175

1 0

Re: [PATCH v4 0/6] MR1857: Draft: comdlg32: IFileDialog navigation bar
by Esme Povirk (＠madewokherd) 28 Dec '22

28 Dec '22

Esme Povirk (@madewokherd) commented about dlls/comdlg32/navbar.c: > + > +static LRESULT NAVBAR_OVERFLOW_DrawIcon(HWND hwnd, NAVBAR_INFO *info, UINT msg, WPARAM wparam, LPARAM lparam) > +{ > + DRAWITEMSTRUCT *dis = (DRAWITEMSTRUCT *)lparam; > + ITEMIDLIST *pidl = (ITEMIDLIST *)dis->itemData; > + SHFILEINFOW file_info = {0}; > + IImageList *icon_list; > + > + icon_list = (IImageList *)SHGetFileInfoW((const WCHAR *)pidl, 0, &file_info, sizeof(file_info), > + SHGFI_PIDL | SHGFI_ICON | SHGFI_SMALLICON | SHGFI_SHELLICONSIZE | SHGFI_SYSICONINDEX); > + if (icon_list) > + { > + int icon_width, icon_height; > + > + IImageList_GetIconSize(icon_list, &icon_width, &icon_height); > + DrawIconEx(dis->hDC, dis->rcItem.left - (icon_width / 2), dis->rcItem.top, Drawing outside of rcItem doesn't seem right. Why is this necessary? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1857#note_20176

1 0

Re: [PATCH v4 0/6] MR1857: Draft: comdlg32: IFileDialog navigation bar
by Esme Povirk (＠madewokherd) 28 Dec '22

28 Dec '22

Esme Povirk (@madewokherd) commented about dlls/comdlg32/navbar.c: > + SendMessageW(info->fwd_btn_hwnd, WM_SETFONT, (WPARAM)gui_font, FALSE); > + icon = ImageList_GetIcon(info->icons, ILI_FORWARD, ILD_NORMAL); > + SendMessageW(info->fwd_btn_hwnd, BM_SETIMAGE, (WPARAM)IMAGE_ICON, (LPARAM)icon); > + DestroyIcon(icon); > + > + SetWindowLongPtrW(hwnd, 0, (DWORD_PTR)info); > + > + return DefWindowProcW(hwnd, msg, wparam, lparam); > +} > + > +static LRESULT NAVBAR_Destroy(HWND hwnd, NAVBAR_INFO *info, UINT msg, WPARAM wparam, LPARAM lparam) > +{ > + SetWindowLongPtrW(hwnd, 0, 0); > + ImageList_Destroy(info->icons); > + DestroyWindow(info->back_btn_hwnd); > + DestroyWindow(info->fwd_btn_hwnd); The system will automatically destroy child windows, so I don't think this is needed. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1857#note_20174

1 0

Re: [PATCH v4 0/6] MR1857: Draft: comdlg32: IFileDialog navigation bar
by Esme Povirk (＠madewokherd) 28 Dec '22

28 Dec '22

Esme Povirk (@madewokherd) commented about dlls/comdlg32/navbar.c: > + PAINTSTRUCT ps; > + HDC hdc = BeginPaint(hwnd, &ps); > + RECT rc; > + HBRUSH old_brush; > + HBRUSH bg_brush = GetSysColorBrush(COLOR_BTNFACE); > + HBRUSH frame_brush = GetSysColorBrush(COLOR_3DSHADOW); > + > + GetClientRect(hwnd, &rc); > + > + /* clear background to avoid border trailing when resizing */ > + FillRect(hdc, &rc, bg_brush); > + > + /* draw the 1px frame */ > + FrameRect(hdc, &rc, frame_brush); > + > + /* erase the left border, retaining the top and bottom pixels */ I think you could remove the need for this by drawing the left border outside the window. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1857#note_20173

1 0