Dialogs, QPT and Qt5

[Kevin Ottens]

Hello,

On Thursday 28 November 2013 17:53:14 Aleix Pol wrote:
> I've seen that when we ask for a QFileDialog in KF5 we get a generic
> QFileDialog at the moment. The reason is simple, KFileDialog is deprecated
> in KDE4Support and we are not implementing
> QPlatformDialogHelper *QPlatformThemecreatePlatformDialogHelper(DialogType
> dialogType).

*And* KFileDialogQtOverride got moved in kde4support too. That's the reason 
why we used to get a KFileDialog for the static calls previously. Got broken 
during the move to kde4support.

We shouldn't use that technique anymore though (covers only the static case 
and is rather intrusive IMO). So using a QPlatformDialogHelper is definitely 
the way forward.

> I would like to make it happen, my only question is then what should we do?
> Should we have a framework for KFileDialog, KColorDialog and KFontDialog?

KColorDialog and KFontDialog are out of the picture. We contributed to the 
upstream QColorDialog and QFontDialog so that they have the features we needed 
out of the box. We only need to do something for the file dialog.

> Should we fork them and get  a copy in PlatformIntegration?

Yes, we need something in the platform theme plugin. Be it a full fledged 
KFileDialog fork or a leaner QDialog which contains a KFileWidget I let you 
judge. I've a slight preference for composition (QDialog + KFileWidget), 
especially as the resulting class will be completely internal with no public 
API.

Cheers.
-- 
Kévin Ottens, http://ervin.ipsquad.net

KDAB - proud supporter of KDE, http://www.kdab.com

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20131129/9a3182b3/attachment.sig>