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

Workaround broken inactive window coloring applies incorrectly to list/tree/dock widgets in the active window #46847

Merged
merged 2 commits into from
Jan 16, 2022

Conversation

nyalldawson
Copy link
Collaborator

This is a theming issue which is apparent on certain default
Linux themes, e.g. the when running QGIS under gnome. It results
in incorrectly deciding that a widget belongs to an inactive
window, causing unfocused list/tree or dock widgets to incorrectly
show in the inactive color palette. On gnome at least this
"inactive" palette defaults to an appearance very similar to
the disabled widget color palette. As a result all unfocused
list/tree/dock widgets look to be disabled until they are focused.

This gross hack just force removes the inactive color palette
under linux environments to block this theme behaviour

Refs FedoraQt/adwaita-qt#126

Without fix: Notice the "disabled" looking text in the browser and the symbols library list in the dock

image

With fix:

image

to list/tree/dock widgets in the active window

This is a theming issue which is apparent on certain default
Linux themes, e.g. the when running QGIS under gnome. It results
in incorrectly deciding that a widget belongs to an inactive
window, causing unfocused list/tree or dock widgets to incorrectly
show in the inactive color palette. On gnome at least this
"inactive" palette defaults to an appearance very similar to
the disabled widget color palette. As a result all unfocused
list/tree/dock widgets look to be disabled until they are focused.

This gross hack just force removes the inactive color palette
under linux environments to block this theme behaviour
Copy link
Contributor

@nirvn nirvn left a comment

Choose a reason for hiding this comment

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

Well that's ugly, and yet status quo is even uglier, so +1.

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.

None yet

2 participants