https://bugs.kde.org/show_bug.cgi?id=404833
--- Comment #23 from RJVB <rjvber...@gmail.com> --- >It's not that you're not allowed to, it can. not. be. done. If Qt were The only thing one can affirm here is that it cannot be done the same way on all platforms and possibly that there are platforms where it is entirely impossible to add anything to the native file widget. One can probably also say that on most platforms this might require changing the QPA rather than a QPT. But we're not talking about those platforms here. >Look at xdg-desktop-portal, it uses a QDialog, slaps a KFileWidget into it, >wires up the QDialogButtonBox to interact with it, and done. 10 lines of code. How is that relevant? >As you said, it's a hack And yet it's what you'd do if Qt gave access to the dialog that is actually being shown ... or if they hadn't gone with this sort of phantom sibling dialog situation (can we agree that's a hack in itself?). >that stuff in a platform-independent way, you might as well do it properly and >use the correct APIs directly rather than relying on some magic properties set >by some plugin. Again, that's like saying the preview widget shouldn't be there because it cannot be provided on all platforms, and so if code wants to rely on that feature it needs to implement it itself. Instead, the KDE QPT implements it, presumable because it's seen as a potential convenience for applications and users that somehow get to use the KDE file dialog. I don't see why that couldn't apply to support for embedding. As to using the KDE APIs directly: you'll notice that I followed that approach before filing this ticket, and that it took considerably more than 10 lines. Every application would have to duplicate that (which is an already cited argument for bringing back a complete KDE file dialog class). You'll notice too that this is not acceptable for at least 1 of the Scribus people as just installing KIO requires about 100 additional packages. That may not sound like a problem for anyone who already has KDE installed ... but if you do you probably also have the plasma integration plugin installed. -- You are receiving this mail because: You are watching all bug changes.
