Re: [PATCH v2 0/2] MR2183: winecfg: Add an option to enable WinRT app dark theme.

[Zhiyi Zhang (＠zhiyi)]

Zhiyi Zhang (@zhiyi) commented about programs/winecfg/winecfg.rc:
> FONT 8, "MS Shell Dlg"
>  BEGIN
>      GROUPBOX        "Appearance",IDC_STATIC,8,4,244,90
> - LTEXT           "&Theme:",IDC_STATIC,15,15,130,8
> - COMBOBOX        IDC_THEME_THEMECOMBO,15,25,130,60,CBS_DROPDOWNLIST | WS_VSCROLL | WS_TABSTOP
> - PUSHBUTTON      "&Install theme...",IDC_THEME_INSTALL,152,25,93,13
> - LTEXT           "System &Theme:",IDC_STATIC,15,15,70,8
> - COMBOBOX        IDC_THEME_THEMECOMBO,15,25,70,60,CBS_DROPDOWNLIST | WS_VSCROLL | WS_TABSTOP
> - LTEXT           "App &Theme:",IDC_STATIC,90,15,70,8

I would prefer you keep the name for IDC_THEME_THEMECOMBO and use "WinRT app theme" for IDC_THEME_APPCOMBO. Because there is also a `SystemUsesLightTheme` registry value that controls whether system WinRT apps use a light theme and "System Theme" with "App Theme" makes it confusing.