Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix warning background "sticky" behavior #387

Merged
merged 1 commit into from
Dec 10, 2023

Conversation

sergeyvfx
Copy link
Contributor

After a warning error has been logged in the LimeSuiteGUI all the following messages had an yellow background, which is an unexpected behavior.

Apparently it was caused by the previous fix in the area which made it so the text color is restored to its default value. Somehow it had an effect on the background color behavior: as if there is some implicit behavior when neither of the foreground and background are set in the style when it is set as default to the message field.

It is hard to tell now if this was an oversight in the previous fix, or is it something changed in the wxWidgets library. In any case the logging behavior has been verified once again on both Dark and Light themes on macOS.

After a warning error has been logged in the LimeSuiteGUI all the following
messages had an yellow background, which is an unexpected behavior.

Apparently it was caused by the previous fix in the area which made it so the
text color is restored to its default value. Somehow it had an effect on the
background color behavior: as if there is some implicit behavior when neither
of the foreground and background are set in the style when it is set as default
to the message field.

It is hard to tell now if this was an oversight in the previous fix, or is it
something changed in the wxWidgets library. In any case the logging behavior
has been verified once again on both Dark and Light themes on macOS.
@rjonaitis rjonaitis merged commit cc09db1 into myriadrf:master Dec 10, 2023
4 checks passed
@sergeyvfx sergeyvfx deleted the log_background branch December 30, 2023 13:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants