Hello,
AUI text is unreadable in high contrast mode. I fixed it in https://github.com/wxWidgets/wxWidgets/commit/697bd0744159e8c0ded7fb4174eecbfa41700e7c. Could you please consider applying this patch to wxWidgets when you will do the next release?
I have been looking the patch for src/aui/barartmsw.cpp, and the text color for disabled buttons is still hardcoded in two methods (DrawButton and DrawDropDownButton). Also, for disabled buttons you set the color twice. I think
dc.SetTextForeground(wxSystemSettings::GetColour(wxSYS_COLOUR_CAPTIONTEXT));
if ( item.GetState() & wxAUI_BUTTON_STATE_DISABLED )
dc.SetTextForeground(DISABLED_TEXT_COLOR);
should be
if ( item.GetState() & wxAUI_BUTTON_STATE_DISABLED )
dc.SetTextForeground(wxSystemSettings::GetColour(wxSYS_COLOUR_INACTIVECAPTIONTEXT));
else
dc.SetTextForeground(wxSystemSettings::GetColour(wxSYS_COLOUR_CAPTIONTEXT));
and this part can be removed
static const unsigned char
DISABLED_TEXT_GREY_HUE = wxColour::AlphaBlend(0, 255, 0.4);
const wxColour DISABLED_TEXT_COLOR(DISABLED_TEXT_GREY_HUE,
DISABLED_TEXT_GREY_HUE,
DISABLED_TEXT_GREY_HUE);
Thanks. I will look at this.
I would actually use `wxSYS_COLOUR_GRAYTEXT` instead of `wxSYS_COLOUR_INACTIVECAPTIONTEXT`.
Quote from: mimi89999 on April 15, 2020, 12:20:07 PM
Could you please consider applying this patch to wxWidgets when you will do the next release?
Only if they make an actual release. If not we'll pick only things which prevent crashes...