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

Enable dark title bar on windows when accent color is not used #8498

Merged
merged 1 commit into from
Oct 15, 2022

Conversation

droidmonkey
Copy link
Member

Screenshots

Testing strategy

Tested on Windows 11

Type of change

  • ✅ Bug fix (non-breaking change that fixes an issue)

@droidmonkey droidmonkey added user interface platform: Windows pr: backport pending Pull request yet to be backported to a previous release labels Sep 23, 2022
@droidmonkey droidmonkey added this to the v2.7.2 milestone Sep 23, 2022
@droidmonkey droidmonkey force-pushed the hotfix/windows-titlebar branch from fcf61db to 7e4db3f Compare September 23, 2022 18:32
@codecov-commenter
Copy link

codecov-commenter commented Sep 23, 2022

Codecov Report

Base: 64.41% // Head: 64.41% // Increases project coverage by +0.00% 🎉

Coverage data is based on head (7e4db3f) compared to base (eaa363d).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #8498   +/-   ##
========================================
  Coverage    64.41%   64.41%           
========================================
  Files          339      339           
  Lines        43953    43953           
========================================
+ Hits         28309    28310    +1     
+ Misses       15644    15643    -1     
Impacted Files Coverage Δ
src/core/FileWatcher.cpp 86.75% <0.00%> (+1.20%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@phoerious
Copy link
Member

I honestly hate the black titlebar. 😒

@droidmonkey
Copy link
Member Author

You can always just delete the qt.conf file in your installation 😉 I am going to merge this for the sake of the discussion, also enabling the accent color in title bars ignores this feature anyway.

@droidmonkey droidmonkey merged commit 56307e6 into develop Oct 15, 2022
@droidmonkey droidmonkey deleted the hotfix/windows-titlebar branch October 15, 2022 21:37
@droidmonkey droidmonkey added pr: backported Pull request backported to previous release and removed pr: backport pending Pull request yet to be backported to a previous release labels Oct 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
platform: Windows pr: backported Pull request backported to previous release user interface
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make the titlebar dark when dark mode is enabled (Windows)
3 participants