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

fix: allow use of qt window decorations #235

Merged
merged 1 commit into from
Jun 24, 2024

Conversation

Merrit
Copy link
Contributor

@Merrit Merrit commented May 31, 2024

Currently, Flutter defaults to always using GTK window decorations regardless of
the window manager, when running Wayland. This makes the app look out of place
on KDE Plasma and other Qt-based desktop environments.

This change checks the XDG_CURRENT_DESKTOP environment variable to determine the
running desktop environment, and if it is GNOME continues to use the GTK header,
otherwise it uses the Qt header.

Before After
image image

Now in the after screenshot, the window has native decorations on KDE with user-customized controls for things like pin to all workspaces.

Currently, Flutter defaults to always using GTK
window decorations regardless of the window
manager, when running Wayland. This makes the app
look out of place on KDE Plasma and other Qt-based
desktop environments.

This change checks the XDG_CURRENT_DESKTOP
environment variable to determine the running
desktop environment, and if it is GNOME continues
to use the GTK header, otherwise it uses the Qt
header.
@anandnet anandnet merged commit 4c8f96f into anandnet:main Jun 24, 2024
1 check passed
@anandnet
Copy link
Owner

@Merrit thanks for the contribution.

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