-
Notifications
You must be signed in to change notification settings - Fork 38
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
Remove bundling Gtk platform plugins #109
Comments
What is the side effect of removing them? |
"Native look" may be gone. But that is overrated anyway. Maybe we can just bundle some standard style like Fusion and make apps use that as a default (unless they change it themselves). Some apps deliberately delete these files already. |
Only on Gtk based desktops like Gnome, not on Qt based ones like KDE Plasma. |
Even Xfce seems to use Fusion nowadays, not Gtk+, at least for many AppImages I came across. As said, "native look" is overrated... I think that Gtk+ platform theme bundling stuff was and still is experimental and has shown to cause more issues than it solves. I'll remove them soon. |
It's definitely a loss to have this gone, native look is not overrated and very much desired. It's one of Qt's biggest weakness on Linux that it does not integrate well with the users visual environment, unless they happen to be on KDE. |
Not really a loss since it was not working and caused a lot of trouble for many people over the years. Just look at the many bug reports linked above. What is worse - the application not working or the application not looking like Gtk applications? |
I haven't seen such reports from my users. Please recommend a method to undo this as an opt-in. |
@vadi2 if you send a PR to make this opt-in, I'll happily merge it. The |
… (#538) linuxdeploy/linuxdeploy-plugin-qt#109 Thanks Jean Ghali
linuxdeploy-plugin-qt/src/deployers/PlatformPluginsDeployer.cpp
Lines 56 to 62 in 43c69b0
Apparently I had been too optimistic at that time.
Bundling Gtk platform plugins created lots of hassle, including
Please remove the bundling of those.
The text was updated successfully, but these errors were encountered: