Skip to content

wx.lib.inspection dark mode issues [Issue #2917] - #2925

Open
CEXT-Dan wants to merge 8 commits into
wxWidgets:masterfrom
CEXT-Dan:issue2917
Open

wx.lib.inspection dark mode issues [Issue #2917]#2925
CEXT-Dan wants to merge 8 commits into
wxWidgets:masterfrom
CEXT-Dan:issue2917

Conversation

@CEXT-Dan

Copy link
Copy Markdown

this commit partially fixes the dark mode issues in wx.lib.inspection
by making the editor window readable when in dark mode

Fixes #2917

this commit partially fixes the dark mode issues by making the
editor window readable when in dark mode
@CEXT-Dan

Copy link
Copy Markdown
Author

sorry, actually, this does not work with the sample I posted in #2917

@CEXT-Dan

Copy link
Copy Markdown
Author

now the sample works

Comment thread wx/py/editwindow.py Outdated
Comment thread wx/py/editwindow.py Outdated
Comment thread wx/py/editwindow.py Outdated
try:
is_dark = wx.SystemSettings.GetAppearance().IsDark()
except wx.PyNoAppError:
is_dark = False

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we fix this for platforms other than MSW as well? Maybe factor out the color setting to be for all platforms, since the colors are the same for all platforms?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks like you tried to address this, but it seems that you need to move the is_dark definition outside of if 'wxMSW' in wx.PlatformInfo

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oof, I'll fix that

Comment thread wx/py/editwindow.py
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.

wx.lib.inspection dark mode issues.

2 participants