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

Apply dark theme on Windows if user enabled it in system settings. #1169

Merged
merged 1 commit into from Sep 21, 2020

Conversation

Slion
Copy link
Contributor

@Slion Slion commented Sep 15, 2020

No description provided.

@Slion Slion requested a review from sthalik September 15, 2020 15:37
@Slion Slion self-assigned this Sep 15, 2020
@Slion Slion added the feature label Sep 15, 2020
@ghost
Copy link

ghost commented Sep 17, 2020

Yes, I would like to have dark mode because I can see for example the opentrack on second screen while playing game. The light mode glows a little bit to bright. You could provide a simple .xml file with color properties and we, the users, will do the rest 👍

gui/init.cpp Outdated
@@ -269,6 +269,38 @@ int otr_main(int argc, char** argv, std::function<std::unique_ptr<QWidget>()> co

QApplication app(argc, argv);

#ifdef Q_OS_WIN
Copy link
Member

Choose a reason for hiding this comment

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

Use #ifdef _WIN32. Put in its own function, like maybe_apply_dark_windows_theme(). Confirm that mapping widget works.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

@sthalik sthalik merged commit 3cc3ef2 into unstable Sep 21, 2020
@sthalik
Copy link
Member

sthalik commented Sep 21, 2020

LGTM. 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

None yet

2 participants