Dark mode: themed message boxes and dark scrollbars

Address PR #1203 feedback: scroll bars and the save-changes dialog were rendered in the light theme even when dark mode is active.

- Enable scroll bar fix in darkmodelib (mode 2: per-window) and rely on
  the existing enableDarkScrollBarForWindowAndChildren(hwndMain) call so
  the Scintilla edit window's scroll bars use the dark theme.
- Route Notepad4's MsgBox() through dmlib::darkMessageBoxW(), which
  renders message boxes as themed task dialogs when dark mode is on.
- Fix two upstream darkmodelib namespacing bugs that surface when
  _DARKMODELIB_USE_SCROLLBAR_FIX is defined (DmlibHook.cpp uses
  ModuleHandle / LoadFn without the dmlib_module:: prefix).

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: lostindark

build/VisualStudio/Notepad4.vcxproj

@@ -158,7 +158,7 @@
   <ItemDefinitionGroup>
     <ClCompile>
       <AdditionalIncludeDirectories>..\..\scintilla\include;..\..\scintilla\lexlib;..\..\scintilla\src;..\..\src;..\..\darkmodelib;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
-      <PreprocessorDefinitions>_WINDOWS;NOMINMAX;WIN32_LEAN_AND_MEAN;STRICT_TYPED_ITEMIDS;BOOST_REGEX_STANDALONE;NO_CXX11_REGEX;UNICODE;_UNICODE;_CRT_SECURE_NO_WARNINGS;_SCL_SECURE_NO_WARNINGS;_DARKMODELIB_NO_INI_CONFIG;%(PreprocessorDefinitions)</PreprocessorDefinitions>
+      <PreprocessorDefinitions>_WINDOWS;NOMINMAX;WIN32_LEAN_AND_MEAN;STRICT_TYPED_ITEMIDS;BOOST_REGEX_STANDALONE;NO_CXX11_REGEX;UNICODE;_UNICODE;_CRT_SECURE_NO_WARNINGS;_SCL_SECURE_NO_WARNINGS;_DARKMODELIB_NO_INI_CONFIG;_DARKMODELIB_USE_SCROLLBAR_FIX=2;%(PreprocessorDefinitions)</PreprocessorDefinitions>
       <WarningLevel>Level4</WarningLevel>
       <LanguageStandard>stdcpplatest</LanguageStandard>
       <LanguageStandard_C>stdc17</LanguageStandard_C>

build/mingw/notepad4.mk

@@ -14,7 +14,7 @@ INCDIR = \
 	-I"$(scintilla_dir)/include" \
 	-I"$(darkmodelib_dir)"
 
-CPPFLAGS += -D_DARKMODELIB_NO_INI_CONFIG
+CPPFLAGS += -D_DARKMODELIB_NO_INI_CONFIG -D_DARKMODELIB_USE_SCROLLBAR_FIX=2
 
 LDFLAGS += -L"$(BINFOLDER)/obj"
 

darkmodelib/DmlibHook.cpp

@@ -168,7 +168,7 @@ static fnOpenNcThemeData pfOpenNcThemeData = nullptr;
 
 bool dmlib_hook::loadOpenNcThemeData(const HMODULE& hUxtheme) noexcept
 {
-	return LoadFn(hUxtheme, pfOpenNcThemeData, 49);
+	return dmlib_module::LoadFn(hUxtheme, pfOpenNcThemeData, 49);
 }
 
 #if defined(_DARKMODELIB_USE_SCROLLBAR_FIX) && (_DARKMODELIB_USE_SCROLLBAR_FIX > 1)
@@ -229,7 +229,7 @@ static HTHEME WINAPI MyOpenNcThemeData(HWND hWnd, LPCWSTR pszClassList)
 
 void dmlib_hook::fixDarkScrollBar()
 {
-	const ModuleHandle moduleComctl(L"comctl32.dll");
+	const dmlib_module::ModuleHandle moduleComctl(L"comctl32.dll");
 	if (moduleComctl.isLoaded())
 	{
 		auto* addr = iat_hook::FindDelayLoadThunkInModule(moduleComctl.get(), "uxtheme.dll", 49); // OpenNcThemeData

src/DarkMode.cpp

@@ -120,3 +120,14 @@ bool DarkMode_HandleSettingChange([[maybe_unused]] HWND hwnd, LPARAM lParam) noe
 bool DarkMode_IsEnabled() noexcept {
 	return dmlib::isExperimentalActive();
 }
+
+int DarkMode_MessageBox(HWND hwnd, LPCWSTR text, LPCWSTR caption, UINT uType, WORD wLanguageId) noexcept {
+	if (dmlib::isExperimentalActive()) {
+		const HRESULT hr = dmlib::darkMessageBoxW(hwnd, text, caption, uType);
+		if (hr > 0) {
+			return static_cast<int>(hr);
+		}
+		// Fall through to MessageBoxEx on failure.
+	}
+	return MessageBoxEx(hwnd, text, caption, uType, wLanguageId);
+}

src/DarkMode.h

@@ -28,3 +28,7 @@ bool DarkMode_HandleSettingChange(HWND hwnd, LPARAM lParam) noexcept;
 
 // Check if dark mode UI is currently active.
 bool DarkMode_IsEnabled() noexcept;
+
+// Show a message box that respects dark mode (uses Task Dialog when dark mode is active).
+// Returns the same kind of value as MessageBoxEx (IDOK, IDYES, IDNO, IDCANCEL, ...).
+int DarkMode_MessageBox(HWND hwnd, LPCWSTR text, LPCWSTR caption, UINT uType, WORD wLanguageId) noexcept;

src/Dialogs.cpp

@@ -34,6 +34,7 @@
 #include "Styles.h"
 #include "Dlapi.h"
 #include "Dialogs.h"
+#include "DarkMode.h"
 #include "resource.h"
 #include "Version.h"
 
@@ -114,7 +115,7 @@ int MsgBox(UINT uType, UINT uIdMsg, ...) noexcept {
 
 	HWND hwnd = GetMsgBoxParent();
 	PostMessage(hwndMain, APPM_CENTER_MESSAGE_BOX, AsInteger<WPARAM>(hwnd), 0);
-	return MessageBoxEx(hwnd, szText, szTitle, uType, lang);
+	return DarkMode_MessageBox(hwnd, szText, szTitle, uType, lang);
 }
 
 //=============================================================================