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

Remove bundling Gtk platform plugins #109

Closed
probonopd opened this issue May 21, 2022 · 8 comments
Closed

Remove bundling Gtk platform plugins #109

probonopd opened this issue May 21, 2022 · 8 comments

Comments

@probonopd
Copy link

// according to probono, only the files shall be deployed, not their dependencies
// either loading succeeds, then the system Gtk shall be used anyway, otherwise loading fails and Qt will fall
// back to the default UI theme
// we don't care whether this works (it's an experimental feature), therefore we ignore the return values
const auto libqgtk2Path = platformThemesPath / "libqgtk2.so";
const auto libqgtk3Path = platformThemesPath / "libqgtk3.so";
const auto libqxdgPath = platformThemesPath / "libqxdgdesktopportal.so";

Apparently I had been too optimistic at that time.

Bundling Gtk platform plugins created lots of hassle, including

Please remove the bundling of those.

probonopd added a commit to AppImageCommunity/AppImageUpdate that referenced this issue May 21, 2022
@vadi2
Copy link

vadi2 commented May 21, 2022

What is the side effect of removing them?

@TheAssassin
Copy link
Member

"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.

@probonopd
Copy link
Author

"Native look" may be gone

Only on Gtk based desktops like Gnome, not on Qt based ones like KDE Plasma.

@TheAssassin
Copy link
Member

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.

@vadi2
Copy link

vadi2 commented May 22, 2022

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.

@probonopd
Copy link
Author

probonopd commented May 22, 2022

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?

@vadi2
Copy link

vadi2 commented May 22, 2022

I haven't seen such reports from my users. Please recommend a method to undo this as an opt-in.

@TheAssassin
Copy link
Member

@vadi2 if you send a PR to make this opt-in, I'll happily merge it. The PlatformPluginDeployer already shows how this needs to be done. I think if you, as the author, decide you want to enable such an experimental feature, that's completely acceptable. We just need to communicate it's an experiment, and give responsibility into the AppImage author's hands.

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

No branches or pull requests

3 participants