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

WIP: Add dark detection #10510

Closed
wants to merge 1 commit into from
Closed

WIP: Add dark detection #10510

wants to merge 1 commit into from

Conversation

larsoner
Copy link
Member

@larsoner larsoner commented Apr 9, 2022

Pointed out by @alexrockhill our examples complain about darkdetect. Let's add it to requirements.txt, which should get picked up by CircleCI.

@larsoner larsoner added the backport-candidate on-merge: backport to maint/1.7 label Apr 9, 2022
@larsoner
Copy link
Member Author

larsoner commented Apr 9, 2022

Argh, looks like PySide6/PyQt6 is not supported by QDarkStyle yet:

ColinDuquesnoy/QDarkStyleSheet#249

I'll modify our automatic theme detection code...

@larsoner larsoner mentioned this pull request Apr 9, 2022
1 task
@larsoner larsoner changed the title MAINT: Add dark detection WIP: Add dark detection Apr 9, 2022
@larsoner
Copy link
Member Author

larsoner commented Apr 9, 2022

This should go in after #10500. The logic in these two overlap and it'll be easier to fix here rather than there

@larsoner
Copy link
Member Author

larsoner commented Apr 9, 2022

Closing for #10509

@larsoner larsoner closed this Apr 9, 2022
@larsoner larsoner deleted the circle3 branch April 9, 2022 17:32
@hoechenberger hoechenberger removed the backport-candidate on-merge: backport to maint/1.7 label May 2, 2022
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