wxSearchCtrl not rendered correctly in wxQt (Issue #23170)
AliKet
Description
Due to this commit f2106dc (which can be safely reverted since wxGTK has its own implementation now anyway) the text control colour is black as can be seen in the image below:
Expected:
Platform and version information
- wxWidgets version you use: latest
- wxWidgets port you use: wxQt (Qt 5.5)
- OS and its version: Ubuntu 18.04
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
VZ
I agree that the other change doesn't seem necessary any more, but I also think that wxQt is wrong here as setting an invalid colour shouldn't set the background to black, but rather reset the background colour to the default value, i.e. undo the effect of any previous SetBackgroundColour() calls. Is it possible to implement this semantics in wxQt?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
dsa-t
Looks like this currently:
Screenshot_2024-02-13_21-09-28.png (view on web)
A check for invalid color can be added in wxQtDCImpl::SetBackground.
Or in wxWindowQt::SetBackgroundColour
But in order to get white color on the full area
But also the order in wxSearchCtrl::Create has to be reversed:
m_text->SetBackgroundColour(wxColour());
SetBackgroundColour( m_text->GetBackgroundColour() );
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
VZ
Closed #23170 as completed via 47b011c.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.