[PATCH v2 0/2] MR3230: comdlg32: Add parent folder button to IFileDialog

[Jacob Czekalla (＠Maaka00524)]

Fixes bug: https://bugs.winehq.org/show_bug.cgi?id=54812

--
  v2: comdlg32: computes the parent arrow icon index

https://gitlab.winehq.org/wine/wine/-/merge_requests/3230

[PATCH v2 1/2] comdlg32: Add parent folder button to IFileDialog

[Jacob Czekalla]

From: Jacob Czekalla

---
 dlls/comdlg32/itemdlg.c | 22 ++++++++++++++++++++--
 include/commctrl.h      |  1 +
 2 files changed, 21 insertions(+), 2 deletions(-)

diff --git a/dlls/comdlg32/itemdlg.c b/dlls/comdlg32/itemdlg.c
index 1feb1deda00..4a030575230 100644
--- a/dlls/comdlg32/itemdlg.c
+++ b/dlls/comdlg32/itemdlg.c
@@ -40,6 +40,7 @@
 #define IDC_NAV_TOOLBAR      200
 #define IDC_NAVBACK          201
 #define IDC_NAVFORWARD       202
+#define IDC_NAVUP            203

#include <initguid.h>
 /* This seems to be another version of IID_IFileDialogCustomize. If
@@ -1895,7 +1896,7 @@ static void init_toolbar(FileDialogImpl *This, HWND hwnd)
 {
     HWND htoolbar;
     TBADDBITMAP tbab;
-    TBBUTTON button[2];
+    TBBUTTON button[3];
     int height;

htoolbar = CreateWindowExW(0, TOOLBARCLASSNAMEW, NULL, TBSTYLE_FLAT | WS_CHILD | WS_VISIBLE,
@@ -1905,6 +1906,8 @@ static void init_toolbar(FileDialogImpl *This, HWND hwnd)
     tbab.hInst = HINST_COMMCTRL;
     tbab.nID = IDB_HIST_LARGE_COLOR;
     SendMessageW(htoolbar, TB_ADDBITMAP, 0, (LPARAM)&tbab);
+    tbab.nID = IDB_VIEW_LARGE_COLOR;
+    SendMessageW(htoolbar, TB_ADDBITMAP, 0, (LPARAM)&tbab);

button[0].iBitmap = HIST_BACK;
     button[0].idCommand = IDC_NAVBACK;
@@ -1920,7 +1923,14 @@ static void init_toolbar(FileDialogImpl *This, HWND hwnd)
     button[1].dwData = 0;
     button[1].iString = 0;

-    SendMessageW(htoolbar, TB_ADDBUTTONSW, 2, (LPARAM)button);
+    button[2].iBitmap = HIST_UP;
+    button[2].idCommand = IDC_NAVUP;
+    button[2].fsState = TBSTATE_ENABLED;
+    button[2].fsStyle = BTNS_BUTTON;
+    button[2].dwData = 0;
+    button[2].iString = 0;
+
+    SendMessageW(htoolbar, TB_ADDBUTTONSW, 3, (LPARAM)button);
     height = MulDiv(24, This->dpi_y, USER_DEFAULT_SCREEN_DPI);
     SendMessageW(htoolbar, TB_SETBUTTONSIZE, 0, MAKELPARAM(height, height));
     SendMessageW(htoolbar, TB_AUTOSIZE, 0, 0);
@@ -2211,6 +2221,13 @@ static LRESULT on_command_filetype(FileDialogImpl *This, WPARAM wparam, LPARAM l
     return FALSE;
 }

+static LRESULT on_browse_up(FileDialogImpl *This)
+{
+    TRACE("%p\n", This);
+    IExplorerBrowser_BrowseToIDList(This->peb, NULL, SBSP_PARENT);
+    return FALSE;
+}
+
 static LRESULT on_wm_command(FileDialogImpl *This, WPARAM wparam, LPARAM lparam)
 {
     switch(LOWORD(wparam))
@@ -2221,6 +2238,7 @@ static LRESULT on_wm_command(FileDialogImpl *This, WPARAM wparam, LPARAM lparam)
     case IDC_NAVBACK:         return on_browse_back(This);
     case IDC_NAVFORWARD:      return on_browse_forward(This);
     case IDC_FILETYPE:        return on_command_filetype(This, wparam, lparam);
+    case IDC_NAVUP:           return on_browse_up(This);
     default:                  TRACE("Unknown command.\n");
     }
     return FALSE;
diff --git a/include/commctrl.h b/include/commctrl.h
index a54de13d8b2..b4653fa8fad 100644
--- a/include/commctrl.h
+++ b/include/commctrl.h
@@ -1472,6 +1472,7 @@ typedef struct tagTBADDBITMAP {
 #define HIST_FAVORITES          2
 #define HIST_ADDTOFAVORITES     3
 #define HIST_VIEWTREE           4
+#define HIST_UP                 13

typedef struct tagTBSAVEPARAMSA {
     HKEY   hkr;

[PATCH v2 2/2] comdlg32: computes the parent arrow icon index

[Jacob Czekalla]

From: Jacob Czekalla

---
 dlls/comdlg32/itemdlg.c | 6 ++++--
 include/commctrl.h      | 1 -
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/dlls/comdlg32/itemdlg.c b/dlls/comdlg32/itemdlg.c
index 4a030575230..f7c5b17a7dd 100644
--- a/dlls/comdlg32/itemdlg.c
+++ b/dlls/comdlg32/itemdlg.c
@@ -1898,6 +1898,7 @@ static void init_toolbar(FileDialogImpl *This, HWND hwnd)
     TBADDBITMAP tbab;
     TBBUTTON button[3];
     int height;
+    int navUpImgIndex;

htoolbar = CreateWindowExW(0, TOOLBARCLASSNAMEW, NULL, TBSTYLE_FLAT | WS_CHILD | WS_VISIBLE,
                                0, 0, 0, 0,
@@ -1907,7 +1908,8 @@ static void init_toolbar(FileDialogImpl *This, HWND hwnd)
     tbab.nID = IDB_HIST_LARGE_COLOR;
     SendMessageW(htoolbar, TB_ADDBITMAP, 0, (LPARAM)&tbab);
     tbab.nID = IDB_VIEW_LARGE_COLOR;
-    SendMessageW(htoolbar, TB_ADDBITMAP, 0, (LPARAM)&tbab);
+    navUpImgIndex = SendMessageW(htoolbar, TB_ADDBITMAP, 0, (LPARAM)&tbab);
+    navUpImgIndex += VIEW_PARENTFOLDER;

button[0].iBitmap = HIST_BACK;
     button[0].idCommand = IDC_NAVBACK;
@@ -1923,7 +1925,7 @@ static void init_toolbar(FileDialogImpl *This, HWND hwnd)
     button[1].dwData = 0;
     button[1].iString = 0;

-    button[2].iBitmap = HIST_UP;
+    button[2].iBitmap = navUpImgIndex;
     button[2].idCommand = IDC_NAVUP;
     button[2].fsState = TBSTATE_ENABLED;
     button[2].fsStyle = BTNS_BUTTON;
diff --git a/include/commctrl.h b/include/commctrl.h
index b4653fa8fad..a54de13d8b2 100644
--- a/include/commctrl.h
+++ b/include/commctrl.h
@@ -1472,7 +1472,6 @@ typedef struct tagTBADDBITMAP {
 #define HIST_FAVORITES          2
 #define HIST_ADDTOFAVORITES     3
 #define HIST_VIEWTREE           4
-#define HIST_UP                 13

typedef struct tagTBSAVEPARAMSA {
     HKEY   hkr;