Skip to content

Stop forcing Qt style override - #6047

Closed
crmne wants to merge 1 commit into
basecamp:devfrom
crmne:cp/drop-qt-style-override
Closed

Stop forcing Qt style override#6047
crmne wants to merge 1 commit into
basecamp:devfrom
crmne:cp/drop-qt-style-override

Conversation

@crmne

@crmne crmne commented Jun 6, 2026

Copy link
Copy Markdown
Contributor

Summary

Remove the global QT_STYLE_OVERRIDE=kvantum export from the default Hyprland environment.

QT_QPA_PLATFORM still defaults Qt apps to Wayland/XCB; this only stops forcing the Qt style for every Qt app.

Why

Forcing QT_STYLE_OVERRIDE globally prevents Qt settings tools and individual Qt apps from applying their configured style. Omarchy can set the platform default without making the visual Qt style a session-wide override.

Testing

  • lua -e 'assert(loadfile("default/hypr/envs.lua"))'
  • git diff --check origin/dev..cp/drop-qt-style-override

Copilot AI review requested due to automatic review settings June 6, 2026 10:20

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Removes a Hyprland Qt environment override that forced the kvantum style, likely to avoid globally overriding Qt theming.

Changes:

  • Removed QT_STYLE_OVERRIDE=kvantum from the Hyprland environment configuration.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@dhh

dhh commented Jun 8, 2026

Copy link
Copy Markdown
Member

Going to need some before/after screenshots here too.

@vander00

Copy link
Copy Markdown
Contributor

I think the sense is all the apps to look the same, and it adds the feature to change all the QT theme on a system just with kvantum.

@dhh

dhh commented Jul 20, 2026

Copy link
Copy Markdown
Member

Happy with the uniform style we have now.

@dhh dhh closed this Jul 20, 2026
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.

4 participants